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

Re: [Sound-open-firmware] Request to create a tag for https://github.com/01org/osadsp-crosstool-ng
by Liam Girdwood 27 Jun '18

27 Jun '18

On Thu, 2018-06-14 at 15:44 +0000, Wang, Long1 wrote: > -----Original Message----- > From: Liam Girdwood [mailto:liam.r.girdwood@linux.intel.com] > Sent: Thursday, June 14, 2018 11:33 PM > To: Wang, Long1 <long1.wang(a)intel.com>; Pan, Xiuli <xiuli.pan(a)linux.intel.com> > ; Keyon Jie <yang.jie(a)linux.intel.com>; sound-open-firmware(a)alsa-project.org > Cc: Li, Jocelyn <jocelyn.li(a)linux.intel.com> > Subject: Re: [Sound-open-firmware] Request to create a tag for https://github. > com/01org/osadsp-crosstool-ng > > On Thu, 2018-06-14 at 15:25 +0000, Wang, Long1 wrote: > > OK. There are configure files as below. Is there missing for > > config-byt- gcc7.3-gdb7.11.2-binutils2.30? This one is a typo, use the configs below. > > config-apl-gcc7.3-gdb7.12.1 > > config-byt-gcc7.3-gdb7.12.1 > > config-cnl-gcc7.3-gdb7.12.1 > > config-hsw-gcc7.3-gdb7.12.1 > > No, why do you ask ? Liam

2 9

[Sound-open-firmware] Topology/Driver mismatch with latest SOFT patches
by Ranjani Sridharan 27 Jun '18

27 Jun '18

Hi All, Some of the latest topology commits are missing the corresponding driver updates. Please reset your local SOFT branch to commit id: 308cf3d to avoid errors. Sorry for the inconvenience. Thakns, Ranjani

1 0

[Sound-open-firmware] [PATCH v2] [RFC]platform: calculate the percentage of cpu cycle
by Wu Zhigang 27 Jun '18

27 Jun '18

calculate the percentage of cpu cyle based on the CCOUNT register. which can tell the percentage of cpu cycle usage. It is the rough calculation. Signed-off-by: Wu Zhigang <zhigang.wu(a)linux.intel.com> --- src/audio/dai.c | 22 ++++++++++++++++++++++ src/include/sof/wait.h | 8 ++++++++ src/tasks/audio.c | 10 ++++++++++ 3 files changed, 40 insertions(+) diff --git a/src/audio/dai.c b/src/audio/dai.c index f6f16d7..11b20c5 100644 --- a/src/audio/dai.c +++ b/src/audio/dai.c @@ -43,11 +43,14 @@ #include <sof/audio/component.h> #include <sof/audio/pipeline.h> #include <platform/dma.h> +#include <platform/clk.h> #include <arch/cache.h> #define DAI_PLAYBACK_STREAM 0 #define DAI_CAPTURE_STREAM 1 +#define CPU_CYCLE_PER_MS (CLK_DEFAULT_CPU_HZ / 1000) + /* tracing */ #define trace_dai(__e) trace_event(TRACE_CLASS_DAI, __e) #define trace_dai_error(__e) trace_error(TRACE_CLASS_DAI, __e) @@ -72,6 +75,22 @@ struct dai_data { uint64_t wallclock; /* wall clock at stream start */ }; +static void calc_cpu_usage(struct pipeline *p) +{ + uint32_t delta = 0; + + delta = global_cycle; + global_cycle = 0; + if (delta > CPU_CYCLE_PER_MS) + delta = CPU_CYCLE_PER_MS; + delta = (delta * 100) / CPU_CYCLE_PER_MS; + + trace_dai_error("PcT"); + + /* the percentage of the cpu usage in 1ms */ + trace_error_value(100 - delta); +} + /* this is called by DMA driver every time descriptor has completed */ static void dai_dma_cb(void *data, uint32_t type, struct dma_sg_elem *next) { @@ -81,6 +100,9 @@ static void dai_dma_cb(void *data, uint32_t type, struct dma_sg_elem *next) tracev_dai("irq"); + if (dev->params.direction == SOF_IPC_STREAM_PLAYBACK) + calc_cpu_usage(dev->pipeline); + /* stop dma copy for pause/stop/xrun */ if (dev->state != COMP_STATE_ACTIVE || dd->xrun) { diff --git a/src/include/sof/wait.h b/src/include/sof/wait.h index feb8b29..4baa8de 100644 --- a/src/include/sof/wait.h +++ b/src/include/sof/wait.h @@ -43,6 +43,7 @@ #include <sof/trace.h> #include <sof/lock.h> #include <platform/interrupt.h> +#include <xtensa/hal.h> #if DEBUG_LOCKS #define wait_atomic_check \ @@ -157,4 +158,11 @@ static inline void wait_delay(uint64_t number_of_clks) idelay(PLATFORM_DEFAULT_DELAY); } +uint32_t global_cycle; + +static inline uint32_t platform_get_cpu_count(void) +{ + return xthal_get_ccount(); +} + #endif diff --git a/src/tasks/audio.c b/src/tasks/audio.c index 5beae08..1a79808 100644 --- a/src/tasks/audio.c +++ b/src/tasks/audio.c @@ -51,6 +51,9 @@ struct audio_data { struct pipeline *p; }; +uint32_t global_cycle; +static uint32_t c1, c2; + int do_task(struct sof *sof) { #ifdef STATIC_PIPE @@ -78,13 +81,20 @@ int do_task(struct sof *sof) /* let host know DSP boot is complete */ platform_boot_complete(0); + global_cycle = 0; + /* main audio IPC processing loop */ while (1) { + c1 = platform_get_cpu_count(); + /* sleep until next IPC or DMA */ sa_enter_idle(sof); wait_for_interrupt(0); + c2 = platform_get_cpu_count(); + global_cycle += c2 - c1; + /* now process any IPC messages from host */ ipc_process_msg_queue(); -- 2.17.1

