September 2017 - Sound-open-firmware - mailman.alsa-project.org

[Sound-open-firmware] [PATCH] SRC: Fix use of int and uint types and remove unused function parameters
by Seppo Ingalsuo 29 Sep '17

29 Sep '17

This patch fixes the warning messages shown with gcc option -Wextra. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo(a)linux.intel.com> --- src/audio/src.c | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/src/audio/src.c b/src/audio/src.c index 3fd9614..3723b4e 100644 --- a/src/audio/src.c +++ b/src/audio/src.c @@ -65,8 +65,7 @@ struct comp_data { void (*src_func)(struct comp_dev *dev, struct comp_buffer *source, struct comp_buffer *sink, - uint32_t source_frames, - uint32_t sink_frames); + int source_frames); }; /* Common mute function for 2s and 1s SRC. This preserves the same @@ -113,8 +112,7 @@ static void src_muted_s32(struct comp_buffer *source, struct comp_buffer *sink, static void fallback_s32(struct comp_dev *dev, struct comp_buffer *source, struct comp_buffer *sink, - uint32_t source_frames, - uint32_t sink_frames) + int source_frames) { struct comp_data *cd = comp_get_drvdata(dev); @@ -129,7 +127,7 @@ static void fallback_s32(struct comp_dev *dev, /* Normal 2 stage SRC */ static void src_2s_s32_default(struct comp_dev *dev, struct comp_buffer *source, struct comp_buffer *sink, - uint32_t source_frames, uint32_t sink_frames) + int source_frames) { int i; int j; @@ -203,7 +201,7 @@ static void src_2s_s32_default(struct comp_dev *dev, /* 1 stage SRC for simple conversions */ static void src_1s_s32_default(struct comp_dev *dev, struct comp_buffer *source, struct comp_buffer *sink, - uint32_t source_frames, uint32_t sink_frames) + int source_frames) { int i; int j; @@ -440,7 +438,7 @@ static int src_params(struct comp_dev *dev) * be too long. */ q = need.blk_out / dev->frames; - if (q * dev->frames < need.blk_out) + if (q * (int)dev->frames < need.blk_out) ++q; if (q * dev->frames < need.blk_out + dev->frames) @@ -543,9 +541,9 @@ static int src_copy(struct comp_dev *dev) need_sink = blk_out * dev->frame_bytes; /* Run as many times as buffers allow */ - while ((source->avail >= need_source) && (sink->free >= need_sink)) { + while (((int)source->avail >= need_source) && ((int)sink->free >= need_sink)) { /* Run src */ - cd->src_func(dev, source, sink, blk_in, blk_out); + cd->src_func(dev, source, sink, blk_in); /* calc new free and available */ comp_update_buffer_consume(source, 0); -- 2.11.0

2 5

[Sound-open-firmware] [PATCH] SRC: Fix use of int and uint types and remove unused function parameters
by Seppo Ingalsuo 29 Sep '17

29 Sep '17

This patch fixes the warning messages shown with gcc option -Wextra. The source and sink buffer pointers advance is moved from filter function to higher level copy function as update consume/produce parameter. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo(a)linux.intel.com> --- src/audio/src.c | 32 +++++++++++--------------------- src/audio/src_core.c | 4 ++-- src/audio/src_core.h | 4 ++-- 3 files changed, 15 insertions(+), 25 deletions(-) diff --git a/src/audio/src.c b/src/audio/src.c index 3fd9614..de84149 100644 --- a/src/audio/src.c +++ b/src/audio/src.c @@ -65,8 +65,7 @@ struct comp_data { void (*src_func)(struct comp_dev *dev, struct comp_buffer *source, struct comp_buffer *sink, - uint32_t source_frames, - uint32_t sink_frames); + int source_frames); }; /* Common mute function for 2s and 1s SRC. This preserves the same @@ -77,7 +76,6 @@ static void src_muted_s32(struct comp_buffer *source, struct comp_buffer *sink, { int i; - int32_t *src = (int32_t *) source->r_ptr; int32_t *dest = (int32_t *) sink->w_ptr; int32_t *end = (int32_t *) sink->end_addr; int n_read = 0; @@ -101,8 +99,6 @@ static void src_muted_s32(struct comp_buffer *source, struct comp_buffer *sink, n_read += nch*blk_in; n_written += nch*blk_out; } - source->r_ptr = src + n_read; - sink->w_ptr = dest; } /* Fallback function to just output muted samples and advance @@ -113,8 +109,7 @@ static void src_muted_s32(struct comp_buffer *source, struct comp_buffer *sink, static void fallback_s32(struct comp_dev *dev, struct comp_buffer *source, struct comp_buffer *sink, - uint32_t source_frames, - uint32_t sink_frames) + int source_frames) { struct comp_data *cd = comp_get_drvdata(dev); @@ -129,7 +124,7 @@ static void fallback_s32(struct comp_dev *dev, /* Normal 2 stage SRC */ static void src_2s_s32_default(struct comp_dev *dev, struct comp_buffer *source, struct comp_buffer *sink, - uint32_t source_frames, uint32_t sink_frames) + int source_frames) { int i; int j; @@ -196,14 +191,12 @@ static void src_2s_s32_default(struct comp_dev *dev, n_written += blk_out; } } - source->r_ptr = s1.x_rptr - nch + 1; - sink->w_ptr = s2.y_wptr - nch + 1; } /* 1 stage SRC for simple conversions */ static void src_1s_s32_default(struct comp_dev *dev, struct comp_buffer *source, struct comp_buffer *sink, - uint32_t source_frames, uint32_t sink_frames) + int source_frames) { int i; int j; @@ -254,8 +247,6 @@ static void src_1s_s32_default(struct comp_dev *dev, } } - source->r_ptr = s1.x_rptr - nch + 1; - sink->w_ptr = s1.y_wptr - nch + 1; } static struct comp_dev *src_new(struct sof_ipc_comp *comp) @@ -404,8 +395,7 @@ static int src_params(struct comp_dev *dev) /* Initize SRC for actual sample rate */ nch = MIN(params->channels, PLATFORM_MAX_CHANNELS); for (i = 0; i < nch; i++) { - n = src_polyphase_init(&cd->src[i], source_rate, sink_rate, - &need, buffer_start); + n = src_polyphase_init(&cd->src[i], &need, buffer_start); buffer_start += need.single_src; } @@ -440,10 +430,10 @@ static int src_params(struct comp_dev *dev) * be too long. */ q = need.blk_out / dev->frames; - if (q * dev->frames < need.blk_out) + if (q * (int)dev->frames < need.blk_out) ++q; - if (q * dev->frames < need.blk_out + dev->frames) + if (q * (int)dev->frames < need.blk_out + (int)dev->frames) ++q; /* Configure downstream buffer */ @@ -543,13 +533,13 @@ static int src_copy(struct comp_dev *dev) need_sink = blk_out * dev->frame_bytes; /* Run as many times as buffers allow */ - while ((source->avail >= need_source) && (sink->free >= need_sink)) { + while (((int) source->avail >= need_source) && ((int) sink->free >= need_sink)) { /* Run src */ - cd->src_func(dev, source, sink, blk_in, blk_out); + cd->src_func(dev, source, sink, blk_in); /* calc new free and available */ - comp_update_buffer_consume(source, 0); - comp_update_buffer_produce(sink, 0); + comp_update_buffer_consume(source, need_source); + comp_update_buffer_produce(sink, need_sink); } return 0; diff --git a/src/audio/src_core.c b/src/audio/src_core.c index 5b804d6..8098d87 100644 --- a/src/audio/src_core.c +++ b/src/audio/src_core.c @@ -269,8 +269,8 @@ void src_polyphase_reset(struct polyphase_src *src) src_state_reset(&src->state2); } -int src_polyphase_init(struct polyphase_src *src, int fs1, int fs2, - struct src_alloc *res, int32_t *delay_lines_start) +int src_polyphase_init(struct polyphase_src *src, struct src_alloc *res, + int32_t *delay_lines_start) { int n_stages; int ret; diff --git a/src/audio/src_core.h b/src/audio/src_core.h index 2965c7d..a9f66d4 100644 --- a/src/audio/src_core.h +++ b/src/audio/src_core.h @@ -130,8 +130,8 @@ static inline int src_polyphase_get_blk_out(struct polyphase_src *src) void src_polyphase_reset(struct polyphase_src *src); -int src_polyphase_init(struct polyphase_src *src, int fs1, int fs2, - struct src_alloc *res, int32_t *delay_lines_start); +int src_polyphase_init(struct polyphase_src *src, struct src_alloc *res, + int32_t *delay_lines_start); int src_polyphase(struct polyphase_src *src, int32_t x[], int32_t y[], int n_in); -- 2.11.0

2 1

[Sound-open-firmware] [PATCH v2] sof: check for missing prototypes
by Pierre-Louis Bossart 27 Sep '17

27 Sep '17

using gcc -Wmissing-prototypes helps identify a series of missing include files or static declarations. the -Wmissing-prototypes option is not enforced yet, there are remaining issues with the Tensilica HAL that will need to be fixed separately. To turn it on, modify configure.ac # General compiler flags -CFLAGS="${CFLAGS:+$CFLAGS } -O2 -g -Wall -Werror -Wl,-EL -fno-inline-functions -nostdlib" +CFLAGS="${CFLAGS:+$CFLAGS } -O2 -g -Wall -Werror -Wl,-EL -fno-inline-functions -nostdlib -Wmissing-prototypes" Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart(a)linux.intel.com> --- src/arch/xtensa/task.c | 1 + src/audio/src_core.c | 6 +++--- src/include/reef/init.h | 2 ++ src/include/reef/interrupt.h | 2 +- src/ipc/pmc-ipc.c | 1 + src/lib/lib.c | 1 + src/lib/schedule.c | 8 +++++--- src/math/numbers.c | 1 + src/math/trig.c | 1 + src/platform/baytrail/platform.c | 1 + src/platform/baytrail/timer.c | 2 +- 11 files changed, 18 insertions(+), 8 deletions(-) diff --git a/src/arch/xtensa/task.c b/src/arch/xtensa/task.c index 7e74c07..349f8ab 100644 --- a/src/arch/xtensa/task.c +++ b/src/arch/xtensa/task.c @@ -33,6 +33,7 @@ #include <reef/interrupt.h> #include <platform/platform.h> #include <reef/debug.h> +#include <arch/task.h> #include <stdint.h> #include <errno.h> diff --git a/src/audio/src_core.c b/src/audio/src_core.c index bce3586..5b804d6 100644 --- a/src/audio/src_core.c +++ b/src/audio/src_core.c @@ -52,21 +52,21 @@ int sof_rates[SOF_RATES_LENGTH] = {8000, 11025, 12000, 16000, 18900, 192000}; /* Calculates the needed FIR delay line length */ -int src_fir_delay_length(struct src_stage *s) +static int src_fir_delay_length(struct src_stage *s) { return s->subfilter_length + (s->num_of_subfilters - 1) * s->idm + s->blk_in; } /* Calculates the FIR output delay line length */ -int src_out_delay_length(struct src_stage *s) +static int src_out_delay_length(struct src_stage *s) { return (s->num_of_subfilters - 1) * s->odm + 1; } /* Returns index of a matching sample rate */ -int src_find_fs(int fs_list[], int list_length, int fs) +static int src_find_fs(int fs_list[], int list_length, int fs) { int i; diff --git a/src/include/reef/init.h b/src/include/reef/init.h index c2eaf87..9d97b48 100644 --- a/src/include/reef/init.h +++ b/src/include/reef/init.h @@ -38,4 +38,6 @@ int main(int argc, char *argv[]); int arch_init(struct reef *reef); +void __memmap_init(void); + #endif diff --git a/src/include/reef/interrupt.h b/src/include/reef/interrupt.h index 3c78e77..ea7d057 100644 --- a/src/include/reef/interrupt.h +++ b/src/include/reef/interrupt.h @@ -79,7 +79,7 @@ static inline void interrupt_global_enable(uint32_t flags) arch_interrupt_global_enable(flags); } -uint32_t platform_interrupt_get_enabled(); +uint32_t platform_interrupt_get_enabled(void); void platform_interrupt_clear(uint32_t irq, uint32_t mask); void platform_interrupt_mask(uint32_t irq, uint32_t mask); void platform_interrupt_unmask(uint32_t irq, uint32_t mask); diff --git a/src/ipc/pmc-ipc.c b/src/ipc/pmc-ipc.c index b8183c0..fdb8d4b 100644 --- a/src/ipc/pmc-ipc.c +++ b/src/ipc/pmc-ipc.c @@ -37,6 +37,7 @@ #include <reef/wait.h> #include <reef/trace.h> #include <platform/interrupt.h> +#include <platform/pmc.h> #include <platform/shim.h> diff --git a/src/lib/lib.c b/src/lib/lib.c index e1b1d9a..a1c9933 100644 --- a/src/lib/lib.c +++ b/src/lib/lib.c @@ -31,6 +31,7 @@ #include <stdint.h> #include <stdlib.h> #include <reef/reef.h> +#include <reef/alloc.h> #if 0 // TODO: only compile if no arch memcpy is available. diff --git a/src/lib/schedule.c b/src/lib/schedule.c index 142a2fb..427ca2e 100644 --- a/src/lib/schedule.c +++ b/src/lib/schedule.c @@ -159,7 +159,7 @@ static uint32_t sch_work(void *data, uint32_t delay) * Schedule task with the earliest deadline from task list. * Can run in IRQ context. */ -struct task *schedule_edf(void) +static struct task *schedule_edf(void) { struct task *task; struct task *next_plus1_task = NULL; @@ -201,8 +201,9 @@ struct task *schedule_edf(void) return next_plus1_task; } +#if 0 /* FIXME: is this needed ? */ /* delete task from scheduler */ -int schedule_task_del(struct task *task) +static int schedule_task_del(struct task *task) { uint32_t flags; int ret = 0; @@ -225,6 +226,7 @@ out: spin_unlock_irq(&sch->lock, flags); return ret; } +#endif /* * Add a new task to the scheduler to be run and define a scheduling @@ -285,7 +287,7 @@ void schedule_task_complete(struct task *task) spin_unlock_irq(&sch->lock, flags); } -void scheduler_run(void *unused) +static void scheduler_run(void *unused) { struct task *next_task; diff --git a/src/math/numbers.c b/src/math/numbers.c index 579de5d..93d41ea 100644 --- a/src/math/numbers.c +++ b/src/math/numbers.c @@ -34,6 +34,7 @@ * pseudocode in * https://en.wikipedia.org/wiki/Euclidean_algorithm#Implementations */ +#include <reef/math/numbers.h> int gcd(int a, int b) { diff --git a/src/math/trig.c b/src/math/trig.c index 8101980..79eb480 100644 --- a/src/math/trig.c +++ b/src/math/trig.c @@ -32,6 +32,7 @@ #include <stdint.h> #include <reef/audio/format.h> +#include <reef/math/trig.h> #define SINE_C_Q20 341782638 /* 2*SINE_NQUART/pi in Q12.20 */ diff --git a/src/platform/baytrail/platform.c b/src/platform/baytrail/platform.c index e3daf48..b701196 100644 --- a/src/platform/baytrail/platform.c +++ b/src/platform/baytrail/platform.c @@ -40,6 +40,7 @@ #include <reef/mailbox.h> #include <reef/dai.h> #include <reef/dma.h> +#include <reef/interrupt.h> #include <reef/reef.h> #include <reef/work.h> #include <reef/clock.h> diff --git a/src/platform/baytrail/timer.c b/src/platform/baytrail/timer.c index 513ede1..9256dd9 100644 --- a/src/platform/baytrail/timer.c +++ b/src/platform/baytrail/timer.c @@ -44,7 +44,7 @@ struct timer_data { static struct timer_data xtimer[1] = {}; -void platform_timer_64_handler(void *arg) +static void platform_timer_64_handler(void *arg) { struct timer *timer = arg; struct timer_data *tdata = timer->timer_data; -- 2.11.0

1 0

[Sound-open-firmware] [PATCH] Fix compilation errors triggered by -Werror=missing-prototypes
by Seppo Ingalsuo 27 Sep '17

27 Sep '17

This patch adds include statements for modules to introduce the function prototypes for trig.c, numbers.c, and lib.c. In addition three unused functions are disabled with #if 0. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo(a)linux.intel.com> --- src/arch/xtensa/hal/set_region_translate.c | 2 ++ src/arch/xtensa/xtos/exc-syscall-c-handler.c | 3 +++ src/lib/lib.c | 1 + src/math/numbers.c | 2 ++ src/math/trig.c | 1 + 5 files changed, 9 insertions(+) diff --git a/src/arch/xtensa/hal/set_region_translate.c b/src/arch/xtensa/hal/set_region_translate.c index 7e885f6..5d5b21d 100644 --- a/src/arch/xtensa/hal/set_region_translate.c +++ b/src/arch/xtensa/hal/set_region_translate.c @@ -69,6 +69,7 @@ static inline unsigned probe_itlb(unsigned addr) { return tmp; } +#if 0 void invalidate_dtlb_entry(unsigned addr) { __asm__ __volatile__("idtlb %0; dsync \n\t" : : "a" (addr)); @@ -78,6 +79,7 @@ void invalidate_itlb_entry(unsigned addr) { __asm__ __volatile__("iitlb %0 ; isync\n\t" : : "a" (addr)); } +#endif static inline unsigned read_dtlbcfg() { unsigned long tmp; diff --git a/src/arch/xtensa/xtos/exc-syscall-c-handler.c b/src/arch/xtensa/xtos/exc-syscall-c-handler.c index 405f7b6..aa43d27 100644 --- a/src/arch/xtensa/xtos/exc-syscall-c-handler.c +++ b/src/arch/xtensa/xtos/exc-syscall-c-handler.c @@ -42,6 +42,8 @@ * _xtos_set_exception_handler( EXCCAUSE_SYSCALL, * (_xtos_handler)_xtos_p_syscall_handler ); */ + +#if 0 UserFrame* _xtos_p_syscall_handler( UserFrame *uf /*, int cause */ ) { uf->pc += 3; /* skip SYSCALL instruction */ @@ -98,4 +100,5 @@ UserFrame* _xtos_p_syscall_handler( UserFrame *uf /*, int cause */ ) return( uf ); } +#endif diff --git a/src/lib/lib.c b/src/lib/lib.c index e1b1d9a..a1c9933 100644 --- a/src/lib/lib.c +++ b/src/lib/lib.c @@ -31,6 +31,7 @@ #include <stdint.h> #include <stdlib.h> #include <reef/reef.h> +#include <reef/alloc.h> #if 0 // TODO: only compile if no arch memcpy is available. diff --git a/src/math/numbers.c b/src/math/numbers.c index 579de5d..67492a0 100644 --- a/src/math/numbers.c +++ b/src/math/numbers.c @@ -35,6 +35,8 @@ * https://en.wikipedia.org/wiki/Euclidean_algorithm#Implementations */ +#include <reef/math/numbers.h> + int gcd(int a, int b) { int t; diff --git a/src/math/trig.c b/src/math/trig.c index 8101980..79eb480 100644 --- a/src/math/trig.c +++ b/src/math/trig.c @@ -32,6 +32,7 @@ #include <stdint.h> #include <reef/audio/format.h> +#include <reef/math/trig.h> #define SINE_C_Q20 341782638 /* 2*SINE_NQUART/pi in Q12.20 */ -- 2.11.0

3 4

[Sound-open-firmware] [PATCH 00/10] MISRA rules - 1st pass
by Pierre-Louis Bossart 27 Sep '17

27 Sep '17

here are a set of patches that start a cleanup for MISRA compliance required for automotive. In general while the MISRA rules are debatable, they helped point at code that can be optimized or made more readable. Additional information on remaining opens can be seen here: https://sonarcloud.io/project/issues?directories=src%2Farch%2Fxtensa%2Csrc%… Pierre-Louis Bossart (10): sof: define each identifier in a dedicated statement pipeline: add missing default case pipeline: simplify prepare loop SRC: replace switch by if for readability tone: replace switch by if for readability volume: replace switch by if for readability ssp: replace switch by if for readability ssp: remove unnecessary direction parameter in ssp_stop wait: reduce number of nested breaks ipc: remove unnecessary tests src/arch/xtensa/timer.c | 9 ++++-- src/audio/dai.c | 17 ++++++---- src/audio/eq_fir.c | 22 ++++++++++--- src/audio/eq_iir.c | 21 +++++++++--- src/audio/host.c | 17 +++++++--- src/audio/iir.c | 6 ++-- src/audio/mixer.c | 15 ++++++--- src/audio/pipeline.c | 26 +++++++-------- src/audio/pipeline_static.c | 5 ++- src/audio/src.c | 38 ++++++++++++++-------- src/audio/src_core.c | 42 ++++++++++++++++++------ src/audio/tone.c | 24 +++++++------- src/audio/volume.c | 69 +++++++++++++++++++++++----------------- src/drivers/dw-dma.c | 15 ++++++--- src/drivers/ssp.c | 24 ++++++++------ src/include/reef/wait.h | 4 +-- src/ipc/byt-ipc.c | 5 +-- src/ipc/dma-copy.c | 14 +++++--- src/ipc/intel-ipc.c | 9 ++++-- src/ipc/ipc.c | 3 +- src/ipc/pmc-ipc.c | 6 ++-- src/lib/alloc.c | 24 ++++++++++---- src/lib/lib.c | 17 +++++++--- src/lib/schedule.c | 13 +++++--- src/lib/work.c | 18 ++++++++--- src/math/trig.c | 8 +++-- src/platform/baytrail/clk.c | 3 +- src/platform/baytrail/platform.c | 18 ++++++++--- src/platform/baytrail/timer.c | 8 +++-- 29 files changed, 334 insertions(+), 166 deletions(-) -- 2.11.0

2 11

[Sound-open-firmware] [PATCH] sof: check for missing prototypes
by Pierre-Louis Bossart 25 Sep '17

25 Sep '17

gcc -Wall is not enough, add -Wmissing-prototypes to align with kernel warnings. Add relevant includes and move functions without prototype as static Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart(a)linux.intel.com> --- configure.ac | 2 +- src/arch/xtensa/task.c | 1 + src/audio/src_core.c | 6 +++--- src/include/reef/init.h | 2 ++ src/include/reef/interrupt.h | 2 +- src/ipc/pmc-ipc.c | 1 + src/lib/schedule.c | 8 +++++--- src/platform/baytrail/platform.c | 1 + src/platform/baytrail/timer.c | 2 +- 9 files changed, 16 insertions(+), 9 deletions(-) diff --git a/configure.ac b/configure.ac index 1c6fd0a..e8c8edf 100644 --- a/configure.ac +++ b/configure.ac @@ -18,7 +18,7 @@ AC_DEFINE_UNQUOTED([REEF_MINOR], reef_minor, [Reef minor version]) AC_CANONICAL_HOST # General compiler flags -CFLAGS="${CFLAGS:+$CFLAGS } -O2 -g -Wall -Werror -Wl,-EL -fno-inline-functions -nostdlib" +CFLAGS="${CFLAGS:+$CFLAGS } -O2 -g -Wall -Werror -Wl,-EL -fno-inline-functions -nostdlib -Wmissing-prototypes" # Cross compiler tool libgcc and headers AC_ARG_WITH([root-dir], diff --git a/src/arch/xtensa/task.c b/src/arch/xtensa/task.c index 7e74c07..349f8ab 100644 --- a/src/arch/xtensa/task.c +++ b/src/arch/xtensa/task.c @@ -33,6 +33,7 @@ #include <reef/interrupt.h> #include <platform/platform.h> #include <reef/debug.h> +#include <arch/task.h> #include <stdint.h> #include <errno.h> diff --git a/src/audio/src_core.c b/src/audio/src_core.c index bce3586..5b804d6 100644 --- a/src/audio/src_core.c +++ b/src/audio/src_core.c @@ -52,21 +52,21 @@ int sof_rates[SOF_RATES_LENGTH] = {8000, 11025, 12000, 16000, 18900, 192000}; /* Calculates the needed FIR delay line length */ -int src_fir_delay_length(struct src_stage *s) +static int src_fir_delay_length(struct src_stage *s) { return s->subfilter_length + (s->num_of_subfilters - 1) * s->idm + s->blk_in; } /* Calculates the FIR output delay line length */ -int src_out_delay_length(struct src_stage *s) +static int src_out_delay_length(struct src_stage *s) { return (s->num_of_subfilters - 1) * s->odm + 1; } /* Returns index of a matching sample rate */ -int src_find_fs(int fs_list[], int list_length, int fs) +static int src_find_fs(int fs_list[], int list_length, int fs) { int i; diff --git a/src/include/reef/init.h b/src/include/reef/init.h index c2eaf87..9d97b48 100644 --- a/src/include/reef/init.h +++ b/src/include/reef/init.h @@ -38,4 +38,6 @@ int main(int argc, char *argv[]); int arch_init(struct reef *reef); +void __memmap_init(void); + #endif diff --git a/src/include/reef/interrupt.h b/src/include/reef/interrupt.h index 3c78e77..ea7d057 100644 --- a/src/include/reef/interrupt.h +++ b/src/include/reef/interrupt.h @@ -79,7 +79,7 @@ static inline void interrupt_global_enable(uint32_t flags) arch_interrupt_global_enable(flags); } -uint32_t platform_interrupt_get_enabled(); +uint32_t platform_interrupt_get_enabled(void); void platform_interrupt_clear(uint32_t irq, uint32_t mask); void platform_interrupt_mask(uint32_t irq, uint32_t mask); void platform_interrupt_unmask(uint32_t irq, uint32_t mask); diff --git a/src/ipc/pmc-ipc.c b/src/ipc/pmc-ipc.c index b8183c0..fdb8d4b 100644 --- a/src/ipc/pmc-ipc.c +++ b/src/ipc/pmc-ipc.c @@ -37,6 +37,7 @@ #include <reef/wait.h> #include <reef/trace.h> #include <platform/interrupt.h> +#include <platform/pmc.h> #include <platform/shim.h> diff --git a/src/lib/schedule.c b/src/lib/schedule.c index 142a2fb..427ca2e 100644 --- a/src/lib/schedule.c +++ b/src/lib/schedule.c @@ -159,7 +159,7 @@ static uint32_t sch_work(void *data, uint32_t delay) * Schedule task with the earliest deadline from task list. * Can run in IRQ context. */ -struct task *schedule_edf(void) +static struct task *schedule_edf(void) { struct task *task; struct task *next_plus1_task = NULL; @@ -201,8 +201,9 @@ struct task *schedule_edf(void) return next_plus1_task; } +#if 0 /* FIXME: is this needed ? */ /* delete task from scheduler */ -int schedule_task_del(struct task *task) +static int schedule_task_del(struct task *task) { uint32_t flags; int ret = 0; @@ -225,6 +226,7 @@ out: spin_unlock_irq(&sch->lock, flags); return ret; } +#endif /* * Add a new task to the scheduler to be run and define a scheduling @@ -285,7 +287,7 @@ void schedule_task_complete(struct task *task) spin_unlock_irq(&sch->lock, flags); } -void scheduler_run(void *unused) +static void scheduler_run(void *unused) { struct task *next_task; diff --git a/src/platform/baytrail/platform.c b/src/platform/baytrail/platform.c index e3daf48..b701196 100644 --- a/src/platform/baytrail/platform.c +++ b/src/platform/baytrail/platform.c @@ -40,6 +40,7 @@ #include <reef/mailbox.h> #include <reef/dai.h> #include <reef/dma.h> +#include <reef/interrupt.h> #include <reef/reef.h> #include <reef/work.h> #include <reef/clock.h> diff --git a/src/platform/baytrail/timer.c b/src/platform/baytrail/timer.c index 513ede1..9256dd9 100644 --- a/src/platform/baytrail/timer.c +++ b/src/platform/baytrail/timer.c @@ -44,7 +44,7 @@ struct timer_data { static struct timer_data xtimer[1] = {}; -void platform_timer_64_handler(void *arg) +static void platform_timer_64_handler(void *arg) { struct timer *timer = arg; struct timer_data *tdata = timer->timer_data; -- 2.11.0

1 0

[Sound-open-firmware] [PATCH] EQ FIR: Get coefficients and response switch via ABI
by Seppo Ingalsuo 25 Sep '17

25 Sep '17

The FIR equalizer configure and control is updated to use the new component ABI. Checks are added to protect equalizer from invalid setup. Also an unused parameter is removed from fir_init_delay() function. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo(a)linux.intel.com> --- src/audio/eq_fir.c | 86 ++++++++++++++++++++++++++++++------------------------ src/audio/eq_fir.h | 34 --------------------- src/audio/fir.c | 3 +- src/audio/fir.h | 3 +- 4 files changed, 50 insertions(+), 76 deletions(-) diff --git a/src/audio/eq_fir.c b/src/audio/eq_fir.c index eb2d864..fc6ae03 100644 --- a/src/audio/eq_fir.c +++ b/src/audio/eq_fir.c @@ -44,6 +44,7 @@ #include <reef/audio/pipeline.h> #include <reef/audio/format.h> #include <uapi/ipc.h> +#include <uapi/eq.h> #include "fir.h" #include "eq_fir.h" @@ -57,7 +58,7 @@ /* src component private data */ struct comp_data { - struct eq_fir_configuration *config; + struct sof_eq_fir_config *config; uint32_t period_bytes; struct fir_state_32x16 fir[PLATFORM_MAX_CHANNELS]; void (*eq_fir_func)(struct comp_dev *dev, @@ -120,7 +121,7 @@ static void eq_fir_s32_default(struct comp_dev *dev, } } -static void eq_fir_free_parameters(struct eq_fir_configuration **config) +static void eq_fir_free_parameters(struct sof_eq_fir_config **config) { if (*config != NULL) rfree(*config); @@ -148,22 +149,26 @@ static void eq_fir_free_delaylines(struct fir_state_32x16 fir[]) } static int eq_fir_setup(struct fir_state_32x16 fir[], - struct eq_fir_configuration *config, int nch) + struct sof_eq_fir_config *config, int nch) { int i, j, idx, length, resp; int32_t *fir_data; + int16_t *coef_data, *assign_response; int response_index[PLATFORM_MAX_CHANNELS]; int length_sum = 0; - if (nch > PLATFORM_MAX_CHANNELS) + if ((nch > PLATFORM_MAX_CHANNELS) + || (config->channels_in_config > PLATFORM_MAX_CHANNELS)) return -EINVAL; /* Collect index of respose start positions in all_coefficients[] */ j = 0; + assign_response = &config->data[0]; + coef_data = &config->data[config->channels_in_config]; for (i = 0; i < PLATFORM_MAX_CHANNELS; i++) { - if (i < config->number_of_responses_defined) { + if (i < config->number_of_responses) { response_index[i] = j; - j += 3 + config->all_coefficients[j]; + j += 3 + coef_data[j]; } else { response_index[i] = 0; } @@ -174,15 +179,17 @@ static int eq_fir_setup(struct fir_state_32x16 fir[], /* Initialize 1st phase */ for (i = 0; i < nch; i++) { - resp = config->assign_response[i]; + resp = assign_response[i]; + if (resp > config->number_of_responses - 1) + return -EINVAL; + if (resp < 0) { /* Initialize EQ channel to bypass */ fir_reset(&fir[i]); } else { /* Initialize EQ coefficients */ idx = response_index[resp]; - length = fir_init_coef(&fir[i], - &config->all_coefficients[idx]); + length = fir_init_coef(&fir[i], &coef_data[idx]); if (length > 0) length_sum += length; } @@ -199,31 +206,29 @@ static int eq_fir_setup(struct fir_state_32x16 fir[], /* Initialize 2nd phase to set EQ delay lines pointers */ for (i = 0; i < nch; i++) { - resp = config->assign_response[i]; + resp = assign_response[i]; if (resp >= 0) { - idx = response_index[resp]; - fir_init_delay(&fir[i], &config->all_coefficients[idx], - &fir_data); + fir_init_delay(&fir[i], &fir_data); } - } return 0; } static int eq_fir_switch_response(struct fir_state_32x16 fir[], - struct eq_fir_configuration *config, struct eq_fir_update *update, - int nch) + struct sof_eq_fir_config *config, + struct sof_ipc_ctrl_value_comp compv[], uint32_t num_elemens, int nch) { - int i, ret; + int i, j, ret; /* Copy assign response from update and re-initilize EQ */ if (config == NULL) return -EINVAL; - for (i = 0; i < config->stream_max_channels; i++) { - if (i < update->stream_max_channels) - config->assign_response[i] = update->assign_response[i]; + for (i = 0; i < num_elemens; i++) { + j = compv[i].index; + if (j < config->channels_in_config) + config->data[i] = compv[i].svalue; } ret = eq_fir_setup(fir, config, nch); @@ -238,6 +243,9 @@ static int eq_fir_switch_response(struct fir_state_32x16 fir[], static struct comp_dev *eq_fir_new(struct sof_ipc_comp *comp) { struct comp_dev *dev; + struct sof_ipc_comp_eq_fir *eq_fir; + struct sof_ipc_comp_eq_fir *ipc_eq_fir + = (struct sof_ipc_comp_eq_fir *) comp; struct comp_data *cd; int i; @@ -248,7 +256,8 @@ static struct comp_dev *eq_fir_new(struct sof_ipc_comp *comp) if (dev == NULL) return NULL; - memcpy(&dev->comp, comp, sizeof(struct sof_ipc_comp_eq_fir)); + eq_fir = (struct sof_ipc_comp_eq_fir *) &dev->comp; + memcpy(eq_fir, ipc_eq_fir, sizeof(struct sof_ipc_comp_eq_fir)); cd = rzalloc(RZONE_RUNTIME, RFLAGS_NONE, sizeof(*cd)); if (cd == NULL) { @@ -282,14 +291,18 @@ static void eq_fir_free(struct comp_dev *dev) /* set component audio stream parameters */ static int eq_fir_params(struct comp_dev *dev) { - struct comp_data *cd = comp_get_drvdata(dev); struct sof_ipc_comp_config *config = COMP_GET_CONFIG(dev); + struct comp_data *cd = comp_get_drvdata(dev); struct comp_buffer *sink; int err; trace_eq("par"); - /* calculate period size based on config */ + /* Calculate period size based on config. First make sure that + * frame_bytes is set. + */ + dev->frame_bytes = + dev->params.sample_container_bytes * dev->params.channels; cd->period_bytes = dev->frames * dev->frame_bytes; /* configure downstream buffer */ @@ -312,27 +325,30 @@ static int eq_fir_params(struct comp_dev *dev) static int fir_cmd(struct comp_dev *dev, struct sof_ipc_ctrl_data *cdata) { struct comp_data *cd = comp_get_drvdata(dev); - struct eq_fir_update *fir_update; /* TODO: move this to IPC as it's ABI */ + struct sof_ipc_ctrl_value_comp *compv; size_t bs; int i, ret = 0; /* TODO: determine if data is DMAed or appended to cdata */ + /* Check version from ABI header */ + if (cdata->data->comp_abi != SOF_EQ_FIR_ABI_VERSION) + return -EINVAL; + switch (cdata->cmd) { case SOF_CTRL_CMD_EQ_SWITCH: trace_eq("EFx"); - fir_update = (struct eq_fir_update *)cdata->data; + compv = (struct sof_ipc_ctrl_value_comp *) cdata->data->data; ret = eq_fir_switch_response(cd->fir, cd->config, - fir_update, PLATFORM_MAX_CHANNELS); + compv, cdata->num_elems, PLATFORM_MAX_CHANNELS); if (ret < 0) { trace_eq_error("ec1"); return ret; } /* Print trace information */ - tracev_value(fir_update->stream_max_channels); - for (i = 0; i < fir_update->stream_max_channels; i++) - tracev_value(fir_update->assign_response[i]); + for (i = 0; i < cd->config->channels_in_config; i++) + tracev_value(cd->config->data[i]); break; case SOF_CTRL_CMD_EQ_CONFIG: @@ -342,22 +358,16 @@ static int fir_cmd(struct comp_dev *dev, struct sof_ipc_ctrl_data *cdata) eq_fir_free_parameters(&cd->config); /* Copy new config, need to decode data to know the size */ - bs = cdata->num_elems; - if (bs > EQ_FIR_MAX_BLOB_SIZE) + bs = cdata->data->size; + if ((bs > SOF_EQ_FIR_MAX_SIZE) || (bs < 1)) return -EINVAL; cd->config = rzalloc(RZONE_RUNTIME, RFLAGS_NONE, bs); if (cd->config == NULL) return -EINVAL; - memcpy(cd->config, cdata->data, bs); + memcpy(cd->config, cdata->data->data, bs); ret = eq_fir_setup(cd->fir, cd->config, PLATFORM_MAX_CHANNELS); - - /* Print trace information */ - tracev_value(cd->config->stream_max_channels); - tracev_value(cd->config->number_of_responses_defined); - for (i = 0; i < cd->config->stream_max_channels; i++) - tracev_value(cd->config->assign_response[i]); break; case SOF_CTRL_CMD_MUTE: trace_eq("EFm"); diff --git a/src/audio/eq_fir.h b/src/audio/eq_fir.h index 7579ba4..4d2480c 100644 --- a/src/audio/eq_fir.h +++ b/src/audio/eq_fir.h @@ -33,38 +33,4 @@ #ifndef EQ_FIR_H #define EQ_FIR_H - -/* - * eq_fir_configuration data structure contains this information - * stream max channels - * number_of_responses_defined - * 0=no respones, 1=one response defined, 2=two responses defined, etc. - * assign_response[STREAM_MAX_CHANNELS] - * -1 = not defined, 0 = use first response, 1 = use 2nd response, etc. - * E.g. {0, 0, 0, 0, -1, -1, -1, -1} would apply to channels 0-3 the - * same first defined response and leave channels 4-7 unequalized. - * all_coefficients[] - * Repeated data { filter_length, input_shift, output_shift, h[] } - * where vector h has filter_length number of coefficients. - * Coefficients in h[] are in Q1.15 format. 16384 = 0.5. The shifts - * are number of right shifts. - */ - -#define NHEADER_EQ_FIR_BLOB 2 /* Header is two words plus assigns plus coef */ - -#define EQ_FIR_MAX_BLOB_SIZE 4096 /* Max size allowed for blob in bytes */ - -struct eq_fir_configuration { - uint16_t stream_max_channels; - uint16_t number_of_responses_defined; - uint16_t assign_response[PLATFORM_MAX_CHANNELS]; - int16_t all_coefficients[]; -}; - -struct eq_fir_update { - uint16_t stream_max_channels; - uint16_t assign_response[PLATFORM_MAX_CHANNELS]; - -}; - #endif diff --git a/src/audio/fir.c b/src/audio/fir.c index beb7e72..044b35d 100644 --- a/src/audio/fir.c +++ b/src/audio/fir.c @@ -83,8 +83,7 @@ int fir_init_coef(struct fir_state_32x16 *fir, int16_t config[]) return fir->length; } -void fir_init_delay(struct fir_state_32x16 *fir, int16_t config[], - int32_t **data) +void fir_init_delay(struct fir_state_32x16 *fir, int32_t **data) { fir->delay = *data; fir->delay_size = fir->length; diff --git a/src/audio/fir.h b/src/audio/fir.h index a99bcde..e682ef2 100644 --- a/src/audio/fir.h +++ b/src/audio/fir.h @@ -58,8 +58,7 @@ void fir_reset(struct fir_state_32x16 *fir); int fir_init_coef(struct fir_state_32x16 *fir, int16_t config[]); -void fir_init_delay(struct fir_state_32x16 *fir, int16_t config[], - int32_t **data); +void fir_init_delay(struct fir_state_32x16 *fir, int32_t **data); /* The next trivial functions are inlined */ -- 2.11.0

1 0

[Sound-open-firmware] [PATCH] EQ IIR: Get filter config and response switch via ABI
by Seppo Ingalsuo 25 Sep '17

25 Sep '17

The IIR equalizer configure and control is updated to use the new component ABI. In addition there are checks added to protect IIR equalizer from invalid setup. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo(a)linux.intel.com> --- src/audio/eq_iir.c | 76 +++++++++++++++++--------------- src/audio/eq_iir.h | 47 -------------------- src/include/uapi/eq.h | 119 ++++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 159 insertions(+), 83 deletions(-) create mode 100644 src/include/uapi/eq.h diff --git a/src/audio/eq_iir.c b/src/audio/eq_iir.c index 58a06a3..86f879a 100644 --- a/src/audio/eq_iir.c +++ b/src/audio/eq_iir.c @@ -44,6 +44,7 @@ #include <reef/audio/pipeline.h> #include <reef/audio/format.h> #include <uapi/ipc.h> +#include <uapi/eq.h> #include "eq_iir.h" #include "iir.h" @@ -57,7 +58,7 @@ /* src component private data */ struct comp_data { - struct eq_iir_configuration *config; + struct sof_eq_iir_config *config; uint32_t period_bytes; struct iir_state_df2t iir[PLATFORM_MAX_CHANNELS]; void (*eq_iir_func)(struct comp_dev *dev, @@ -120,7 +121,7 @@ static void eq_iir_s32_default(struct comp_dev *dev, } } -static void eq_iir_free_parameters(struct eq_iir_configuration **config) +static void eq_iir_free_parameters(struct sof_eq_iir_config **config) { if (*config != NULL) rfree(*config); @@ -150,27 +151,31 @@ static void eq_iir_free_delaylines(struct iir_state_df2t *iir) } static int eq_iir_setup(struct iir_state_df2t iir[], - struct eq_iir_configuration *config, int nch) + struct sof_eq_iir_config *config, int nch) { int i, j, idx, resp; size_t s; size_t size_sum = 0; int64_t *iir_delay; /* TODO should not need to know the type */ + int32_t *coef_data, *assign_response; int response_index[PLATFORM_MAX_CHANNELS]; - if (nch > PLATFORM_MAX_CHANNELS) - return -EINVAL; - /* Free existing IIR channels data if it was allocated */ eq_iir_free_delaylines(iir); + if ((nch > PLATFORM_MAX_CHANNELS) + || (config->channels_in_config > PLATFORM_MAX_CHANNELS)) + return -EINVAL; + /* Collect index of respose start positions in all_coefficients[] */ j = 0; + assign_response = &config->data[0]; + coef_data = &config->data[config->channels_in_config]; for (i = 0; i < PLATFORM_MAX_CHANNELS; i++) { - if (i < config->number_of_responses_defined) { + if (i < config->number_of_responses) { response_index[i] = j; j += NHEADER_DF2T - + NBIQUAD_DF2T * config->all_coefficients[j]; + + NBIQUAD_DF2T * coef_data[j]; } else { response_index[i] = 0; } @@ -178,15 +183,17 @@ static int eq_iir_setup(struct iir_state_df2t iir[], /* Initialize 1st phase */ for (i = 0; i < nch; i++) { - resp = config->assign_response[i]; + resp = assign_response[i]; + if (resp > config->number_of_responses - 1) + return -EINVAL; + if (resp < 0) { /* Initialize EQ channel to bypass */ iir_reset_df2t(&iir[i]); } else { /* Initialize EQ coefficients */ idx = response_index[resp]; - s = iir_init_coef_df2t(&iir[i], - &config->all_coefficients[idx]); + s = iir_init_coef_df2t(&iir[i], &coef_data[idx]); if (s > 0) size_sum += s; else @@ -204,9 +211,8 @@ static int eq_iir_setup(struct iir_state_df2t iir[], /* Initialize 2nd phase to set EQ delay lines pointers */ for (i = 0; i < nch; i++) { - resp = config->assign_response[i]; + resp = assign_response[i]; if (resp >= 0) { - idx = response_index[resp]; iir_init_delay_df2t(&iir[i], &iir_delay); } @@ -216,18 +222,19 @@ static int eq_iir_setup(struct iir_state_df2t iir[], } static int eq_iir_switch_response(struct iir_state_df2t iir[], - struct eq_iir_configuration *config, struct eq_iir_update *update, - int nch) + struct sof_eq_iir_config *config, + struct sof_ipc_ctrl_value_comp compv[], uint32_t num_elemens, int nch) { - int i, ret; + int i, j, ret; /* Copy assign response from update and re-initilize EQ */ if (config == NULL) return -EINVAL; - for (i = 0; i < config->stream_max_channels; i++) { - if (i < update->stream_max_channels) - config->assign_response[i] = update->assign_response[i]; + for (i = 0; i < num_elemens; i++) { + j = compv[i].index; + if (j < config->channels_in_config) + config->data[i] = compv[i].svalue; } ret = eq_iir_setup(iir, config, nch); @@ -293,7 +300,11 @@ static int eq_iir_params(struct comp_dev *dev) trace_eq_iir("par"); - /* calculate period size based on config */ + /* Calculate period size based on config. First make sure that + * frame_bytes is set. + */ + dev->frame_bytes = + dev->params.sample_container_bytes * dev->params.channels; cd->period_bytes = dev->frames * dev->frame_bytes; /* configure downstream buffer */ @@ -316,21 +327,21 @@ static int eq_iir_params(struct comp_dev *dev) static int iir_cmd(struct comp_dev *dev, struct sof_ipc_ctrl_data *cdata) { struct comp_data *cd = comp_get_drvdata(dev); - struct eq_iir_update *iir_update; /* TODO: move to IPC header as part of ABI */ + struct sof_ipc_ctrl_value_comp *compv; int i, ret = 0; size_t bs; switch (cdata->cmd) { case SOF_CTRL_CMD_EQ_SWITCH: trace_eq_iir("EFx"); - iir_update = (struct eq_iir_update *) cdata->data; + compv = (struct sof_ipc_ctrl_value_comp *) cdata->data->data; ret = eq_iir_switch_response(cd->iir, cd->config, - iir_update, PLATFORM_MAX_CHANNELS); + compv, cdata->num_elems, PLATFORM_MAX_CHANNELS); /* Print trace information */ - tracev_value(iir_update->stream_max_channels); - for (i = 0; i < iir_update->stream_max_channels; i++) - tracev_value(iir_update->assign_response[i]); + tracev_value(cd->config->channels_in_config); + for (i = 0; i < cd->config->channels_in_config; i++) + tracev_value(cd->config->data[i]); break; case SOF_CTRL_CMD_EQ_CONFIG: @@ -339,8 +350,8 @@ static int iir_cmd(struct comp_dev *dev, struct sof_ipc_ctrl_data *cdata) eq_iir_free_parameters(&cd->config); /* Copy new config, need to decode data to know the size */ - bs = cdata->num_elems; - if (bs > EQ_IIR_MAX_BLOB_SIZE) + bs = cdata->data->size; + if ((bs > SOF_EQ_IIR_MAX_SIZE) || (bs < 1)) return -EINVAL; /* Allocate and make a copy of the blob and setup IIR */ @@ -348,18 +359,11 @@ static int iir_cmd(struct comp_dev *dev, struct sof_ipc_ctrl_data *cdata) if (cd->config == NULL) return -EINVAL; - memcpy(cd->config, cdata->data, bs); + memcpy(cd->config, cdata->data->data, bs); /* Initialize all channels, the actual number of channels may * not be set yet. */ ret = eq_iir_setup(cd->iir, cd->config, PLATFORM_MAX_CHANNELS); - - /* Print trace information */ - tracev_value(cd->config->stream_max_channels); - tracev_value(cd->config->number_of_responses_defined); - for (i = 0; i < cd->config->stream_max_channels; i++) - tracev_value(cd->config->assign_response[i]); - break; case SOF_CTRL_CMD_MUTE: trace_eq_iir("EFm"); diff --git a/src/audio/eq_iir.h b/src/audio/eq_iir.h index 42d95c1..e28a1a3 100644 --- a/src/audio/eq_iir.h +++ b/src/audio/eq_iir.h @@ -33,51 +33,4 @@ #ifndef EQ_IIR_H #define EQ_IIR_H - - -/* eq_iir_configuration - * uint32_t platform max channels - * uint32_t number_of_responses_defined - * 0=no responses, 1=one response defined, 2=two responses defined, etc. - * uint32_t assign_response[PLATFORM_MAX_CHANNELS] - * -1 = not defined, 0 = use first response, 1 = use 2nd response, etc. - * E.g. {0, 0, 0, 0, -1, -1, -1, -1} would apply to channels 0-3 the - * same first defined response and leave channels 4-7 unequalized. - * all_coefficients[] - * <1st EQ> - * uint32_t num_biquads - * uint32_t num_biquads_in_series - * <1st biquad> - * int32_t coef_a2 Q2.30 format - * int32_t coef_a1 Q2.30 format - * int32_t coef_b2 Q2.30 format - * int32_t coef_b1 Q2.30 format - * int32_t coef_b0 Q2.30 format - * int32_t output_shift number of shifts right, shift left is negative - * int32_t output_gain Q2.14 format - * <2nd biquad> - * ... - * <2nd EQ> - * - * Note: A flat response biquad can be made with a section set to - * b0 = 1.0, gain = 1.0, and other parameters set to 0 - * {0, 0, 0, 0, 1073741824, 0, 16484} - */ - -#define EQ_IIR_MAX_BLOB_SIZE 1024 /* In bytes or size_t */ - -#define NHEADER_EQ_IIR_BLOB 2 /* Blob is two words plus asssigns plus coef */ - -struct eq_iir_configuration { - int32_t stream_max_channels; - int32_t number_of_responses_defined; - int32_t assign_response[PLATFORM_MAX_CHANNELS]; - int32_t all_coefficients[]; -}; - -struct eq_iir_update { - int32_t stream_max_channels; - int32_t assign_response[PLATFORM_MAX_CHANNELS]; -}; - #endif diff --git a/src/include/uapi/eq.h b/src/include/uapi/eq.h new file mode 100644 index 0000000..9c42a51 --- /dev/null +++ b/src/include/uapi/eq.h @@ -0,0 +1,119 @@ +/* + * Copyright (c) 2017, Intel Corporation + * All rights reserved. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions are met: + * * Redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer. + * * Redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in the + * documentation and/or other materials provided with the distribution. + * * Neither the name of the Intel Corporation nor the + * names of its contributors may be used to endorse or promote products + * derived from this software without specific prior written permission. + * + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" + * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE + * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE + * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR + * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF + * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS + * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN + * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) + * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE + * POSSIBILITY OF SUCH DAMAGE. + * + * Author: Seppo Ingalsuo <seppo.ingalsuo(a)linux.intel.com> + */ + +#ifndef EQ_H +#define EQ_H + +/* FIR EQ type */ + +/* Component will reject non-matching configuration. The version number need + * to be incremented with any ABI changes in function fir_cmd(). + */ +#define SOF_EQ_FIR_ABI_VERSION 1 + +#define SOF_EQ_FIR_MAX_SIZE 4096 /* Max size allowed for coef data in bytes */ + +/* + * eq_fir_configuration data structure contains this information + * uint16_t channels_in_config + * This describes the number of channels in this EQ config data. It + * can be different from PLATFORM_MAX_CHANNELS. + * uint16_t number_of_responses + * 0=no responses, 1=one response defined, 2=two responses defined, etc. + * int16_t data[] + * assign_response[STREAM_MAX_CHANNELS] + * -1 = not defined, 0 = use first response, 1 = use 2nd response, etc. + * E.g. {0, 0, 0, 0, -1, -1, -1, -1} would apply to channels 0-3 the + * same first defined response and leave channels 4-7 unequalized. + * coef_data[] + * Repeated data { filter_length, input_shift, output_shift, h[] } + * for every EQ response defined where vector h has filter_length + * number of coefficients. Coefficients in h[] are in Q1.15 format. + * E.g. 16384 (Q1.15) = 0.5. The shifts are number of right shifts. + */ + +struct sof_eq_fir_config { + uint16_t channels_in_config; + uint16_t number_of_responses; + int16_t data[]; +}; + +/* IIR EQ type */ + +/* Component will reject non-matching configuration. The version number need + * to be incremented with any ABI changes in function fir_cmd(). + */ +#define SOF_EQ_FIR_ABI_VERSION 1 + +#define SOF_EQ_IIR_MAX_SIZE 1024 /* Max size allowed for coef data in bytes */ + +/* eq_iir_configuration + * uint32_t channels_in_config + * This describes the number of channels in this EQ config data. It + * can be different from PLATFORM_MAX_CHANNELS. + * uint32_t number_of_responses_defined + * 0=no responses, 1=one response defined, 2=two responses defined, etc. + * int32_t data[] + * Data consist of two parts. First is the response assign vector that + * has length of channels_in_config. The latter part is coefficient + * data. + * uint32_t assign_response[channels_in_config] + * -1 = not defined, 0 = use first response, 1 = use 2nd, etc. + * E.g. {0, 0, 0, 0, -1, -1, -1, -1} would apply to channels 0-3 the + * same first defined response and leave channels 4-7 unequalized. + * coefficient_data[] + * <1st EQ> + * uint32_t num_biquads + * uint32_t num_biquads_in_series + * <1st biquad> + * int32_t coef_a2 Q2.30 format + * int32_t coef_a1 Q2.30 format + * int32_t coef_b2 Q2.30 format + * int32_t coef_b1 Q2.30 format + * int32_t coef_b0 Q2.30 format + * int32_t output_shift number of shifts right, shift left is negative + * int32_t output_gain Q2.14 format + * <2nd biquad> + * ... + * <2nd EQ> + * + * Note: A flat response biquad can be made with a section set to + * b0 = 1.0, gain = 1.0, and other parameters set to 0 + * {0, 0, 0, 0, 1073741824, 0, 16484} + */ + +struct sof_eq_iir_config { + uint32_t channels_in_config; + uint32_t number_of_responses; + int32_t data[]; +}; + +#endif /* EQ_H */ + -- 2.11.0

1 0

[Sound-open-firmware] [PATCH] Component ABI: Check for valid ABI header for COMP_CMD_SET_DATA command
by Seppo Ingalsuo 25 Sep '17

25 Sep '17

This patch adds the code to check the generic ABI header fields and reject data with invalid header. The individual components those use ABI need to check the component specific fields in addition. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo(a)linux.intel.com> --- src/include/reef/audio/component.h | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/src/include/reef/audio/component.h b/src/include/reef/audio/component.h index 5ae8084..436a758 100644 --- a/src/include/reef/audio/component.h +++ b/src/include/reef/audio/component.h @@ -245,6 +245,17 @@ static inline int comp_host_buffer(struct comp_dev *dev, /* send component command - mandatory */ static inline int comp_cmd(struct comp_dev *dev, int cmd, void *data) { + struct sof_ipc_ctrl_data *cdata = data; + + if ((cmd == COMP_CMD_SET_DATA) + && ((cdata->data->magic != SOF_ABI_MAGIC) + || (cdata->data->abi != SOF_ABI_VERSION))) { + trace_comp_error("abi"); + trace_value(cdata->data->magic); + trace_value(cdata->data->abi); + return -EINVAL; + } + return dev->drv->ops.cmd(dev, cmd, data); } -- 2.11.0

1 0

[Sound-open-firmware] [PATCH] Volume: Add 24 to 24 bit samples volume processing
by Seppo Ingalsuo 25 Sep '17

25 Sep '17

This patch adds to component volume S24_4LE to S24_4LE processing. The macro from audio/format.h is used to compute shifts needed for product. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo(a)linux.intel.com> --- src/audio/volume.c | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/src/audio/volume.c b/src/audio/volume.c index be6197e..b7e8e39 100644 --- a/src/audio/volume.c +++ b/src/audio/volume.c @@ -41,6 +41,7 @@ #include <reef/clock.h> #include <reef/audio/component.h> #include <reef/audio/pipeline.h> +#include <reef/audio/format.h> #define trace_volume(__e) trace_event(TRACE_CLASS_VOLUME, __e) #define tracev_volume(__e) tracev_event(TRACE_CLASS_VOLUME, __e) @@ -214,6 +215,26 @@ static void vol_s24_to_s32(struct comp_dev *dev, struct comp_buffer *sink, } } +/* Copy and scale volume from 24 bit source buffer to 24 bit on 32 bit boundary + * dest buffer. + */ +static void vol_s24_to_s24(struct comp_dev *dev, struct comp_buffer *sink, + struct comp_buffer *source, uint32_t frames) +{ + struct comp_data *cd = comp_get_drvdata(dev); + int32_t i, *src = (int32_t*) source->r_ptr; + int32_t *dest = (int32_t*) sink->w_ptr; + + /* buffer sizes are always divisible by period frames */ + /* Samples are Q1.23 and volume is Q1.16 */ + for (i = 0; i < frames * 2; i += 2) { + dest[i] = (int32_t)(Q_MULTS_32X32( + (int64_t)src[i], cd->volume[0], 23, 16, 23)); + dest[i + 1] = (int32_t)(Q_MULTS_32X32( + (int64_t)src[i+1], cd->volume[1], 23, 16, 23)); + } +} + /* map of source and sink buffer formats to volume function */ static const struct comp_func_map func_map[] = { {SOF_IPC_FRAME_S16_LE, SOF_IPC_FRAME_S16_LE, 2, vol_s16_to_s16}, @@ -224,6 +245,7 @@ static const struct comp_func_map func_map[] = { {SOF_IPC_FRAME_S24_4LE, SOF_IPC_FRAME_S16_LE, 2, vol_s24_to_s16}, {SOF_IPC_FRAME_S32_LE, SOF_IPC_FRAME_S24_4LE, 2, vol_s32_to_s24}, {SOF_IPC_FRAME_S24_4LE, SOF_IPC_FRAME_S32_LE, 2, vol_s24_to_s32}, + {SOF_IPC_FRAME_S24_4LE, SOF_IPC_FRAME_S24_4LE, 2, vol_s24_to_s24}, }; /* synchronise host mmap() volume with real value */ -- 2.11.0

1 0

[Sound-open-firmware] [PATCH] comp: host: Add trace output for buffer size errors
by Liam Girdwood 25 Sep '17

25 Sep '17

Signed-off-by: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> --- src/audio/host.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/audio/host.c b/src/audio/host.c index a1d312a..76124fc 100644 --- a/src/audio/host.c +++ b/src/audio/host.c @@ -415,6 +415,7 @@ static int host_params(struct comp_dev *dev) err = buffer_set_size(hd->dma_buffer, buffer_size); if (err < 0) { trace_host_error("eSz"); + trace_value(buffer_size); return err; } -- 2.11.0

1 1

[Sound-open-firmware] [PATCH] topology: fix capture pipeline ID for SSP DAI
by Liam Girdwood 24 Sep '17

24 Sep '17

Capture pipeline for SSP DAI should be 2. Signed-off-by: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> --- topology/reef-apl-nocodec.m4 | 2 +- topology/reef-bdw-rt286.m4 | 2 +- topology/reef-bdw-rt5640.m4 | 2 +- topology/reef-bxt-nocodec.m4 | 2 +- topology/reef-byt-nocodec.m4 | 2 +- topology/reef-byt-rt5640.m4 | 2 +- topology/reef-byt-rt5651.m4 | 2 +- topology/reef-cht-nocodec.m4 | 2 +- topology/reef-hsw-rt5640.m4 | 2 +- 9 files changed, 9 insertions(+), 9 deletions(-) diff --git a/topology/reef-apl-nocodec.m4 b/topology/reef-apl-nocodec.m4 index f87a4e9..5e497f6 100644 --- a/topology/reef-apl-nocodec.m4 +++ b/topology/reef-apl-nocodec.m4 @@ -93,7 +93,7 @@ DAI_ADD(sof/pipe-dai-playback.m4, # capture DAI is SSP2 using I2S DAPM stream and 2 periods # Buffers use s24le format, with 48 frame per 1000us on core 0 with priority 0 DAI_ADD(sof/pipe-dai-capture.m4, - 1, SSP, 2, I2S, + 2, SSP, 2, I2S, PIPELINE_SINK_2, 2, s24le, 48, 1000, 0, 0) diff --git a/topology/reef-bdw-rt286.m4 b/topology/reef-bdw-rt286.m4 index eae2065..cb11ad6 100644 --- a/topology/reef-bdw-rt286.m4 +++ b/topology/reef-bdw-rt286.m4 @@ -93,7 +93,7 @@ DAI_ADD(sof/pipe-dai-playback.m4, # capture DAI is SSP0 using I2S DAPM stream and 2 periods # Buffers use s24le format, with 48 frame per 1000us on core 0 with priority 0 DAI_ADD(sof/pipe-dai-capture.m4, - 1, SSP, 0, I2S, + 2, SSP, 0, I2S, PIPELINE_SINK_2, 2, s24le, 48, 1000, 0, 0) diff --git a/topology/reef-bdw-rt5640.m4 b/topology/reef-bdw-rt5640.m4 index 60feb82..b3c19c2 100644 --- a/topology/reef-bdw-rt5640.m4 +++ b/topology/reef-bdw-rt5640.m4 @@ -93,7 +93,7 @@ DAI_ADD(sof/pipe-dai-playback.m4, # capture DAI is SSP0 using I2S DAPM stream and 2 periods # Buffers use s24le format, with 48 frame per 1000us on core 0 with priority 0 DAI_ADD(sof/pipe-dai-capture.m4, - 1, SSP, 0, I2S, + 2, SSP, 0, I2S, PIPELINE_SINK_2, 2, s24le, 48, 1000, 0, 0) diff --git a/topology/reef-bxt-nocodec.m4 b/topology/reef-bxt-nocodec.m4 index 744e9b1..58bc6f6 100644 --- a/topology/reef-bxt-nocodec.m4 +++ b/topology/reef-bxt-nocodec.m4 @@ -93,7 +93,7 @@ DAI_ADD(sof/pipe-dai-playback.m4, # capture DAI is SSP2 using I2S DAPM stream and 2 periods # Buffers use s24le format, with 48 frame per 1000us on core 0 with priority 0 DAI_ADD(sof/pipe-dai-capture.m4, - 1, SSP, 2, I2S, + 2, SSP, 2, I2S, PIPELINE_SINK_2, 2, s24le, 48, 1000, 0, 0) diff --git a/topology/reef-byt-nocodec.m4 b/topology/reef-byt-nocodec.m4 index 05373f9..5f6cd1d 100644 --- a/topology/reef-byt-nocodec.m4 +++ b/topology/reef-byt-nocodec.m4 @@ -82,7 +82,7 @@ DAI_ADD(sof/pipe-dai-playback.m4, # capture DAI is SSP2 using I2S DAPM stream and 2 periods # Buffers use s24le format, with 48 frame per 1000us on core 0 with priority 0 DAI_ADD(sof/pipe-dai-capture.m4, - 1, SSP, 2, I2S, + 2, SSP, 2, I2S, PIPELINE_SINK_2, 2, s24le, 48, 1000, 0, 0) diff --git a/topology/reef-byt-rt5640.m4 b/topology/reef-byt-rt5640.m4 index 911b4d1..c5e8b52 100644 --- a/topology/reef-byt-rt5640.m4 +++ b/topology/reef-byt-rt5640.m4 @@ -82,7 +82,7 @@ DAI_ADD(sof/pipe-dai-playback.m4, # capture DAI is SSP2 using I2S DAPM stream and 2 periods # Buffers use s24le format, with 48 frame per 1000us on core 0 with priority 0 DAI_ADD(sof/pipe-dai-capture.m4, - 1, SSP, 2, Audio, + 2, SSP, 2, Audio, PIPELINE_SINK_2, 2, s24le, 48, 1000, 0, 0) diff --git a/topology/reef-byt-rt5651.m4 b/topology/reef-byt-rt5651.m4 index c6f20a4..e335d66 100644 --- a/topology/reef-byt-rt5651.m4 +++ b/topology/reef-byt-rt5651.m4 @@ -82,7 +82,7 @@ DAI_ADD(sof/pipe-dai-playback.m4, # capture DAI is SSP2 using "Audio" DAPM stream name and 2 periods # Buffers use s24le format, with 48 frame per 1000us on core 0 with priority 0 DAI_ADD(sof/pipe-dai-capture.m4, - 1, SSP, 2, Audio, + 2, SSP, 2, Audio, PIPELINE_SINK_2, 2, s24le, 48, 1000, 0, 0) diff --git a/topology/reef-cht-nocodec.m4 b/topology/reef-cht-nocodec.m4 index 43e0e6e..ddeb71e 100644 --- a/topology/reef-cht-nocodec.m4 +++ b/topology/reef-cht-nocodec.m4 @@ -82,7 +82,7 @@ DAI_ADD(sof/pipe-dai-playback.m4, # capture DAI is SSP2 using I2S DAPM stream and 2 periods # Buffers use s24le format, with 48 frame per 1000us on core 0 with priority 0 DAI_ADD(sof/pipe-dai-capture.m4, - 1, SSP, 2, I2S, + 2, SSP, 2, I2S, PIPELINE_SINK_2, 2, s24le, 48, 1000, 0, 0) diff --git a/topology/reef-hsw-rt5640.m4 b/topology/reef-hsw-rt5640.m4 index 3d26149..efd815b 100644 --- a/topology/reef-hsw-rt5640.m4 +++ b/topology/reef-hsw-rt5640.m4 @@ -93,7 +93,7 @@ DAI_ADD(sof/pipe-dai-playback.m4, # capture DAI is SSP0 using I2S DAPM stream and 2 periods # Buffers use s24le format, with 48 frame per 1000us on core 0 with priority 0 DAI_ADD(sof/pipe-dai-capture.m4, - 1, SSP, 0, I2S, + 2, SSP, 0, I2S, PIPELINE_SINK_2, 2, s24le, 48, 1000, 0, 0) -- 2.11.0

1 0

[Sound-open-firmware] [PATCH] pipeline: fix timestamp for capture streams.
by Liam Girdwood 24 Sep '17

24 Sep '17

Make sure we timestamp upstream in capture streams Signed-off-by: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> --- src/audio/pipeline.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/audio/pipeline.c b/src/audio/pipeline.c index 62c21b7..4e830c6 100644 --- a/src/audio/pipeline.c +++ b/src/audio/pipeline.c @@ -851,7 +851,7 @@ upstream: continue; /* continue downstream */ - res = timestamp_upstream(start, buffer->sink, posn); + res = timestamp_upstream(start, buffer->source, posn); if (res == 1) break; } -- 2.11.0

1 3

[Sound-open-firmware] [PATCH] ssp: add missing break in dai format switch
by Pierre-Louis Bossart 23 Sep '17

23 Sep '17

Support for DSP_A is obviously broken. Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart(a)linux.intel.com> --- src/drivers/ssp.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/drivers/ssp.c b/src/drivers/ssp.c index c97b60e..a472974 100644 --- a/src/drivers/ssp.c +++ b/src/drivers/ssp.c @@ -224,6 +224,7 @@ static inline int ssp_set_config(struct dai *dai, sscr0 |= SSCR0_PSP | SSCR0_MOD | SSCR0_FRDC(config->num_slots); sscr1 |= SSCR1_TRAIL; sspsp |= SSPSP_SFRMWDTH(1) | SSPSP_SFRMDLY(2); + break; case SOF_DAI_FMT_DSP_B: sscr0 |= SSCR0_PSP | SSCR0_MOD | SSCR0_FRDC(config->num_slots); sscr1 |= SSCR1_TRAIL; -- 2.11.0

2 1

[Sound-open-firmware] [PATCH 1/4] SRC: Add checks for number of channels when initializing from params
by Seppo Ingalsuo 22 Sep '17

22 Sep '17

This patch prevents SRCs init to loop past PLATFORM_MAX_CHANNELS. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo(a)linux.intel.com> --- src/audio/src.c | 5 +++-- src/audio/src_core.h | 1 + 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/audio/src.c b/src/audio/src.c index 1f84a5a..203e802 100644 --- a/src/audio/src.c +++ b/src/audio/src.c @@ -322,7 +322,7 @@ static int src_params(struct comp_dev *dev) size_t delay_lines_size; uint32_t source_rate, sink_rate; int32_t *buffer_start; - int n = 0, i, err, frames_is_for_source, q; + int n = 0, i, err, frames_is_for_source, nch, q; trace_src("par"); @@ -392,7 +392,8 @@ static int src_params(struct comp_dev *dev) buffer_start = cd->delay_lines + need.scratch; /* Initize SRC for actual sample rate */ - for (i = 0; i < params->channels; i++) { + nch = MIN(params->channels, PLATFORM_MAX_CHANNELS); + for (i = 0; i < nch; i++) { n = src_polyphase_init(&cd->src[i], source_rate, sink_rate, &need, buffer_start); buffer_start += need.single_src; diff --git a/src/audio/src_core.h b/src/audio/src_core.h index 5977421..886cc23 100644 --- a/src/audio/src_core.h +++ b/src/audio/src_core.h @@ -33,6 +33,7 @@ #define SRC_CORE_H #define MAX(a, b) (((a) > (b)) ? (a) : (b)) +#define MIN(a, b) (((a) < (b)) ? (a) : (b)) struct src_alloc { int fir_s1; -- 2.11.0

2 4

[Sound-open-firmware] [PATCH 1/3] ipc: add support for timer topology scheduling source.
by Liam Girdwood 22 Sep '17

22 Sep '17

Add an option to schedule based on timer or on DAI/DMA IRQ Signed-off-by: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> --- src/include/uapi/ipc.h | 1 + 1 file changed, 1 insertion(+) diff --git a/src/include/uapi/ipc.h b/src/include/uapi/ipc.h index acfa05e..472fb41 100644 --- a/src/include/uapi/ipc.h +++ b/src/include/uapi/ipc.h @@ -678,6 +678,7 @@ struct sof_ipc_pipe_new { uint32_t mips; /* worst case instruction count per period */ uint32_t frames_per_sched; /* output frames of pipeline, 0 is variable */ uint32_t xrun_limit_usecs; /* report xruns greater than limit */ + uint32_t timer; /* non zero if timer scheduled otherwise DAI scheduled */ } __attribute__((packed)); /* pipeline construction complete - SOF_IPC_TPLG_PIPE_COMPLETE */ -- 2.11.0

1 2

[Sound-open-firmware] [PATCH] comp: enforce state transition checking for each component.
by Liam Girdwood 21 Sep '17

21 Sep '17

Make sure each component checks the state transition prior to performing any action for the new state. This ensure that only supported transitions are used and any non supported transition is flagged as an error. This also removes DRAIN state and performs some cleanup on mixer state handling. Signed-off-by: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> --- src/audio/component.c | 7 +++--- src/audio/dai.c | 16 ++++-------- src/audio/eq_fir.c | 10 +++++--- src/audio/eq_iir.c | 28 ++++----------------- src/audio/host.c | 15 +++++------ src/audio/mixer.c | 51 +++++++++++++++++++++----------------- src/audio/src.c | 14 +++++------ src/audio/tone.c | 14 +++++------ src/audio/volume.c | 15 +++++------ src/include/reef/audio/component.h | 1 - src/ipc/intel-ipc.c | 3 --- 11 files changed, 74 insertions(+), 100 deletions(-) diff --git a/src/audio/component.c b/src/audio/component.c index 1cf4799..ec170c6 100644 --- a/src/audio/component.c +++ b/src/audio/component.c @@ -126,6 +126,7 @@ int comp_set_state(struct comp_dev *dev, int cmd) dev->state = COMP_STATE_ACTIVE; } else { trace_comp_error("CES"); + trace_value(dev->state); ret = -EINVAL; } break; @@ -134,6 +135,7 @@ int comp_set_state(struct comp_dev *dev, int cmd) dev->state = COMP_STATE_ACTIVE; } else { trace_comp_error("CEr"); + trace_value(dev->state); ret = -EINVAL; } break; @@ -142,6 +144,7 @@ int comp_set_state(struct comp_dev *dev, int cmd) dev->state = COMP_STATE_READY; } else { trace_comp_error("CEs"); + trace_value(dev->state); ret = -EINVAL; } break; @@ -151,13 +154,11 @@ int comp_set_state(struct comp_dev *dev, int cmd) dev->state = COMP_STATE_PAUSED; else { trace_comp_error("CEp"); + trace_value(dev->state); ret = -EINVAL; } break; default: - trace_comp_error("CEd"); - trace_value(cmd); - ret = -EINVAL; break; } diff --git a/src/audio/dai.c b/src/audio/dai.c index cf8de72..e22d21d 100644 --- a/src/audio/dai.c +++ b/src/audio/dai.c @@ -444,20 +444,18 @@ static int dai_reset(struct comp_dev *dev) static int dai_cmd(struct comp_dev *dev, int cmd, void *data) { struct dai_data *dd = comp_get_drvdata(dev); -// struct sof_ipc_ctrl_values *ctrl = data; int ret; trace_dai("cmd"); tracev_value(cmd); + ret = comp_set_state(dev, cmd); + if (ret < 0) + return ret; + switch (cmd) { - case COMP_CMD_PAUSE: - case COMP_CMD_STOP: - dev->state = COMP_STATE_PAUSED; - break; case COMP_CMD_RELEASE: case COMP_CMD_START: - ret = dma_start(dd->dma, dd->chan); if (ret < 0) return ret; @@ -465,16 +463,12 @@ static int dai_cmd(struct comp_dev *dev, int cmd, void *data) /* update starting wallclock */ platform_dai_wallclock(dev, &dd->wallclock); - dev->state = COMP_STATE_ACTIVE; - break; - case COMP_CMD_SUSPEND: - case COMP_CMD_RESUME: break; default: break; } - return 0; + return ret; } /* copy and process stream data from source to sink buffers */ diff --git a/src/audio/eq_fir.c b/src/audio/eq_fir.c index ef3b7d2..ba42643 100644 --- a/src/audio/eq_fir.c +++ b/src/audio/eq_fir.c @@ -389,16 +389,18 @@ static int eq_fir_cmd(struct comp_dev *dev, int cmd, void *data) trace_src("cmd"); + ret = comp_set_state(dev, cmd); + if (ret < 0) + return ret; + switch (cmd) { case COMP_CMD_SET_DATA: ret = fir_cmd(dev, cdata); break; - case COMP_CMD_START: case COMP_CMD_STOP: - case COMP_CMD_PAUSE: - case COMP_CMD_RELEASE: + comp_buffer_reset(dev); + break; default: - ret = comp_set_state(dev, cmd); break; } diff --git a/src/audio/eq_iir.c b/src/audio/eq_iir.c index 4a4cff1..995721b 100644 --- a/src/audio/eq_iir.c +++ b/src/audio/eq_iir.c @@ -392,36 +392,18 @@ static int eq_iir_cmd(struct comp_dev *dev, int cmd, void *data) trace_eq_iir("cmd"); + ret = comp_set_state(dev, cmd); + if (ret < 0) + return ret; + switch (cmd) { case COMP_CMD_SET_DATA: ret = iir_cmd(dev, cdata); break; - case COMP_CMD_START: - trace_eq_iir("EFs"); - dev->state = COMP_STATE_ACTIVE; - break; case COMP_CMD_STOP: - trace_eq_iir("ESp"); - if (dev->state == COMP_STATE_ACTIVE || - dev->state == COMP_STATE_PAUSED) { - comp_buffer_reset(dev); - dev->state = COMP_STATE_READY; - } - break; - case COMP_CMD_PAUSE: - trace_eq_iir("EPe"); - /* only support pausing for running */ - if (dev->state == COMP_STATE_ACTIVE) - dev->state = COMP_STATE_PAUSED; - - break; - case COMP_CMD_RELEASE: - trace_eq_iir("ERl"); - dev->state = COMP_STATE_ACTIVE; + comp_buffer_reset(dev); break; default: - trace_eq_iir("EDf"); - ret = -EINVAL; break; } diff --git a/src/audio/host.c b/src/audio/host.c index 8aa29ad..a1d312a 100644 --- a/src/audio/host.c +++ b/src/audio/host.c @@ -278,7 +278,7 @@ static struct comp_dev *host_new(struct sof_ipc_comp *comp) /* init posn data. TODO: other fields */ hd->posn.comp_id = comp->id; - + dev->state = COMP_STATE_READY; return dev; error: @@ -549,19 +549,16 @@ static int host_cmd(struct comp_dev *dev, int cmd, void *data) trace_host("cmd"); + ret = comp_set_state(dev, cmd); + if (ret < 0) + return ret; + switch (cmd) { case COMP_CMD_PAUSE: case COMP_CMD_STOP: - if (dev->state == COMP_STATE_ACTIVE) - ret = host_stop(dev); - break; - case COMP_CMD_SUSPEND: - case COMP_CMD_RESUME: + ret = host_stop(dev); break; - case COMP_CMD_START: - case COMP_CMD_RELEASE: default: - ret = comp_set_state(dev, cmd); break; } diff --git a/src/audio/mixer.c b/src/audio/mixer.c index 85aa601..3861b7c 100644 --- a/src/audio/mixer.c +++ b/src/audio/mixer.c @@ -151,55 +151,60 @@ static int mixer_params(struct comp_dev *dev) return 0; } -static int mixer_status_change(struct comp_dev *dev/* , uint32_t target_state */) +static int mixer_source_status_count(struct comp_dev *mixer, uint32_t status) { - int finish = 0; struct comp_buffer *source; struct list_item * blist; - uint32_t stream_target = COMP_STATE_INIT; + int count = 0; - /* calculate the highest status between input streams */ - list_for_item(blist, &dev->bsource_list) { + /* count source with state == status */ + list_for_item(blist, &mixer->bsource_list) { source = container_of(blist, struct comp_buffer, sink_list); - if (source->source->state > stream_target) - stream_target = source->source->state; + if (source->source->state == status) + count++; } - if (dev->state == stream_target) - finish = 1; - else - dev->state = stream_target; + return count; +} + +static inline int mixer_sink_status(struct comp_dev *mixer) +{ + struct comp_buffer *sink; - return finish; + sink = list_first_item(&mixer->bsink_list, struct comp_buffer, + source_list); + return sink->sink->state; } /* used to pass standard and bespoke commands (with data) to component */ static int mixer_cmd(struct comp_dev *dev, int cmd, void *data) { - int finish = 0; + int ret; trace_mixer("cmd"); + ret = comp_set_state(dev, cmd); + if (ret < 0) + return ret; + switch(cmd) { case COMP_CMD_START: - trace_mixer("MSa"); - case COMP_CMD_PAUSE: case COMP_CMD_RELEASE: - case COMP_CMD_DRAIN: - case COMP_CMD_SUSPEND: - case COMP_CMD_RESUME: - finish = mixer_status_change(dev); + if (mixer_sink_status(dev) == COMP_STATE_ACTIVE) + return 1; /* no need to go downstream */ break; + case COMP_CMD_PAUSE: case COMP_CMD_STOP: - finish = mixer_status_change(dev); - if (finish == 0) - comp_buffer_reset(dev); + if (mixer_source_status_count(dev, COMP_STATE_ACTIVE) > 0) { + dev->state = COMP_STATE_ACTIVE; + return 1; /* no need to go downstream */ + } break; default: break; } - return finish; + return 0; /* send cmd downstream */ } /* diff --git a/src/audio/src.c b/src/audio/src.c index bc32f95..1f84a5a 100644 --- a/src/audio/src.c +++ b/src/audio/src.c @@ -292,6 +292,7 @@ static struct comp_dev *src_new(struct sof_ipc_comp *comp) for (i = 0; i < PLATFORM_MAX_CHANNELS; i++) src_polyphase_reset(&cd->src[i]); + dev->state = COMP_STATE_READY; return dev; } @@ -489,17 +490,16 @@ static int src_cmd(struct comp_dev *dev, int cmd, void *data) struct sof_ipc_ctrl_data *cdata = data; int ret = 0; - trace_src("SCm"); + trace_src("cmd"); + + ret = comp_set_state(dev, cmd); + if (ret < 0) + return ret; switch (cmd) { case COMP_CMD_SET_VALUE: return src_ctrl_cmd(dev, cdata); - case COMP_CMD_START: - case COMP_CMD_STOP: - case COMP_CMD_PAUSE: - case COMP_CMD_RELEASE: default: - ret = comp_set_state(dev, cmd); break; } @@ -564,7 +564,7 @@ static int src_reset(struct comp_dev *dev) for (i = 0; i < PLATFORM_MAX_CHANNELS; i++) src_polyphase_reset(&cd->src[i]); - dev->state = COMP_STATE_INIT; + dev->state = COMP_STATE_READY; return 0; } diff --git a/src/audio/tone.c b/src/audio/tone.c index aec8a72..94d9aa7 100644 --- a/src/audio/tone.c +++ b/src/audio/tone.c @@ -395,6 +395,7 @@ static struct comp_dev *tone_new(struct sof_ipc_comp *comp) /* Reset tone generator and set channels volumes to default */ tonegen_reset(&cd->sg); + dev->state = COMP_STATE_READY; return dev; } @@ -472,17 +473,16 @@ static int tone_cmd(struct comp_dev *dev, int cmd, void *data) struct sof_ipc_ctrl_data *cdata = data; int ret = 0; - trace_tone("tri"); + trace_tone("cmd"); + + ret = comp_set_state(dev, cmd); + if (ret < 0) + return ret; switch (cmd) { case COMP_CMD_SET_VALUE: return tone_ctrl_cmd(dev, cdata); - case COMP_CMD_START: - case COMP_CMD_STOP: - case COMP_CMD_PAUSE: - case COMP_CMD_RELEASE: default: - ret = comp_set_state(dev, cmd); break; } @@ -550,7 +550,7 @@ static int tone_reset(struct comp_dev *dev) /* Initialize with the defaults */ tonegen_reset(&cd->sg); - dev->state = COMP_STATE_INIT; + dev->state = COMP_STATE_READY; return 0; } diff --git a/src/audio/volume.c b/src/audio/volume.c index 45bbe27..960a144 100644 --- a/src/audio/volume.c +++ b/src/audio/volume.c @@ -466,25 +466,22 @@ static int volume_ctrl_get_cmd(struct comp_dev *dev, struct sof_ipc_ctrl_data *c static int volume_cmd(struct comp_dev *dev, int cmd, void *data) { struct sof_ipc_ctrl_data *cdata = data; - int ret = 0; + int ret; trace_volume("cmd"); + ret = comp_set_state(dev, cmd); + if (ret < 0) + return ret; + switch (cmd) { case COMP_CMD_SET_VALUE: return volume_ctrl_set_cmd(dev, cdata); case COMP_CMD_GET_VALUE: return volume_ctrl_get_cmd(dev, cdata); - case COMP_CMD_START: - case COMP_CMD_STOP: - case COMP_CMD_PAUSE: - case COMP_CMD_RELEASE: default: - ret = comp_set_state(dev, cmd); - break; + return ret; } - - return ret; } /* copy and process stream data from source to sink buffers */ diff --git a/src/include/reef/audio/component.h b/src/include/reef/audio/component.h index d092210..5ae8084 100644 --- a/src/include/reef/audio/component.h +++ b/src/include/reef/audio/component.h @@ -76,7 +76,6 @@ #define COMP_CMD_START 1 /* start component stream */ #define COMP_CMD_PAUSE 2 /* immediately pause the component stream */ #define COMP_CMD_RELEASE 3 /* release paused component stream */ -#define COMP_CMD_DRAIN 4 /* drain component buffers */ #define COMP_CMD_SUSPEND 5 /* suspend component */ #define COMP_CMD_RESUME 6 /* resume component */ diff --git a/src/ipc/intel-ipc.c b/src/ipc/intel-ipc.c index 752ea0c..92dfd73 100644 --- a/src/ipc/intel-ipc.c +++ b/src/ipc/intel-ipc.c @@ -394,9 +394,6 @@ static int ipc_stream_trigger(uint32_t header) case iCS(SOF_IPC_STREAM_TRIG_RELEASE): cmd = COMP_CMD_RELEASE; break; - case iCS(SOF_IPC_STREAM_TRIG_DRAIN): - cmd = COMP_CMD_DRAIN; - break; /* XRUN is special case- TODO */ case iCS(SOF_IPC_STREAM_TRIG_XRUN): return 0; -- 2.11.0

1 0

[Sound-open-firmware] [PATCH] comps: make sure correct state transitions are used and verified.
by Liam Girdwood 21 Sep '17

21 Sep '17

Use the correct component states and verify this at runtime. Signed-off-by: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> --- src/audio/component.c | 18 ++++++++++++++---- src/audio/host.c | 10 ++++------ src/audio/mixer.c | 6 ++++-- src/audio/volume.c | 3 ++- 4 files changed, 24 insertions(+), 13 deletions(-) diff --git a/src/audio/component.c b/src/audio/component.c index fd0d65e..1cf4799 100644 --- a/src/audio/component.c +++ b/src/audio/component.c @@ -122,13 +122,23 @@ int comp_set_state(struct comp_dev *dev, int cmd) switch (cmd) { case COMP_CMD_START: + if (dev->state == COMP_STATE_PREPARE) { + dev->state = COMP_STATE_ACTIVE; + } else { + trace_comp_error("CES"); + ret = -EINVAL; + } + break; case COMP_CMD_RELEASE: - dev->state = COMP_STATE_ACTIVE; + if (dev->state == COMP_STATE_PAUSED) { + dev->state = COMP_STATE_ACTIVE; + } else { + trace_comp_error("CEr"); + ret = -EINVAL; + } break; case COMP_CMD_STOP: - if (dev->state == COMP_STATE_ACTIVE || - dev->state == COMP_STATE_PAUSED) { - comp_buffer_reset(dev); + if (dev->state == COMP_STATE_ACTIVE) { dev->state = COMP_STATE_READY; } else { trace_comp_error("CEs"); diff --git a/src/audio/host.c b/src/audio/host.c index 36e4c29..8aa29ad 100644 --- a/src/audio/host.c +++ b/src/audio/host.c @@ -527,7 +527,7 @@ static int host_stop(struct comp_dev *dev) /* now reset downstream buffer */ comp_buffer_reset(dev); - dev->state = COMP_STATE_READY; + dev->state = COMP_STATE_PAUSED; return 0; } @@ -549,18 +549,16 @@ static int host_cmd(struct comp_dev *dev, int cmd, void *data) trace_host("cmd"); - // TODO: align cmd macros. switch (cmd) { + case COMP_CMD_PAUSE: case COMP_CMD_STOP: - if (dev->state == COMP_STATE_ACTIVE || - dev->state == COMP_STATE_PAUSED) + if (dev->state == COMP_STATE_ACTIVE) ret = host_stop(dev); break; case COMP_CMD_SUSPEND: case COMP_CMD_RESUME: break; case COMP_CMD_START: - case COMP_CMD_PAUSE: case COMP_CMD_RELEASE: default: ret = comp_set_state(dev, cmd); @@ -616,7 +614,7 @@ static int host_reset(struct comp_dev *dev) hd->host_pos = NULL; hd->source = NULL; hd->sink = NULL; - dev->state = COMP_STATE_INIT; + dev->state = COMP_STATE_READY; return 0; } diff --git a/src/audio/mixer.c b/src/audio/mixer.c index 30ac8cc..85aa601 100644 --- a/src/audio/mixer.c +++ b/src/audio/mixer.c @@ -99,7 +99,7 @@ static struct comp_dev *mixer_new(struct sof_ipc_comp *comp) } comp_set_drvdata(dev, md); - + dev->state = COMP_STATE_READY; return dev; } @@ -274,7 +274,7 @@ static int mixer_reset(struct comp_dev *dev) return 1; /* should not reset the downstream components */ } - dev->state = COMP_STATE_INIT; + dev->state = COMP_STATE_READY; return 0; } @@ -312,6 +312,8 @@ static int mixer_prepare(struct comp_dev *dev) } } + dev->state = COMP_STATE_PREPARE; + /* prepare downstream */ return downstream; } diff --git a/src/audio/volume.c b/src/audio/volume.c index 13cba42..45bbe27 100644 --- a/src/audio/volume.c +++ b/src/audio/volume.c @@ -332,6 +332,7 @@ static struct comp_dev *volume_new(struct sof_ipc_comp *comp) cd->tvolume[i] = VOL_MAX; } + dev->state = COMP_STATE_READY; return dev; } @@ -649,7 +650,7 @@ static int volume_reset(struct comp_dev *dev) { trace_volume("res"); - dev->state = COMP_STATE_INIT; + dev->state = COMP_STATE_READY; return 0; } -- 2.11.0

1 0

[Sound-open-firmware] [PATCH] timer: use 64 bit timestamps
by Liam Girdwood 21 Sep '17

21 Sep '17

Use 64bit timestamps now that 64 bit timers are implemented. Signed-off-by: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> --- src/platform/baytrail/timer.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/platform/baytrail/timer.c b/src/platform/baytrail/timer.c index 17c1f0b..ce58fc5 100644 --- a/src/platform/baytrail/timer.c +++ b/src/platform/baytrail/timer.c @@ -172,7 +172,7 @@ void platform_host_timestamp(struct comp_dev *host, /* get host postion */ err = comp_position(host, posn); if (err == 0) - posn->flags |= SOF_TIME_HOST_VALID; + posn->flags |= SOF_TIME_HOST_VALID | SOF_TIME_HOST_64; } /* get timestamp for DAI stream DMA position */ @@ -187,15 +187,15 @@ void platform_dai_timestamp(struct comp_dev *dai, posn->flags |= SOF_TIME_DAI_VALID; /* get SSP wallclock - DAI sets this to stream start value */ - posn->wallclock = shim_read(SHIM_EXT_TIMER_STAT) - posn->wallclock; - posn->flags |= SOF_TIME_WALL_VALID; + posn->wallclock = platform_timer_get(platform_timer) - posn->wallclock; + posn->flags |= SOF_TIME_WALL_VALID | SOF_TIME_WALL_64; } /* get current wallclock for componnent */ void platform_dai_wallclock(struct comp_dev *dai, uint64_t *wallclock) { /* only 1 wallclock on BYT */ - *wallclock = shim_read(SHIM_EXT_TIMER_STAT); + *wallclock = platform_timer_get(platform_timer); } static int platform_timer_register(struct timer *timer, -- 2.11.0

1 0

[Sound-open-firmware] [PATCH] ssp: irq: fix handler comment and remove FIFO empty loop
by Liam Girdwood 21 Sep '17

21 Sep '17

Comment is wrong and FIFO empty not needed in IRQ handler. Signed-off-by: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> --- src/drivers/ssp.c | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/src/drivers/ssp.c b/src/drivers/ssp.c index b87806e..7538edd 100644 --- a/src/drivers/ssp.c +++ b/src/drivers/ssp.c @@ -370,19 +370,14 @@ static int ssp_trigger(struct dai *dai, int cmd, int direction) return 0; } -/* The IRQ handler allows the SSP port to drain the playback FIFO to make sure - * every sample has been played */ +/* clear IRQ sources atm */ static void ssp_irq_handler(void *data) { struct dai *dai = data; - int i; + trace_ssp("irq"); trace_value(ssp_read(dai, SSSR)); - /* empty Rx FIFO */ - for (i = 0; i < 16; i++) - ssp_read(dai, SSDR); - /* clear IRQ */ ssp_write(dai, SSSR, ssp_read(dai, SSSR)); platform_interrupt_clear(ssp_irq(dai), 1); -- 2.11.0

1 0

[Sound-open-firmware] [PATCH] dai: cache: Make sure we only writeback the amount copied
by Liam Girdwood 21 Sep '17

21 Sep '17

The DAI IRQ hanlder was always writing back full period size even when less data was copied. Only writeback the amount of data copied. Signed-off-by: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> --- src/audio/dai.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/audio/dai.c b/src/audio/dai.c index ebec519..cf8de72 100644 --- a/src/audio/dai.c +++ b/src/audio/dai.c @@ -103,7 +103,7 @@ static void dai_dma_cb(void *data, uint32_t type, struct dma_sg_elem *next) comp_update_buffer_consume(dma_buffer, copied_size); /* writeback buffer contents from cache */ - dcache_writeback_region(dma_buffer->r_ptr, dd->period_bytes); + dcache_writeback_region(dma_buffer->r_ptr, copied_size); /* update host position(in bytes offset) for drivers */ dev->position += copied_size; @@ -498,6 +498,7 @@ static int dai_position(struct comp_dev *dev, struct sof_ipc_stream_posn *posn) /* set stream start wallclock */ posn->wallclock = dd->wallclock; + return 0; } -- 2.11.0

1 0

[Sound-open-firmware] [PATCH] platform: byt: Add support for masking all DMACs and SSPs
by Liam Girdwood 20 Sep '17

20 Sep '17

Add the remaining DMACs and SSP to platform IRQ mask/unmask and clear. Signed-off-by: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> --- src/platform/baytrail/platform.c | 99 ++++++++++++++++++++++++++++++++++++++-- 1 file changed, 96 insertions(+), 3 deletions(-) diff --git a/src/platform/baytrail/platform.c b/src/platform/baytrail/platform.c index ce8a689..651c3ce 100644 --- a/src/platform/baytrail/platform.c +++ b/src/platform/baytrail/platform.c @@ -105,6 +105,18 @@ int platform_boot_complete(uint32_t boot_message) void platform_interrupt_clear(uint32_t irq, uint32_t mask) { switch (irq) { + case IRQ_NUM_EXT_SSP0: + shim_write(SHIM_PISR, mask << 3); + interrupt_clear(irq); + break; + case IRQ_NUM_EXT_SSP1: + shim_write(SHIM_PISR, mask << 4); + interrupt_clear(irq); + break; + case IRQ_NUM_EXT_SSP2: + shim_write(SHIM_PISR, mask << 5); + interrupt_clear(irq); + break; case IRQ_NUM_EXT_DMAC0: shim_write(SHIM_PISR, mask << 16); interrupt_clear(irq); @@ -118,6 +130,18 @@ void platform_interrupt_clear(uint32_t irq, uint32_t mask) shim_write(SHIM_PISRH, mask << 0); interrupt_clear(irq); break; + case IRQ_NUM_EXT_SSP3: + shim_write(SHIM_PISRH, mask << 8); + interrupt_clear(irq); + break; + case IRQ_NUM_EXT_SSP4: + shim_write(SHIM_PISRH, mask << 9); + interrupt_clear(irq); + break; + case IRQ_NUM_EXT_SSP5: + shim_write(SHIM_PISRH, mask << 10); + interrupt_clear(irq); + break; #endif default: break; @@ -132,12 +156,76 @@ uint32_t platform_interrupt_get_enabled(void) void platform_interrupt_mask(uint32_t irq, uint32_t mask) { - + switch (irq) { + case IRQ_NUM_EXT_SSP0: + shim_write(SHIM_PIMR, mask << 3); + break; + case IRQ_NUM_EXT_SSP1: + shim_write(SHIM_PIMR, mask << 4); + break; + case IRQ_NUM_EXT_SSP2: + shim_write(SHIM_PIMR, mask << 5); + break; + case IRQ_NUM_EXT_DMAC0: + shim_write(SHIM_PIMR, mask << 16); + break; + case IRQ_NUM_EXT_DMAC1: + shim_write(SHIM_PIMR, mask << 24); + break; +#if defined CONFIG_CHERRYTRAIL + case IRQ_NUM_EXT_DMAC2: + shim_write(SHIM_PISMH, mask << 8); + break; + case IRQ_NUM_EXT_SSP3: + shim_write(SHIM_PIMRH, mask << 0); + break; + case IRQ_NUM_EXT_SSP4: + shim_write(SHIM_PIMRH, mask << 1); + break; + case IRQ_NUM_EXT_SSP5: + shim_write(SHIM_PIMRH, mask << 2); + break; +#endif + default: + break; + } } void platform_interrupt_unmask(uint32_t irq, uint32_t mask) { - + switch (irq) { + case IRQ_NUM_EXT_SSP0: + shim_write(SHIM_PIMR, shim_read(SHIM_PIMR) & ~(mask << 3)); + break; + case IRQ_NUM_EXT_SSP1: + shim_write(SHIM_PIMR, shim_read(SHIM_PIMR) & ~(mask << 4)); + break; + case IRQ_NUM_EXT_SSP2: + shim_write(SHIM_PIMR, shim_read(SHIM_PIMR) & ~(mask << 5)); + break; + case IRQ_NUM_EXT_DMAC0: + shim_write(SHIM_PIMR, shim_read(SHIM_PIMR) & ~(mask << 16)); + break; + case IRQ_NUM_EXT_DMAC1: + shim_write(SHIM_PIMR, shim_read(SHIM_PIMR) & ~(mask << 24)); + break; +#if defined CONFIG_CHERRYTRAIL + case IRQ_NUM_EXT_DMAC2: + shim_write(SHIM_PIMRH, shim_read(SHIM_PIMRH) & ~(mask << 8)); + break; + case IRQ_NUM_EXT_SSP3: + shim_write(SHIM_PIMRH, shim_read(SHIM_PIMRH) & ~(mask << 0)); + break; + case IRQ_NUM_EXT_SSP4: + shim_write(SHIM_PIMRH, shim_read(SHIM_PIMRH) & ~(mask << 1)); + break; + case IRQ_NUM_EXT_SSP5: + shim_write(SHIM_PIMRH, shim_read(SHIM_PIMRH) & ~(mask << 2)); + break; +#endif + default: + break; + } } int platform_init(struct reef *reef) @@ -213,9 +301,14 @@ int platform_init(struct reef *reef) dma_probe(dmac2); #endif - /* mask SSP interrupts */ + /* mask SSP 0 - 2 interrupts */ shim_write(SHIM_PIMR, shim_read(SHIM_PIMR) | 0x00000038); +#if defined CONFIG_CHERRYTRAIL + /* mask SSP 3 - 5 interrupts */ + shim_write(SHIM_PIMRH, shim_read(SHIM_PIMRH) | 0x00000700); +#endif + /* init SSP ports */ trace_point(TRACE_BOOT_PLATFORM_SSP); ssp0 = dai_get(SOF_DAI_INTEL_SSP, 0); -- 2.11.0

2 15

[Sound-open-firmware] [PATCH v2 1/3] interrupt-map: add cpu param for REEF_IRQ generation
by Keyon Jie 20 Sep '17

20 Sep '17

This will be needed for multi-core, add it here. Signed-off-by: Keyon Jie <yang.jie(a)linux.intel.com> --- src/include/reef/interrupt-map.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/include/reef/interrupt-map.h b/src/include/reef/interrupt-map.h index a292279..b968258 100644 --- a/src/include/reef/interrupt-map.h +++ b/src/include/reef/interrupt-map.h @@ -31,6 +31,8 @@ #ifndef __INCLUDE_INTERRUPT_MAP__ #define __INCLUDE_INTERRUPT_MAP__ +#include <config.h> + #define REEF_IRQ_BIT_SHIFT 24 #define REEF_IRQ_LEVEL_SHIFT 16 #define REEF_IRQ_CPU_SHIFT 8 @@ -43,6 +45,7 @@ #define REEF_IRQ(_bit, _level, _cpu, _number) \ ((_bit << REEF_IRQ_BIT_SHIFT) \ | (_level << REEF_IRQ_LEVEL_SHIFT)\ + | (_cpu << REEF_IRQ_CPU_SHIFT)\ | (_number << REEF_IRQ_NUM_SHIFT)) #ifdef CONFIG_IRQ_MAP -- 2.11.0

2 3

[Sound-open-firmware] [PATCH 1/4] Add DMA callback for dma_copy_to_host() and dma_copy_from_host().
by yan.wang＠linux.intel.com 20 Sep '17

20 Sep '17

From: Yan Wang <yan.wang(a)linux.intel.com> It is necessary to add dma callback to set completion_t variable to notify DMA copying is finished. Otherwise it will always enter into error handler. Signed-off-by: Yan Wang <yan.wang(a)linux.intel.com> --- src/ipc/dma-copy.c | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/src/ipc/dma-copy.c b/src/ipc/dma-copy.c index b8d27be..d295089 100644 --- a/src/ipc/dma-copy.c +++ b/src/ipc/dma-copy.c @@ -64,6 +64,14 @@ static struct dma_sg_elem *sg_get_elem_at(struct dma_sg_config *host_sg, return NULL; } +static void dma_complete(void *data, uint32_t type, struct dma_sg_elem *next) +{ + completion_t *comp = (completion_t *)data; + + if (type == DMA_IRQ_TYPE_LLIST) + wait_completed(comp); +} + int dma_copy_to_host(struct dma_sg_config *host_sg, int32_t host_offset, void *local_ptr, int32_t size) { @@ -102,6 +110,8 @@ int dma_copy_to_host(struct dma_sg_config *host_sg, int32_t host_offset, local_sg_elem.size = HOST_PAGE_SIZE - offset; list_item_prepend(&local_sg_elem.list, &config.elem_list); + dma_set_cb(dma, chan, DMA_IRQ_TYPE_LLIST, dma_complete, &complete); + /* transfer max PAGE size at a time to SG buffer */ while (size > 0) { @@ -179,6 +189,8 @@ int dma_copy_from_host(struct dma_sg_config *host_sg, int32_t host_offset, local_sg_elem.size = HOST_PAGE_SIZE - offset; list_item_prepend(&local_sg_elem.list, &config.elem_list); + dma_set_cb(dma, chan, DMA_IRQ_TYPE_LLIST, dma_complete, &complete); + /* transfer max PAGE size at a time to SG buffer */ while (size > 0) { -- 2.7.4

2 1

[Sound-open-firmware] [PATCH 1/5] SRC: Remove old coefficient set with bug
by Seppo Ingalsuo 20 Sep '17

20 Sep '17

This set prevented passtrough conversions to work, e.g. 48 kHz to 48 kHz due to a bug in table. This set is replaced with coefficients with a profile keyword to help selecting suitable set for a platform. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo(a)linux.intel.com> --- .../coefficients/src/src_int16_1_3_4375_5000.h | 251 ----- .../coefficients/src/src_int16_20_21_4020_5000.h | 1066 -------------------- .../coefficients/src/src_int16_21_20_4020_5000.h | 1056 ------------------- .../coefficients/src/src_int16_3_1_4375_5000.h | 252 ----- .../coefficients/src/src_int16_7_8_4375_5000.h | 657 ------------ .../coefficients/src/src_int16_8_7_4375_5000.h | 662 ------------ .../reef/audio/coefficients/src/src_int16_define.h | 11 - .../reef/audio/coefficients/src/src_int16_table.h | 32 - 8 files changed, 3987 deletions(-) delete mode 100644 src/include/reef/audio/coefficients/src/src_int16_1_3_4375_5000.h delete mode 100644 src/include/reef/audio/coefficients/src/src_int16_20_21_4020_5000.h delete mode 100644 src/include/reef/audio/coefficients/src/src_int16_21_20_4020_5000.h delete mode 100644 src/include/reef/audio/coefficients/src/src_int16_3_1_4375_5000.h delete mode 100644 src/include/reef/audio/coefficients/src/src_int16_7_8_4375_5000.h delete mode 100644 src/include/reef/audio/coefficients/src/src_int16_8_7_4375_5000.h delete mode 100644 src/include/reef/audio/coefficients/src/src_int16_define.h delete mode 100644 src/include/reef/audio/coefficients/src/src_int16_table.h diff --git a/src/include/reef/audio/coefficients/src/src_int16_1_3_4375_5000.h b/src/include/reef/audio/coefficients/src/src_int16_1_3_4375_5000.h deleted file mode 100644 index 34e177c..0000000 --- a/src/include/reef/audio/coefficients/src/src_int16_1_3_4375_5000.h +++ /dev/null @@ -1,251 +0,0 @@ -const int16_t src_int16_1_3_4375_5000_fir[245] = { - 0, - 0, - -1, - 0, - 1, - 1, - 1, - -1, - -2, - -1, - 1, - 3, - 2, - 0, - -4, - -4, - 0, - 4, - 6, - 2, - -5, - -8, - -4, - 5, - 11, - 7, - -4, - -13, - -12, - 1, - 16, - 17, - 2, - -17, - -23, - -8, - 17, - 30, - 16, - -15, - -37, - -26, - 10, - 42, - 39, - -2, - -46, - -52, - -10, - 47, - 67, - 26, - -44, - -82, - -47, - 36, - 95, - 72, - -21, - -104, - -100, - -1, - 109, - 129, - 31, - -106, - -159, - -69, - 94, - 187, - 115, - -71, - -210, - -167, - 35, - 225, - 225, - 16, - -228, - -284, - -82, - 216, - 342, - 163, - -184, - -395, - -259, - 130, - 437, - 368, - -49, - -463, - -489, - -62, - 467, - 616, - 208, - -439, - -748, - -393, - 372, - 880, - 624, - -252, - -1007, - -914, - 60, - 1125, - 1285, - 240, - -1228, - -1791, - -720, - 1313, - 2565, - 1582, - -1377, - -4078, - -3627, - 1416, - 9650, - 17303, - 20414, - 17303, - 9650, - 1416, - -3627, - -4078, - -1377, - 1582, - 2565, - 1313, - -720, - -1791, - -1228, - 240, - 1285, - 1125, - 60, - -914, - -1007, - -252, - 624, - 880, - 372, - -393, - -748, - -439, - 208, - 616, - 467, - -62, - -489, - -463, - -49, - 368, - 437, - 130, - -259, - -395, - -184, - 163, - 342, - 216, - -82, - -284, - -228, - 16, - 225, - 225, - 35, - -167, - -210, - -71, - 115, - 187, - 94, - -69, - -159, - -106, - 31, - 129, - 109, - -1, - -100, - -104, - -21, - 72, - 95, - 36, - -47, - -82, - -44, - 26, - 67, - 47, - -10, - -52, - -46, - -2, - 39, - 42, - 10, - -26, - -37, - -15, - 16, - 30, - 17, - -8, - -23, - -17, - 2, - 17, - 16, - 1, - -12, - -13, - -4, - 7, - 11, - 5, - -4, - -8, - -5, - 2, - 6, - 4, - 0, - -4, - -4, - 0, - 2, - 3, - 1, - -1, - -2, - -1, - 1, - 1, - 1, - 0, - -1, - 0, - 0 - -}; -struct src_stage src_int16_1_3_4375_5000 = { - 1, 0, 1, 245, 245, 3, 1, 0, 1, - src_int16_1_3_4375_5000_fir}; diff --git a/src/include/reef/audio/coefficients/src/src_int16_20_21_4020_5000.h b/src/include/reef/audio/coefficients/src/src_int16_20_21_4020_5000.h deleted file mode 100644 index 51aade7..0000000 --- a/src/include/reef/audio/coefficients/src/src_int16_20_21_4020_5000.h +++ /dev/null @@ -1,1066 +0,0 @@ -const int16_t src_int16_20_21_4020_5000_fir[1060] = { - 1, - -2, - 2, - 1, - -8, - 19, - -34, - 46, - -47, - 28, - 20, - -96, - 188, - -274, - 316, - -277, - 125, - 150, - -525, - 939, - -1288, - 1439, - -1233, - 472, - 1182, - -4843, - 21640, - 18847, - -5651, - 2095, - -277, - -717, - 1157, - -1205, - 997, - -663, - 311, - -18, - -175, - 261, - -260, - 203, - -125, - 51, - 3, - -31, - 38, - -32, - 21, - -10, - 3, - 0, - -1, - 1, - -2, - 2, - 0, - -6, - 18, - -34, - 49, - -54, - 40, - 3, - -78, - 175, - -273, - 336, - -323, - 197, - 61, - -439, - 882, - -1296, - 1547, - -1470, - 856, - 658, - -4253, - 22894, - 17336, - -5873, - 2472, - -629, - -447, - 989, - -1132, - 999, - -713, - 382, - -88, - -120, - 228, - -247, - 205, - -135, - 64, - -9, - -23, - 34, - -31, - 21, - -11, - 4, - 0, - -1, - 1, - -2, - 3, - -2, - -4, - 16, - -33, - 51, - -61, - 52, - -14, - -58, - 159, - -268, - 351, - -364, - 266, - -30, - -343, - 808, - -1280, - 1629, - -1687, - 1237, - 97, - -3539, - 24033, - 15769, - -5978, - 2790, - -960, - -177, - 807, - -1040, - 983, - -749, - 444, - -155, - -64, - 191, - -229, - 203, - -143, - 76, - -21, - -14, - 29, - -29, - 21, - -12, - 5, - -1, - -1, - 1, - -3, - 4, - -3, - -2, - 14, - -32, - 52, - -66, - 63, - -32, - -37, - 139, - -257, - 359, - -400, - 333, - -123, - -238, - 718, - -1241, - 1683, - -1879, - 1608, - -491, - -2701, - 25047, - 14163, - -5972, - 3046, - -1263, - 89, - 616, - -932, - 950, - -771, - 497, - -217, - -9, - 151, - -208, - 198, - -148, - 86, - -32, - -5, - 23, - -26, - 20, - -12, - 5, - -1, - 0, - 1, - -3, - 4, - -4, - 0, - 11, - -29, - 52, - -70, - 74, - -49, - -14, - 115, - -241, - 361, - -428, - 394, - -215, - -127, - 613, - -1178, - 1708, - -2042, - 1962, - -1097, - -1745, - 25925, - 12534, - -5861, - 3238, - -1534, - 347, - 417, - -810, - 900, - -778, - 540, - -274, - 46, - 109, - -183, - 189, - -151, - 95, - -41, - 3, - 18, - -23, - 19, - -12, - 6, - -2, - 0, - 1, - -3, - 5, - -6, - 2, - 8, - -27, - 51, - -74, - 83, - -66, - 10, - 90, - -221, - 356, - -450, - 450, - -306, - -11, - 494, - -1092, - 1703, - -2173, - 2293, - -1713, - -675, - 26656, - 10897, - -5655, - 3365, - -1770, - 591, - 216, - -676, - 836, - -771, - 573, - -326, - 98, - 67, - -156, - 177, - -150, - 101, - -50, - 11, - 12, - -20, - 18, - -12, - 6, - -2, - 0, - 1, - -3, - 5, - -7, - 4, - 5, - -23, - 49, - -76, - 92, - -82, - 33, - 62, - -196, - 344, - -464, - 498, - -393, - 108, - 364, - -984, - 1666, - -2267, - 2593, - -2327, - 502, - 27233, - 9269, - -5360, - 3428, - -1968, - 819, - 16, - -532, - 757, - -751, - 594, - -370, - 148, - 24, - -127, - 162, - -147, - 106, - -58, - 19, - 6, - -16, - 17, - -12, - 6, - -2, - 0, - 1, - -3, - 6, - -8, - 6, - 2, - -20, - 46, - -76, - 98, - -97, - 57, - 32, - -167, - 326, - -469, - 539, - -475, - 228, - 224, - -856, - 1597, - -2323, - 2856, - -2928, - 1779, - 27649, - 7665, - -4988, - 3428, - -2124, - 1027, - -180, - -383, - 667, - -718, - 604, - -406, - 194, - -18, - -96, - 145, - -142, - 108, - -64, - 26, - 0, - -13, - 15, - -12, - 7, - -3, - 1, - 1, - -3, - 6, - -9, - 9, - -2, - -15, - 42, - -75, - 104, - -111, - 81, - 0, - -134, - 302, - -466, - 570, - -550, - 345, - 76, - -709, - 1498, - -2337, - 3076, - -3506, - 3145, - 27901, - 6101, - -4547, - 3367, - -2238, - 1211, - -368, - -230, - 567, - -672, - 604, - -435, - 235, - -59, - -64, - 125, - -134, - 109, - -70, - 32, - -5, - -9, - 13, - -11, - 7, - -3, - 1, - 1, - -3, - 6, - -10, - 11, - -5, - -10, - 38, - -73, - 107, - -123, - 104, - -32, - -98, - 271, - -455, - 592, - -616, - 459, - -76, - -546, - 1368, - -2309, - 3248, - -4050, - 4589, - 27985, - 4589, - -4050, - 3248, - -2309, - 1368, - -546, - -76, - 459, - -616, - 592, - -455, - 271, - -98, - -32, - 104, - -123, - 107, - -73, - 38, - -10, - -5, - 11, - -10, - 6, - -3, - 1, - 1, - -3, - 7, - -11, - 13, - -9, - -5, - 32, - -70, - 109, - -134, - 125, - -64, - -59, - 235, - -435, - 604, - -672, - 567, - -230, - -368, - 1211, - -2238, - 3367, - -4547, - 6101, - 27901, - 3145, - -3506, - 3076, - -2337, - 1498, - -709, - 76, - 345, - -550, - 570, - -466, - 302, - -134, - 0, - 81, - -111, - 104, - -75, - 42, - -15, - -2, - 9, - -9, - 6, - -3, - 1, - 1, - -3, - 7, - -12, - 15, - -13, - 0, - 26, - -64, - 108, - -142, - 145, - -96, - -18, - 194, - -406, - 604, - -718, - 667, - -383, - -180, - 1027, - -2124, - 3428, - -4988, - 7665, - 27649, - 1779, - -2928, - 2856, - -2323, - 1597, - -856, - 224, - 228, - -475, - 539, - -469, - 326, - -167, - 32, - 57, - -97, - 98, - -76, - 46, - -20, - 2, - 6, - -8, - 6, - -3, - 1, - 0, - -2, - 6, - -12, - 17, - -16, - 6, - 19, - -58, - 106, - -147, - 162, - -127, - 24, - 148, - -370, - 594, - -751, - 757, - -532, - 16, - 819, - -1968, - 3428, - -5360, - 9269, - 27233, - 502, - -2327, - 2593, - -2267, - 1666, - -984, - 364, - 108, - -393, - 498, - -464, - 344, - -196, - 62, - 33, - -82, - 92, - -76, - 49, - -23, - 5, - 4, - -7, - 5, - -3, - 1, - 0, - -2, - 6, - -12, - 18, - -20, - 12, - 11, - -50, - 101, - -150, - 177, - -156, - 67, - 98, - -326, - 573, - -771, - 836, - -676, - 216, - 591, - -1770, - 3365, - -5655, - 10897, - 26656, - -675, - -1713, - 2293, - -2173, - 1703, - -1092, - 494, - -11, - -306, - 450, - -450, - 356, - -221, - 90, - 10, - -66, - 83, - -74, - 51, - -27, - 8, - 2, - -6, - 5, - -3, - 1, - 0, - -2, - 6, - -12, - 19, - -23, - 18, - 3, - -41, - 95, - -151, - 189, - -183, - 109, - 46, - -274, - 540, - -778, - 900, - -810, - 417, - 347, - -1534, - 3238, - -5861, - 12534, - 25925, - -1745, - -1097, - 1962, - -2042, - 1708, - -1178, - 613, - -127, - -215, - 394, - -428, - 361, - -241, - 115, - -14, - -49, - 74, - -70, - 52, - -29, - 11, - 0, - -4, - 4, - -3, - 1, - 0, - -1, - 5, - -12, - 20, - -26, - 23, - -5, - -32, - 86, - -148, - 198, - -208, - 151, - -9, - -217, - 497, - -771, - 950, - -932, - 616, - 89, - -1263, - 3046, - -5972, - 14163, - 25047, - -2701, - -491, - 1608, - -1879, - 1683, - -1241, - 718, - -238, - -123, - 333, - -400, - 359, - -257, - 139, - -37, - -32, - 63, - -66, - 52, - -32, - 14, - -2, - -3, - 4, - -3, - 1, - -1, - -1, - 5, - -12, - 21, - -29, - 29, - -14, - -21, - 76, - -143, - 203, - -229, - 191, - -64, - -155, - 444, - -749, - 983, - -1040, - 807, - -177, - -960, - 2790, - -5978, - 15769, - 24033, - -3539, - 97, - 1237, - -1687, - 1629, - -1280, - 808, - -343, - -30, - 266, - -364, - 351, - -268, - 159, - -58, - -14, - 52, - -61, - 51, - -33, - 16, - -4, - -2, - 3, - -2, - 1, - -1, - 0, - 4, - -11, - 21, - -31, - 34, - -23, - -9, - 64, - -135, - 205, - -247, - 228, - -120, - -88, - 382, - -713, - 999, - -1132, - 989, - -447, - -629, - 2472, - -5873, - 17336, - 22894, - -4253, - 658, - 856, - -1470, - 1547, - -1296, - 882, - -439, - 61, - 197, - -323, - 336, - -273, - 175, - -78, - 3, - 40, - -54, - 49, - -34, - 18, - -6, - 0, - 2, - -2, - 1, - -1, - 0, - 3, - -10, - 21, - -32, - 38, - -31, - 3, - 51, - -125, - 203, - -260, - 261, - -175, - -18, - 311, - -663, - 997, - -1205, - 1157, - -717, - -277, - 2095, - -5651, - 18847, - 21640, - -4843, - 1182, - 472, - -1233, - 1439, - -1288, - 939, - -525, - 150, - 125, - -277, - 316, - -274, - 188, - -96, - 20, - 28, - -47, - 46, - -34, - 19, - -8, - 1, - 2, - -2, - 1, - -1, - 1, - 2, - -9, - 20, - -33, - 43, - -39, - 15, - 36, - -111, - 198, - -269, - 291, - -228, - 53, - 233, - -600, - 977, - -1257, - 1308, - -980, - 93, - 1663, - -5309, - 20286, - 20286, - -5309, - 1663, - 93, - -980, - 1308, - -1257, - 977, - -600, - 233, - 53, - -228, - 291, - -269, - 198, - -111, - 36, - 15, - -39, - 43, - -33, - 20, - -9, - 2, - 1, - -1, - 0 - -}; -struct src_stage src_int16_20_21_4020_5000 = { - 1, 1, 20, 53, 1060, 21, 20, 0, 0, - src_int16_20_21_4020_5000_fir}; diff --git a/src/include/reef/audio/coefficients/src/src_int16_21_20_4020_5000.h b/src/include/reef/audio/coefficients/src/src_int16_21_20_4020_5000.h deleted file mode 100644 index 678ed4f..0000000 --- a/src/include/reef/audio/coefficients/src/src_int16_21_20_4020_5000.h +++ /dev/null @@ -1,1056 +0,0 @@ -const int16_t src_int16_21_20_4020_5000_fir[1050] = { - 1, - -3, - 5, - -9, - 11, - -9, - 0, - 19, - -51, - 96, - -151, - 208, - -253, - 270, - -235, - 130, - 64, - -357, - 750, - -1232, - 1783, - -2376, - 2996, - -3680, - 4818, - 29296, - 1867, - -2442, - 2405, - -2140, - 1762, - -1339, - 921, - -547, - 243, - -19, - -124, - 196, - -213, - 193, - -153, - 107, - -65, - 32, - -10, - -2, - 6, - -6, - 5, - -2, - 1, - -3, - 6, - -9, - 13, - -13, - 6, - 11, - -42, - 87, - -146, - 210, - -267, - 300, - -287, - 204, - -31, - -248, - 640, - -1142, - 1744, - -2434, - 3227, - -4251, - 6405, - 29032, - 527, - -1798, - 2058, - -1969, - 1705, - -1355, - 980, - -626, - 324, - -91, - -66, - 155, - -188, - 181, - -150, - 109, - -70, - 38, - -15, - 2, - 4, - -5, - 4, - -2, - 1, - -3, - 6, - -10, - 14, - -16, - 12, - 3, - -32, - 77, - -138, - 208, - -277, - 326, - -335, - 276, - -128, - -132, - 516, - -1030, - 1672, - -2450, - 3408, - -4773, - 8048, - 28594, - -709, - -1152, - 1686, - -1768, - 1619, - -1346, - 1020, - -691, - 397, - -160, - -9, - 113, - -160, - 166, - -144, - 110, - -73, - 42, - -19, - 5, - 2, - -4, - 3, - -2, - 1, - -3, - 6, - -11, - 16, - -20, - 17, - -5, - -21, - 65, - -127, - 202, - -281, - 347, - -377, - 345, - -224, - -11, - 380, - -895, - 1568, - -2421, - 3532, - -5235, - 9732, - 27989, - -1832, - -515, - 1297, - -1541, - 1506, - -1314, - 1041, - -743, - 462, - -225, - 47, - 69, - -130, - 148, - -136, - 108, - -76, - 46, - -23, - 8, - 0, - -3, - 3, - -2, - 1, - -3, - 6, - -11, - 18, - -23, - 23, - -14, - -9, - 51, - -113, - 193, - -281, - 362, - -414, - 409, - -318, - 113, - 233, - -742, - 1433, - -2346, - 3596, - -5626, - 11441, - 27221, - -2836, - 102, - 898, - -1292, - 1369, - -1259, - 1043, - -780, - 517, - -284, - 102, - 25, - -98, - 128, - -125, - 105, - -76, - 49, - -26, - 11, - -2, - -2, - 2, - -2, - 1, - -2, - 6, - -12, - 19, - -26, - 28, - -23, - 3, - 36, - -97, - 179, - -275, - 370, - -443, - 467, - -409, - 237, - 80, - -571, - 1268, - -2227, - 3596, - -5935, - 13160, - 26300, - -3715, - 690, - 495, - -1027, - 1211, - -1183, - 1026, - -802, - 561, - -337, - 153, - -18, - -66, - 106, - -113, - 99, - -76, - 51, - -29, - 13, - -4, - 0, - 2, - -1, - 0, - -2, - 6, - -12, - 20, - -28, - 34, - -31, - 15, - 20, - -79, - 162, - -264, - 372, - -466, - 517, - -494, - 360, - -79, - -385, - 1075, - -2063, - 3531, - -6153, - 14870, - 25235, - -4465, - 1240, - 97, - -751, - 1035, - -1087, - 991, - -810, - 595, - -383, - 200, - -60, - -32, - 83, - -99, - 93, - -74, - 51, - -31, - 16, - -6, - 1, - 1, - -1, - 0, - -2, - 5, - -12, - 20, - -30, - 38, - -39, - 28, - 3, - -59, - 142, - -248, - 367, - -480, - 559, - -572, - 479, - -240, - -189, - 858, - -1856, - 3397, - -6268, - 16557, - 24038, - -5085, - 1745, - -290, - -469, - 845, - -974, - 939, - -803, - 617, - -421, - 243, - -100, - 0, - 59, - -84, - 84, - -71, - 51, - -32, - 17, - -7, - 2, - 0, - -1, - 0, - -2, - 5, - -11, - 20, - -32, - 42, - -47, - 40, - -14, - -37, - 118, - -227, - 355, - -486, - 592, - -641, - 592, - -400, - 16, - 619, - -1609, - 3196, - -6275, - 18202, - 22722, - -5573, - 2198, - -660, - -186, - 644, - -846, - 872, - -782, - 628, - -450, - 280, - -138, - 32, - 34, - -67, - 75, - -66, - 50, - -33, - 19, - -9, - 3, - 0, - 0, - 0, - -1, - 4, - -11, - 20, - -33, - 46, - -54, - 52, - -32, - -14, - 92, - -201, - 337, - -483, - 615, - -700, - 696, - -556, - 226, - 363, - -1324, - 2927, - -6165, - 19788, - 21300, - -5932, - 2594, - -1006, - 93, - 437, - -706, - 790, - -747, - 627, - -471, - 312, - -172, - 63, - 10, - -50, - 64, - -61, - 48, - -33, - 20, - -10, - 4, - -1, - 0, - 0, - -1, - 4, - -10, - 20, - -33, - 48, - -61, - 64, - -50, - 10, - 63, - -172, - 312, - -471, - 627, - -747, - 790, - -706, - 437, - 93, - -1006, - 2594, - -5932, - 21300, - 19788, - -6165, - 2927, - -1324, - 363, - 226, - -556, - 696, - -700, - 615, - -483, - 337, - -201, - 92, - -14, - -32, - 52, - -54, - 46, - -33, - 20, - -11, - 4, - -1, - 0, - 0, - 0, - 3, - -9, - 19, - -33, - 50, - -66, - 75, - -67, - 34, - 32, - -138, - 280, - -450, - 628, - -782, - 872, - -846, - 644, - -186, - -660, - 2198, - -5573, - 22722, - 18202, - -6275, - 3196, - -1609, - 619, - 16, - -400, - 592, - -641, - 592, - -486, - 355, - -227, - 118, - -37, - -14, - 40, - -47, - 42, - -32, - 20, - -11, - 5, - -2, - 0, - -1, - 0, - 2, - -7, - 17, - -32, - 51, - -71, - 84, - -84, - 59, - 0, - -100, - 243, - -421, - 617, - -803, - 939, - -974, - 845, - -469, - -290, - 1745, - -5085, - 24038, - 16557, - -6268, - 3397, - -1856, - 858, - -189, - -240, - 479, - -572, - 559, - -480, - 367, - -248, - 142, - -59, - 3, - 28, - -39, - 38, - -30, - 20, - -12, - 5, - -2, - 0, - -1, - 1, - 1, - -6, - 16, - -31, - 51, - -74, - 93, - -99, - 83, - -32, - -60, - 200, - -383, - 595, - -810, - 991, - -1087, - 1035, - -751, - 97, - 1240, - -4465, - 25235, - 14870, - -6153, - 3531, - -2063, - 1075, - -385, - -79, - 360, - -494, - 517, - -466, - 372, - -264, - 162, - -79, - 20, - 15, - -31, - 34, - -28, - 20, - -12, - 6, - -2, - 0, - -1, - 2, - 0, - -4, - 13, - -29, - 51, - -76, - 99, - -113, - 106, - -66, - -18, - 153, - -337, - 561, - -802, - 1026, - -1183, - 1211, - -1027, - 495, - 690, - -3715, - 26300, - 13160, - -5935, - 3596, - -2227, - 1268, - -571, - 80, - 237, - -409, - 467, - -443, - 370, - -275, - 179, - -97, - 36, - 3, - -23, - 28, - -26, - 19, - -12, - 6, - -2, - 1, - -2, - 2, - -2, - -2, - 11, - -26, - 49, - -76, - 105, - -125, - 128, - -98, - 25, - 102, - -284, - 517, - -780, - 1043, - -1259, - 1369, - -1292, - 898, - 102, - -2836, - 27221, - 11441, - -5626, - 3596, - -2346, - 1433, - -742, - 233, - 113, - -318, - 409, - -414, - 362, - -281, - 193, - -113, - 51, - -9, - -14, - 23, - -23, - 18, - -11, - 6, - -3, - 1, - -2, - 3, - -3, - 0, - 8, - -23, - 46, - -76, - 108, - -136, - 148, - -130, - 69, - 47, - -225, - 462, - -743, - 1041, - -1314, - 1506, - -1541, - 1297, - -515, - -1832, - 27989, - 9732, - -5235, - 3532, - -2421, - 1568, - -895, - 380, - -11, - -224, - 345, - -377, - 347, - -281, - 202, - -127, - 65, - -21, - -5, - 17, - -20, - 16, - -11, - 6, - -3, - 1, - -2, - 3, - -4, - 2, - 5, - -19, - 42, - -73, - 110, - -144, - 166, - -160, - 113, - -9, - -160, - 397, - -691, - 1020, - -1346, - 1619, - -1768, - 1686, - -1152, - -709, - 28594, - 8048, - -4773, - 3408, - -2450, - 1672, - -1030, - 516, - -132, - -128, - 276, - -335, - 326, - -277, - 208, - -138, - 77, - -32, - 3, - 12, - -16, - 14, - -10, - 6, - -3, - 1, - -2, - 4, - -5, - 4, - 2, - -15, - 38, - -70, - 109, - -150, - 181, - -188, - 155, - -66, - -91, - 324, - -626, - 980, - -1355, - 1705, - -1969, - 2058, - -1798, - 527, - 29032, - 6405, - -4251, - 3227, - -2434, - 1744, - -1142, - 640, - -248, - -31, - 204, - -287, - 300, - -267, - 210, - -146, - 87, - -42, - 11, - 6, - -13, - 13, - -9, - 6, - -3, - 1, - -2, - 5, - -6, - 6, - -2, - -10, - 32, - -65, - 107, - -153, - 193, - -213, - 196, - -124, - -19, - 243, - -547, - 921, - -1339, - 1762, - -2140, - 2405, - -2442, - 1867, - 29296, - 4818, - -3680, - 2996, - -2376, - 1783, - -1232, - 750, - -357, - 64, - 130, - -235, - 270, - -253, - 208, - -151, - 96, - -51, - 19, - 0, - -9, - 11, - -9, - 5, - -3, - 1, - -3, - 5, - -8, - 8, - -5, - -5, - 26, - -58, - 102, - -153, - 202, - -235, - 235, - -181, - 55, - 156, - -457, - 844, - -1298, - 1788, - -2277, - 2720, - -3073, - 3302, - 29385, - 3302, - -3073, - 2720, - -2277, - 1788, - -1298, - 844, - -457, - 156, - 55, - -181, - 235, - -235, - 202, - -153, - 102, - -58, - 26, - -5, - -5, - 8, - -8, - 5, - -3, - 0 - -}; -struct src_stage src_int16_21_20_4020_5000 = { - 19, 20, 21, 50, 1050, 20, 21, 0, 0, - src_int16_21_20_4020_5000_fir}; diff --git a/src/include/reef/audio/coefficients/src/src_int16_3_1_4375_5000.h b/src/include/reef/audio/coefficients/src/src_int16_3_1_4375_5000.h deleted file mode 100644 index a8d5813..0000000 --- a/src/include/reef/audio/coefficients/src/src_int16_3_1_4375_5000.h +++ /dev/null @@ -1,252 +0,0 @@ -const int16_t src_int16_3_1_4375_5000_fir[246] = { - 0, - -1, - 2, - -3, - 4, - -7, - 9, - -11, - 13, - -15, - 14, - -12, - 7, - 2, - -15, - 33, - -55, - 82, - -112, - 143, - -175, - 204, - -226, - 239, - -237, - 218, - -176, - 108, - -11, - -118, - 281, - -478, - 711, - -979, - 1286, - -1636, - 2044, - -2544, - 3231, - -4424, - 8001, - 29412, - -2495, - 372, - 409, - -786, - 974, - -1051, - 1053, - -1004, - 919, - -810, - 688, - -560, - 434, - -316, - 209, - -117, - 41, - 18, - -61, - 90, - -106, - 111, - -107, - 98, - -85, - 70, - -55, - 41, - -28, - 18, - -10, - 4, - 0, - -2, - 3, - -3, - 3, - -2, - 2, - -1, - 0, - 0, - 0, - -1, - 2, - -4, - 7, - -11, - 16, - -22, - 29, - -35, - 41, - -44, - 45, - -40, - 30, - -13, - -12, - 46, - -89, - 140, - -198, - 260, - -323, - 384, - -436, - 473, - -489, - 477, - -428, - 333, - -183, - -36, - 339, - -755, - 1332, - -2180, - 3582, - -6592, - 20737, - 20737, - -6592, - 3582, - -2180, - 1332, - -755, - 339, - -36, - -183, - 333, - -428, - 477, - -489, - 473, - -436, - 384, - -323, - 260, - -198, - 140, - -89, - 46, - -12, - -13, - 30, - -40, - 45, - -44, - 41, - -35, - 29, - -22, - 16, - -11, - 7, - -4, - 2, - -1, - 0, - 0, - 0, - -1, - 2, - -2, - 3, - -3, - 3, - -2, - 0, - 4, - -10, - 18, - -28, - 41, - -55, - 70, - -85, - 98, - -107, - 111, - -106, - 90, - -61, - 18, - 41, - -117, - 209, - -316, - 434, - -560, - 688, - -810, - 919, - -1004, - 1053, - -1051, - 974, - -786, - 409, - 372, - -2495, - 29412, - 8001, - -4424, - 3231, - -2544, - 2044, - -1636, - 1286, - -979, - 711, - -478, - 281, - -118, - -11, - 108, - -176, - 218, - -237, - 239, - -226, - 204, - -175, - 143, - -112, - 82, - -55, - 33, - -15, - 2, - 7, - -12, - 14, - -15, - 13, - -11, - 9, - -7, - 4, - -3, - 2, - -1, - 0 - -}; -struct src_stage src_int16_3_1_4375_5000 = { - 0, 1, 3, 82, 246, 1, 3, 0, 0, - src_int16_3_1_4375_5000_fir}; diff --git a/src/include/reef/audio/coefficients/src/src_int16_7_8_4375_5000.h b/src/include/reef/audio/coefficients/src/src_int16_7_8_4375_5000.h deleted file mode 100644 index 4b0eb73..0000000 --- a/src/include/reef/audio/coefficients/src/src_int16_7_8_4375_5000.h +++ /dev/null @@ -1,657 +0,0 @@ -const int16_t src_int16_7_8_4375_5000_fir[651] = { - 0, - 0, - 0, - -2, - 4, - -5, - 4, - 0, - -7, - 14, - -19, - 16, - -5, - -14, - 36, - -50, - 49, - -25, - -17, - 68, - -108, - 117, - -81, - 0, - 106, - -200, - 242, - -199, - 66, - 130, - -329, - 452, - -431, - 235, - 106, - -502, - 816, - -902, - 656, - -61, - -786, - 1673, - -2295, - 2280, - -1124, - -2459, - 23097, - 16168, - -5503, - 1769, - 321, - -1397, - 1681, - -1391, - 780, - -103, - -434, - 711, - -708, - 489, - -168, - -135, - 332, - -384, - 307, - -151, - -18, - 144, - -198, - 178, - -108, - 20, - 52, - -91, - 91, - -63, - 23, - 13, - -35, - 39, - -30, - 15, - 0, - -10, - 13, - -11, - 6, - -1, - -2, - 3, - -2, - 1, - 0, - -1, - 1, - 0, - -1, - 3, - -5, - 6, - -3, - -3, - 12, - -19, - 22, - -15, - -2, - 26, - -48, - 59, - -48, - 13, - 41, - -96, - 130, - -121, - 61, - 43, - -160, - 247, - -260, - 174, - 4, - -229, - 425, - -508, - 414, - -132, - -282, - 706, - -980, - 952, - -531, - -271, - 1315, - -2341, - 2983, - -2730, - 328, - 25336, - 12012, - -5744, - 2742, - -665, - -691, - 1357, - -1419, - 1047, - -464, - -112, - 517, - -672, - 587, - -340, - 42, - 205, - -337, - 336, - -231, - 78, - 66, - -157, - 179, - -141, - 67, - 9, - -64, - 85, - -74, - 43, - -8, - -20, - 33, - -32, - 21, - -7, - -4, - 10, - -11, - 8, - -3, - 0, - 2, - -2, - 1, - -1, - -1, - 1, - -1, - 0, - 3, - -6, - 7, - -6, - 1, - 8, - -18, - 25, - -24, - 11, - 12, - -40, - 62, - -65, - 42, - 7, - -70, - 125, - -147, - 116, - -28, - -97, - 219, - -288, - 263, - -128, - -95, - 341, - -520, - 544, - -361, - -15, - 496, - -929, - 1132, - -951, - 308, - 756, - -2069, - 3342, - -4177, - 3814, - 26503, - 7795, - -5248, - 3273, - -1492, - 59, - 876, - -1263, - 1168, - -751, - 212, - 263, - -550, - 605, - -462, - 208, - 56, - -247, - 320, - -279, - 160, - -18, - -98, - 157, - -154, - 104, - -34, - -30, - 67, - -74, - 56, - -26, - -4, - 24, - -30, - 25, - -14, - 2, - 6, - -9, - 8, - -5, - 1, - 1, - -2, - 1, - -1, - -1, - 1, - -2, - 1, - 1, - -5, - 8, - -9, - 6, - 2, - -14, - 25, - -30, - 24, - -4, - -26, - 56, - -74, - 67, - -30, - -34, - 104, - -154, - 157, - -98, - -18, - 160, - -279, - 320, - -247, - 56, - 208, - -462, - 605, - -550, - 263, - 212, - -751, - 1168, - -1263, - 876, - 59, - -1492, - 3273, - -5248, - 7795, - 26503, - 3814, - -4177, - 3342, - -2069, - 756, - 308, - -951, - 1132, - -929, - 496, - -15, - -361, - 544, - -520, - 341, - -95, - -128, - 263, - -288, - 219, - -97, - -28, - 116, - -147, - 125, - -70, - 7, - 42, - -65, - 62, - -40, - 12, - 11, - -24, - 25, - -18, - 8, - 1, - -6, - 7, - -6, - 3, - 0, - -1, - 1, - -1, - -1, - 1, - -2, - 2, - 0, - -3, - 8, - -11, - 10, - -4, - -7, - 21, - -32, - 33, - -20, - -8, - 43, - -74, - 85, - -64, - 9, - 67, - -141, - 179, - -157, - 66, - 78, - -231, - 336, - -337, - 205, - 42, - -340, - 587, - -672, - 517, - -112, - -464, - 1047, - -1419, - 1357, - -691, - -665, - 2742, - -5744, - 12012, - 25336, - 328, - -2730, - 2983, - -2341, - 1315, - -271, - -531, - 952, - -980, - 706, - -282, - -132, - 414, - -508, - 425, - -229, - 4, - 174, - -260, - 247, - -160, - 43, - 61, - -121, - 130, - -96, - 41, - 13, - -48, - 59, - -48, - 26, - -2, - -15, - 22, - -19, - 12, - -3, - -3, - 6, - -5, - 3, - -1, - 0, - 1, - -1, - 0, - 1, - -2, - 3, - -2, - -1, - 6, - -11, - 13, - -10, - 0, - 15, - -30, - 39, - -35, - 13, - 23, - -63, - 91, - -91, - 52, - 20, - -108, - 178, - -198, - 144, - -18, - -151, - 307, - -384, - 332, - -135, - -168, - 489, - -708, - 711, - -434, - -103, - 780, - -1391, - 1681, - -1397, - 321, - 1769, - -5503, - 16168, - 23097, - -2459, - -1124, - 2280, - -2295, - 1673, - -786, - -61, - 656, - -902, - 816, - -502, - 106, - 235, - -431, - 452, - -329, - 130, - 66, - -199, - 242, - -200, - 106, - 0, - -81, - 117, - -108, - 68, - -17, - -25, - 49, - -50, - 36, - -14, - -5, - 16, - -19, - 14, - -7, - 0, - 4, - -5, - 4, - -2, - 0, - 0, - 0, - 0, - 1, - -2, - 4, - -3, - 1, - 4, - -10, - 15, - -15, - 9, - 6, - -24, - 40, - -45, - 32, - -1, - -44, - 86, - -106, - 90, - -31, - -59, - 153, - -213, - 206, - -114, - -48, - 236, - -382, - 418, - -300, - 32, - 322, - -650, - 816, - -710, - 288, - 395, - -1173, - 1796, - -1960, - 1348, - 436, - -4418, - 19959, - 19959, - -4418, - 436, - 1348, - -1960, - 1796, - -1173, - 395, - 288, - -710, - 816, - -650, - 322, - 32, - -300, - 418, - -382, - 236, - -48, - -114, - 206, - -213, - 153, - -59, - -31, - 90, - -106, - 86, - -44, - -1, - 32, - -45, - 40, - -24, - 6, - 9, - -15, - 15, - -10, - 4, - 1, - -3, - 4, - -2, - 1, - 0, - 0 - -}; -struct src_stage src_int16_7_8_4375_5000 = { - 1, 1, 7, 93, 651, 8, 7, 0, 0, - src_int16_7_8_4375_5000_fir}; diff --git a/src/include/reef/audio/coefficients/src/src_int16_8_7_4375_5000.h b/src/include/reef/audio/coefficients/src/src_int16_8_7_4375_5000.h deleted file mode 100644 index 1ad7797..0000000 --- a/src/include/reef/audio/coefficients/src/src_int16_8_7_4375_5000.h +++ /dev/null @@ -1,662 +0,0 @@ -const int16_t src_int16_8_7_4375_5000_fir[656] = { - 0, - 0, - 1, - -2, - 4, - -7, - 10, - -13, - 17, - -21, - 23, - -23, - 21, - -14, - 3, - 14, - -37, - 65, - -99, - 136, - -174, - 211, - -242, - 263, - -269, - 256, - -219, - 152, - -53, - -81, - 251, - -458, - 702, - -979, - 1290, - -1635, - 2021, - -2471, - 3050, - -3986, - 6580, - 29785, - -1328, - -372, - 958, - -1207, - 1295, - -1286, - 1212, - -1095, - 951, - -792, - 628, - -469, - 322, - -190, - 79, - 10, - -77, - 123, - -150, - 161, - -159, - 146, - -128, - 105, - -82, - 60, - -40, - 23, - -10, - 1, - 5, - -9, - 10, - -9, - 8, - -6, - 5, - -3, - 2, - -1, - 0, - 0, - 0, - -1, - 3, - -5, - 9, - -13, - 18, - -23, - 28, - -32, - 34, - -33, - 27, - -15, - -3, - 29, - -62, - 102, - -147, - 195, - -243, - 287, - -322, - 342, - -341, - 314, - -253, - 155, - -14, - -175, - 414, - -709, - 1065, - -1493, - 2018, - -2695, - 3672, - -5455, - 11305, - 27819, - -4054, - 1304, - -212, - -361, - 684, - -859, - 934, - -938, - 889, - -804, - 696, - -574, - 449, - -327, - 215, - -117, - 36, - 28, - -74, - 104, - -119, - 123, - -117, - 105, - -89, - 71, - -53, - 37, - -23, - 12, - -4, - -1, - 4, - -6, - 6, - -5, - 4, - -3, - 2, - -1, - -1, - 1, - -1, - 0, - 1, - -3, - 6, - -11, - 16, - -23, - 30, - -38, - 44, - -48, - 49, - -44, - 33, - -13, - -15, - 52, - -99, - 153, - -212, - 274, - -333, - 385, - -422, - 438, - -426, - 378, - -287, - 143, - 60, - -332, - 686, - -1143, - 1741, - -2566, - 3846, - -6377, - 16134, - 24716, - -5794, - 2637, - -1264, - 480, - 17, - -342, - 547, - -665, - 715, - -712, - 670, - -600, - 513, - -417, - 319, - -225, - 140, - -68, - 9, - 35, - -66, - 84, - -92, - 91, - -83, - 72, - -59, - 46, - -33, - 22, - -13, - 6, - -1, - -2, - 3, - -3, - 3, - -2, - 2, - -1, - -1, - 1, - -2, - 2, - -1, - 0, - 3, - -7, - 12, - -19, - 28, - -38, - 48, - -57, - 64, - -68, - 65, - -55, - 36, - -6, - -35, - 88, - -151, - 223, - -300, - 377, - -449, - 509, - -548, - 559, - -531, - 454, - -316, - 106, - 197, - -619, - 1209, - -2077, - 3504, - -6542, - 20718, - 20718, - -6542, - 3504, - -2077, - 1209, - -619, - 197, - 106, - -316, - 454, - -531, - 559, - -548, - 509, - -449, - 377, - -300, - 223, - -151, - 88, - -35, - -6, - 36, - -55, - 65, - -68, - 64, - -57, - 48, - -38, - 28, - -19, - 12, - -7, - 3, - 0, - -1, - 2, - -2, - 1, - -1, - -1, - 2, - -2, - 3, - -3, - 3, - -2, - -1, - 6, - -13, - 22, - -33, - 46, - -59, - 72, - -83, - 91, - -92, - 84, - -66, - 35, - 9, - -68, - 140, - -225, - 319, - -417, - 513, - -600, - 670, - -712, - 715, - -665, - 547, - -342, - 17, - 480, - -1264, - 2637, - -5794, - 24716, - 16134, - -6377, - 3846, - -2566, - 1741, - -1143, - 686, - -332, - 60, - 143, - -287, - 378, - -426, - 438, - -422, - 385, - -333, - 274, - -212, - 153, - -99, - 52, - -15, - -13, - 33, - -44, - 49, - -48, - 44, - -38, - 30, - -23, - 16, - -11, - 6, - -3, - 1, - 0, - -1, - 1, - -1, - -1, - 2, - -3, - 4, - -5, - 6, - -6, - 4, - -1, - -4, - 12, - -23, - 37, - -53, - 71, - -89, - 105, - -117, - 123, - -119, - 104, - -74, - 28, - 36, - -117, - 215, - -327, - 449, - -574, - 696, - -804, - 889, - -938, - 934, - -859, - 684, - -361, - -212, - 1304, - -4054, - 27819, - 11305, - -5455, - 3672, - -2695, - 2018, - -1493, - 1065, - -709, - 414, - -175, - -14, - 155, - -253, - 314, - -341, - 342, - -322, - 287, - -243, - 195, - -147, - 102, - -62, - 29, - -3, - -15, - 27, - -33, - 34, - -32, - 28, - -23, - 18, - -13, - 9, - -5, - 3, - -1, - 0, - 0, - 0, - -1, - 2, - -3, - 5, - -6, - 8, - -9, - 10, - -9, - 5, - 1, - -10, - 23, - -40, - 60, - -82, - 105, - -128, - 146, - -159, - 161, - -150, - 123, - -77, - 10, - 79, - -190, - 322, - -469, - 628, - -792, - 951, - -1095, - 1212, - -1286, - 1295, - -1207, - 958, - -372, - -1328, - 29785, - 6580, - -3986, - 3050, - -2471, - 2021, - -1635, - 1290, - -979, - 702, - -458, - 251, - -81, - -53, - 152, - -219, - 256, - -269, - 263, - -242, - 211, - -174, - 136, - -99, - 65, - -37, - 14, - 3, - -14, - 21, - -23, - 23, - -21, - 17, - -13, - 10, - -7, - 4, - -2, - 1, - 0, - 0, - -1, - 2, - -3, - 5, - -7, - 10, - -12, - 14, - -16, - 15, - -12, - 5, - 5, - -20, - 40, - -64, - 92, - -122, - 151, - -178, - 199, - -209, - 205, - -184, - 140, - -72, - -24, - 147, - -298, - 474, - -673, - 887, - -1112, - 1339, - -1559, - 1765, - -1947, - 2097, - -2210, - 2280, - 30458, - 2280, - -2210, - 2097, - -1947, - 1765, - -1559, - 1339, - -1112, - 887, - -673, - 474, - -298, - 147, - -24, - -72, - 140, - -184, - 205, - -209, - 199, - -178, - 151, - -122, - 92, - -64, - 40, - -20, - 5, - 5, - -12, - 15, - -16, - 14, - -12, - 10, - -7, - 5, - -3, - 2, - -1, - 0 - -}; -struct src_stage src_int16_8_7_4375_5000 = { - 6, 7, 8, 82, 656, 7, 8, 0, 0, - src_int16_8_7_4375_5000_fir}; diff --git a/src/include/reef/audio/coefficients/src/src_int16_define.h b/src/include/reef/audio/coefficients/src/src_int16_define.h deleted file mode 100644 index 65954b0..0000000 --- a/src/include/reef/audio/coefficients/src/src_int16_define.h +++ /dev/null @@ -1,11 +0,0 @@ -/* SRC constants */ -#define MAX_FIR_DELAY_SIZE 450 -#define MAX_OUT_DELAY_SIZE 401 -#define MAX_BLK_IN 21 -#define MAX_BLK_OUT 21 -#define NUM_IN_FS 3 -#define NUM_OUT_FS 3 -#define STAGE1_TIMES_MAX 21 -#define STAGE2_TIMES_MAX 21 -#define STAGE_BUF_SIZE 168 -#define NUM_ALL_COEFFICIENTS 3908 diff --git a/src/include/reef/audio/coefficients/src/src_int16_table.h b/src/include/reef/audio/coefficients/src/src_int16_table.h deleted file mode 100644 index e37dfae..0000000 --- a/src/include/reef/audio/coefficients/src/src_int16_table.h +++ /dev/null @@ -1,32 +0,0 @@ -/* SRC conversions */ -#include <reef/audio/coefficients/src/src_int16_1_3_4375_5000.h> -#include <reef/audio/coefficients/src/src_int16_3_1_4375_5000.h> -#include <reef/audio/coefficients/src/src_int16_7_8_4375_5000.h> -#include <reef/audio/coefficients/src/src_int16_8_7_4375_5000.h> -#include <reef/audio/coefficients/src/src_int16_20_21_4020_5000.h> -#include <reef/audio/coefficients/src/src_int16_21_20_4020_5000.h> - -/* SRC table */ -int16_t fir_one = 16384; -struct src_stage src_int16_1_1_0_0 = { 0, 0, 1, 1, 1, 1, 1, 0, -1, &fir_one }; -struct src_stage src_int16_0_0_0_0 = { 0, 0, 0, 0, 0, 0, 0, 0, 0, &fir_one }; -int src_in_fs[3] = { 16000, 44100, 48000}; -int src_out_fs[3] = { 16000, 44100, 48000}; -struct src_stage *src_table1[3][3] = { - { &src_int16_1_1_0_0, &src_int16_0_0_0_0, - &src_int16_1_3_4375_5000}, - { &src_int16_0_0_0_0, - &src_int16_1_1_0_0, &src_int16_21_20_4020_5000 - }, - { &src_int16_3_1_4375_5000, &src_int16_8_7_4375_5000, - &src_int16_1_1_0_0} -}; -struct src_stage *src_table2[3][3] = { - { &src_int16_1_1_0_0, &src_int16_0_0_0_0, - &src_int16_1_1_0_0}, - { &src_int16_0_0_0_0, - &src_int16_1_1_0_0, &src_int16_7_8_4375_5000 - }, - { &src_int16_1_1_0_0, &src_int16_20_21_4020_5000, - &src_int16_1_1_0_0} -}; -- 2.11.0

2 4