From 9a32a7b40442e54ee6d8e2403e0f9a154dc4c77f Mon Sep 17 00:00:00 2001 From: Michael Sevakis Date: Sat, 18 Feb 2012 04:50:33 -0500 Subject: Split CPU-optimized beep code into the firmware/asm tree. For now due to current lack of an apps/asm, place the ASM/generic code in the firmware/asm directory. Additionally, make generic beep code more generic. Change-Id: I4a69b6ffcbb97d9e6dfde2209c5a118de19e5638 --- apps/beep.c | 80 +++++++++++++++---------------------------------------------- 1 file changed, 19 insertions(+), 61 deletions(-) (limited to 'apps/beep.c') diff --git a/apps/beep.c b/apps/beep.c index a6244d932c..79c7f1d2bc 100644 --- a/apps/beep.c +++ b/apps/beep.c @@ -26,71 +26,22 @@ #include "pcm_mixer.h" #include "misc.h" -static int32_t beep_phase; /* Phase of square wave generator */ +/** Beep generation, CPU optimized **/ +#include "../firmware/asm/beep.c" + +static uint32_t beep_phase; /* Phase of square wave generator */ static uint32_t beep_step; /* Step of square wave generator on each sample */ +#ifdef BEEP_GENERIC +static int16_t beep_amplitude; /* Amplitude of square wave generator */ +#else +/* Optimized routines do XOR with phase sign bit in both channels at once */ static uint32_t beep_amplitude; /* Amplitude of square wave generator */ +#endif static int beep_count; /* Number of samples remaining to generate */ /* Reserve enough static space for keyclick to fit */ #define BEEP_BUF_COUNT (NATIVE_FREQUENCY / 1000 * KEYCLICK_DURATION) -static uint32_t beep_buf[BEEP_BUF_COUNT] IBSS_ATTR; - -/* Actually output samples into beep_buf */ -#if defined(CPU_ARM) -static FORCE_INLINE void beep_generate(int count) -{ - uint32_t *out = beep_buf; - uint32_t s; - - asm volatile ( - "1: \n" - "eor %3, %5, %1, asr #31 \n" - "subs %2, %2, #1 \n" - "str %3, [%0], #4 \n" - "add %1, %1, %4 \n" - "bgt 1b \n" - : "+r"(out), "+r"(beep_phase), "+r"(count), - "=&r"(s) - : "r"(beep_step), "r"(beep_amplitude)); -} -#elif defined (CPU_COLDFIRE) -static FORCE_INLINE void beep_generate(int count) -{ - uint32_t *out = beep_buf; - uint32_t s; - - asm volatile ( - "1: \n" - "move.l %1, %3 \n" - "add.l %4, %1 \n" - "add.l %3, %3 \n" - "subx.l %3, %3 \n" - "eor.l %5, %3 \n" - "move.l %3, (%0)+ \n" - "subq.l #1, %2 \n" - "bgt.b 1b \n" - : "+a"(out), "+d"(beep_phase), "+d"(count), - "=&d"(s) - : "r"(beep_step), "d"(beep_amplitude)); -} -#else -static FORCE_INLINE void beep_generate(int count) -{ - uint32_t *out = beep_buf; - uint32_t amplitude = beep_amplitude; - uint32_t step = beep_step; - int32_t phase = beep_phase; - - do - { - *out++ = (phase >> 31) ^ amplitude; - phase += step; - } - while (--count > 0); - - beep_phase = phase; -} -#endif +static int16_t beep_buf[BEEP_BUF_COUNT*2] IBSS_ATTR __attribute__((aligned(4))); /* Callback to generate the beep frames - also don't want inlining of call below in beep_play */ @@ -104,8 +55,9 @@ beep_get_more(unsigned char **start, size_t *size) count = MIN(count, BEEP_BUF_COUNT); beep_count -= count; *start = (unsigned char *)beep_buf; - *size = count * sizeof(uint32_t); - beep_generate(count); + *size = count * 2 * sizeof (int16_t); + beep_generate((void *)beep_buf, count, &beep_phase, + beep_step, beep_amplitude); } } @@ -126,7 +78,13 @@ void beep_play(unsigned int frequency, unsigned int duration, beep_phase = 0; beep_step = 0xffffffffu / NATIVE_FREQUENCY * frequency; beep_count = NATIVE_FREQUENCY / 1000 * duration; + +#ifdef BEEP_GENERIC + beep_amplitude = amplitude; +#else + /* Optimized routines do XOR with phase sign bit in both channels at once */ beep_amplitude = amplitude | (amplitude << 16); /* Word:|AMP16|AMP16| */ +#endif /* If it fits - avoid cb overhead */ unsigned char *start; -- cgit v1.2.3