3 2

[Sound-open-firmware] thesof/crosstool-NG: Request to change makefile to regard common tool as dependencies
by Wang, Long1 25 Jun '18

25 Jun '18

Hi Liam, Here I worked out a spec file to fit thesof/crosstool-NG in Clear Linux building environment. It worked by adding below tar-balls and some changes. But except for the crosstool-ng and xtensa-overlay, the others are common tool not real source. Could you change the make file to regard those common tool as dependencies? Source0 : https://github.com/thesofproject/crosstool-ng/archive/sof-v1.2-gcc8.1.tar.gz Source1 : https://github.com/thesofproject/xtensa-overlay/archive/sof-v1.2-gcc8.1.zip Source20 : https://ftpmirror.gnu.org/gnu/autoconf/autoconf-2.69.tar.xz Source21 : https://ftpmirror.gnu.org/gnu/automake/automake-1.16.1.tar.xz Source22 : https://gmplib.org/download/gmp/gmp-6.1.2.tar.xz Source23 : http://www.mpfr.org/mpfr-3.1.6/mpfr-3.1.6.tar.xz Source24 : http://isl.gforge.inria.fr/isl-0.19.tar.xz Source25 : http://www.multiprecision.org/downloads/mpc-1.0.3.tar.gz Source26 : http://downloads.sourceforge.net/project/expat/expat/2.2.5/expat-2.2.5.tar.… Source27 : https://ftpmirror.gnu.org/gnu/ncurses/ncurses-6.0.tar.gz Source28 : https://ftpmirror.gnu.org/gnu/binutils/binutils-2.30.tar.xz Source29 : https://ftpmirror.gnu.org/gnu/gcc/gcc-8.1.0/gcc-8.1.0.tar.xz Source30 : https://ftpmirror.gnu.org/gnu/gdb/gdb-8.1.tar.xz Best Regards & Thanks, Wang, Long

2 3

[Sound-open-firmware] [PATCH] [RFC]platform: calculate the percentage of cpu cycle
by Wu Zhigang 25 Jun '18

25 Jun '18

calculate the percentage of cpu cyle based on the CCOUNT register. which can tell the percentage of cpu cycle usage. Signed-off-by: Wu Zhigang <zhigang.wu(a)linux.intel.com> --- src/audio/dai.c | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/src/audio/dai.c b/src/audio/dai.c index 99749d1..7811e38 100644 --- a/src/audio/dai.c +++ b/src/audio/dai.c @@ -72,6 +72,21 @@ struct dai_data { uint64_t wallclock; /* wall clock at stream start */ }; +/* for apollolake, it is 400MHz per 1s */ +#define CPU_CYCLE_PER_MS 400000 + +static void calc_mcps(struct pipeline *p) +{ + uint32_t cpu_cycle = 0; + uint32_t deta = 0; + + __asm__ __volatile__ ("rsr %0, CCOUNT" : "=a" (cpu_cycle) : : "memory"); + deta = cpu_cycle - p->cpu_cycle; + p->cpu_cycle = cpu_cycle; + deta = (deta * 100) / CPU_CYCLE_PER_MS; + trace_error_value(deta); /* the percentage of the cpu cycle in 1ms */ +} + /* this is called by DMA driver every time descriptor has completed */ static void dai_dma_cb(void *data, uint32_t type, struct dma_sg_elem *next) { @@ -81,6 +96,8 @@ static void dai_dma_cb(void *data, uint32_t type, struct dma_sg_elem *next) tracev_dai("irq"); + calc_mcps(dev->pipeline); + /* stop dma copy for pause/stop/xrun */ if (dev->state != COMP_STATE_ACTIVE || dd->xrun) { -- 2.17.1

5 11

[Sound-open-firmware] [PATCH v2 1/3] topology: use switch IO handler for tone switch
by Ranjani Sridharan 24 Jun '18

24 Jun '18

Use the bespoke switch IO handler in the driver for the tone switch kcontrol. Signed-off-by: Ranjani Sridharan <ranjani.sridharan(a)linux.intel.com> --- topology/sof/pipe-tone.m4 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/topology/sof/pipe-tone.m4 b/topology/sof/pipe-tone.m4 index 53cef5a..fc5eb00 100644 --- a/topology/sof/pipe-tone.m4 +++ b/topology/sof/pipe-tone.m4 @@ -30,7 +30,7 @@ C_CONTROLMIXER(Tone Volume, PIPELINE_ID, # Switch type Mixer Control with max value of 1 C_CONTROLMIXER(Tone Switch, PIPELINE_ID, - CONTROLMIXER_OPS(volsw, 256 binds the mixer control to volume get/put handlers, 256, 256), + CONTROLMIXER_OPS(volsw, 259 binds the mixer control to switch get/put handlers, 259, 259), CONTROLMIXER_MAX(max 1 indicates switch type control, 1), false, , -- 2.17.1

1 2

[Sound-open-firmware] [PATCH v2 0/6] Tone pipeline related patches
by Ranjani Sridharan 24 Jun '18

24 Jun '18

This patches includes some changes to enable the hostless tone pipeline. The main changes are: 1. skip host page table for hostless pipelines 2. read tone sample_rate from topology 3. Fix period bytes calculation 4. change tone default frequency/gain to adhere to AES17 requirements Ranjani Sridharan (6): ipc: skip host page table set up for hostless tone pipeline tone: add get command handler to return tone state ipc: add sample_rate member to ipc tone comp def tone: set sample rate from topology tone: fix calculation for period_bytes tone: fix default frequency and gain to match AES17 requirement src/audio/tone.c | 43 +++++++++++++++++++----- src/include/uapi/ipc.h | 1 + src/ipc/handler.c | 76 +++++++++++++++++++++++++++++++++++++++++- 3 files changed, 110 insertions(+), 10 deletions(-) -- 2.17.1

1 6

[Sound-open-firmware] Audio playback error with commit 7c281d3 on up squared
by Ranjani Sridharan 22 Jun '18

22 Jun '18

Hi All, Audio playback seems to be broken on my up squared due to the this commit: 7c281d3d894ce8e9b9ea5a6a314c5c9d8e631d3e. I get "unable to install hw_params" error when I start aplay. Please find attached the dmesg log below. Thanks, Ranjani [ 30.124183] sof-audio sof-audio: pfn i 15 idx 37 pfn 17dc7f [ 30.124203] sof-audio sof-audio: period_bytes:0x3f00 [ 30.124210] sof-audio sof-audio: stream_tag 1 [ 30.432301] sof-audio sof-audio: error: ipc timed out for 0x60010000 size 0x60 [ 30.432755] sof-audio sof-audio: status: fw entered - code 00000005 [ 30.433101] sof-audio sof-audio: error: runtime exception [ 30.433386] sof-audio sof-audio: error: trace point 00004000 [ 30.433684] sof-audio sof-audio: error: DSP Firmware Oops [ 30.433970] sof-audio sof-audio: error: Exception Cause: LoadStorePIFDataErrorCause, Synchronous PIF data error during LoadStore access [ 30.434600] sof-audio sof-audio: EXCCAUSE 0x0000000d EXCVADDR 0x00000040 PS 0x00070025 SAR 0x00000010 [ 30.435123] sof-audio sof-audio: EPC1 0xbe01afc0 EPC2 0xbe01afbd EPC3 0xbe010b5e EPC4 0x00000000 [ 30.435645] sof-audio sof-audio: EPC5 0xbe01afbd EPC6 0x00000000 EPC7 0x00000000 DEPC 0x00000000 [ 30.436168] sof-audio sof-audio: EPS2 0x00060120 EPS3 0x00060520 EPS4 0x00000000 EPS5 0x00060120 [ 30.436691] sof-audio sof-audio: EPS6 0x00000000 EPS7 0x00000000 INTENABL 0x000101d8 INTERRU 0x00000222 [ 30.437214] sof-audio sof-audio: stack dump from 0xbe07ff40 [ 30.437509] sof-audio sof-audio: 0xbe07ff40: 0xbe01afc0 0x00000000 0x00000010 0x00000000 [ 30.437951] sof-audio sof-audio: 0xbe07ff44: 0xbe044024 0x00000000 0x00000008 0x00000000 [ 30.438372] sof-audio sof-audio: 0xbe07ff48: 0x0000000d 0x00000000 0xbe020ae9 0xbe020aef [ 30.438794] sof-audio sof-audio: 0xbe07ff4c: 0xbe010b5b 0x00000000 0xbe046aa4 0x00000000 [ 30.439216] sof-audio sof-audio: 0xbe07ff50: 0x00000001 0x00000000 0x00000002 0x00000000 [ 30.439638] sof-audio sof-audio: 0xbe07ff54: 0x7e0104c6 0x00000000 0x00004000 0x00000000 [ 30.440059] sof-audio sof-audio: 0xbe07ff58: 0x00000008 0x00000000 0xbe041d64 0xbe051800 [ 30.440482] sof-audio sof-audio: 0xbe07ff5c: 0x00000000 0x00000000 0x01000000 0x00000000 [ 30.440903] sof-audio sof-audio: error: waking up any trace sleepers [ 30.441289] sof-audio sof-audio: error: got wrong posn offset 0xffffffff for PCM -110 [ 30.441714] sof-audio sof-audio: ASoC: sof-audio hw params failed: -110 [ 30.442367] sof-audio sof-audio: pcm: free stream 0 dir 0 [ 30.752315] sof-audio sof-audio: error: ipc timed out for 0x60030000 size 0xc [ 30.752752] sof-audio sof-audio: status: fw entered - code 00000005 [ 30.753098] sof-audio sof-audio: error: runtime exception [ 30.753384] sof-audio sof-audio: error: trace point 00004000 [ 30.753683] sof-audio sof-audio: error: DSP Firmware Oops [ 30.753968] sof-audio sof-audio: error: Exception Cause: LoadStorePIFDataErrorCause, Synchronous PIF data error during LoadStore access [ 30.754617] sof-audio sof-audio: EXCCAUSE 0x0000000d EXCVADDR 0x00000040 PS 0x00070025 SAR 0x00000010 [ 30.755141] sof-audio sof-audio: EPC1 0xbe01afc0 EPC2 0xbe01afbd EPC3 0xbe010b5e EPC4 0x00000000 [ 30.755664] sof-audio sof-audio: EPC5 0xbe01afbd EPC6 0x00000000 EPC7 0x00000000 DEPC 0x00000000 [ 30.756187] sof-audio sof-audio: EPS2 0x00060120 EPS3 0x00060520 EPS4 0x00000000 EPS5 0x00060120 [ 30.756711] sof-audio sof-audio: EPS6 0x00000000 EPS7 0x00000000 INTENABL 0x000101d8 INTERRU 0x00000222 [ 30.757234] sof-audio sof-audio: stack dump from 0xbe07ff40 [ 30.757529] sof-audio sof-audio: 0xbe07ff40: 0xbe01afc0 0x00000000 0x00000010 0x00000000 [ 30.757951] sof-audio sof-audio: 0xbe07ff44: 0xbe044024 0x00000000 0x00000008 0x00000000 [ 30.758373] sof-audio sof-audio: 0xbe07ff48: 0x0000000d 0x00000000 0xbe020ae9 0xbe020aef [ 30.758795] sof-audio sof-audio: 0xbe07ff4c: 0xbe010b5b 0x00000000 0xbe046aa4 0x00000000 [ 30.759217] sof-audio sof-audio: 0xbe07ff50: 0x00000001 0x00000000 0x00000002 0x00000000 [ 30.759639] sof-audio sof-audio: 0xbe07ff54: 0x7e0104c6 0x00000000 0x00004000 0x00000000 [ 30.760061] sof-audio sof-audio: 0xbe07ff58: 0x00000008 0x00000000 0xbe041d64 0xbe051800 [ 30.760484] sof-audio sof-audio: 0xbe07ff5c: 0x00000000 0x00000000 0x01000000 0x00000000 [ 30.760906] sof-audio sof-audio: error: waking up any trace sleepers [ 30.761550] sof-audio sof-audio: pcm: close stream 0 dir 0

5 5

[Sound-open-firmware] [PATCH] scripts: xtensa-build: fix ROOT var alais for BXT/APL gcc
by Xiuli Pan 22 Jun '18

22 Jun '18

From: Pan Xiuli <xiuli.pan(a)linux.intel.com> Need to change ROOT as well as HOST target. Signed-off-by: Pan Xiuli <xiuli.pan(a)linux.intel.com> --- scripts/xtensa-build-all.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/scripts/xtensa-build-all.sh b/scripts/xtensa-build-all.sh index 9a0f3e4..abf4e87 100755 --- a/scripts/xtensa-build-all.sh +++ b/scripts/xtensa-build-all.sh @@ -95,7 +95,6 @@ do then PLATFORM="apollolake" XTENSA_CORE="X4H3I16w2D48w3a_2017_8" - ROOT="$pwd/../xtensa-root/xtensa-bxt-elf" # test APL compiler aliases and ignore set -e here type xtensa-bxt-elf-gcc > /dev/null 2>&1 && true @@ -105,6 +104,8 @@ do else HOST="xtensa-apl-elf" fi + + ROOT="$pwd/../xtensa-root/$HOST" XTENSA_TOOLS_VERSION="RG-2017.8-linux" fi if [ $j == "cnl" ] -- 2.7.4

2 1

[Sound-open-firmware] [PATCH 0/3] Add MCLK ID to SSP config
by Ranjani Sridharan 21 Jun '18

21 Jun '18

Some platforms expose more than one MCLK's. Add a MCLK_ID token to use in such cases. Please note that there arent any examples of MCLK ID other than 0 in any of the test topologies yet. But future platforms might have the need for it. Ranjani Sridharan (3): topology: add new token for SSP MCLK ID topology: modify ssp config definition to add the tuple for mclk id topology: replace SSP_SAMPLE_BITS() users with SSP_CONFIG_DATA() topology/m4/dai.m4 | 23 ++++++++++++++++++++--- topology/sof-apl-nocodec.m4 | 2 +- topology/sof-apl-tdf8532.m4 | 12 ++++++------ topology/sof-bdw-rt286.m4 | 2 +- topology/sof-bdw-rt5640.m4 | 2 +- topology/sof-byt-da7213.m4 | 2 +- topology/sof-byt-nocodec.m4 | 2 +- topology/sof-byt-rt5640.m4 | 2 +- topology/sof-byt-rt5645.m4 | 2 +- topology/sof-byt-rt5651.m4 | 2 +- topology/sof-cht-max98090.m4 | 2 +- topology/sof-cht-nocodec.m4 | 2 +- topology/sof-cnl-rt274.m4 | 2 +- topology/sof-hsw-rt5640.m4 | 2 +- topology/sof/tokens.m4 | 1 + topology/test/test-all.m4 | 2 +- topology/test/test-capture.m4 | 2 +- topology/test/test-playback.m4 | 2 +- topology/test/test-tone-playback.m4 | 2 +- 19 files changed, 43 insertions(+), 25 deletions(-) -- 2.17.1

3 6