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

[Sound-open-firmware] [RFC PATCH 0/6] Cleanups for SSP slave support on APL/CNL
by Pierre-Louis Bossart 18 Jul '18

18 Jul '18

While looking at the settings needed for the SSP slave support, I came across severial confusions or errors. I still can't get this mode to work but I could use feedback on the following patches, and also double-check if there is any impact in master mode. Comments/test welcome. Pierre-Louis Bossart (6): apl-ssp: fix SSIOC_SCOE apl-ssp: fix SSIOC_SFCR apl-ssp: fix SCR1_SCFR apl-ssp: move SSCR0_ACS,MOD as default settings apl-ssp: fix SSCR0_ECS apl-ssp: fix SSCR1_SFRMDIR src/drivers/apl-ssp.c | 26 +++++++++----------------- 1 file changed, 9 insertions(+), 17 deletions(-) -- 2.14.1

1 6

[Sound-open-firmware] code stabilization
by Pierre-Louis Bossart 13 Jul '18

13 Jul '18

Hi, Since there's been a lot of churn and issues this week, I thought I would share what works for me in a simple setup (Up^2 board with Hifiberry DAC+/PCM512x). The commits below seem to work decently, without the usual topology load failures, IPC timeouts or corrupted sound that were reported by others. It would be nice if others could try and reproduce those results and try on different APL/CNL boards with the xt-xcc compiler. Don't try on Baytrail/Cherrytrail/Broadwell for now until the GCC compilations issues are fixed. I would recommend we stop merging new features or fixes for corner cases to reach our goal of a stable release in the coming days, or push them to -next while we deal with those annoying first-order issues. Thanks -Pierre Kernel (2 branches maintained) topic/sof-v4.14 commit cae4773bbd83 ('ASoC: SOF: apply IPC changes to byt') topic/sof-dev (4.17) commit a3a39f21218c ('Merge pull request #28 from RanderWang/ipx_fix') SOF master commit 295d0c131c79 ('Merge pull request #81 from tlauda/topic/volume_processing') SOFT master commit 2abbfc0910c ('Merge pull request #28 from plbossart/debug/pcm512x')

1 0

[Sound-open-firmware] [RFC PATCH 0/2] GLK/APL topology edits
by Pierre-Louis Bossart 12 Jul '18

12 Jul '18

Posting this to the mailing list so that its archived and searchable. Basically I am trying to modify the GLK topology to support the ApolloLake Chromebooks. It's only a matter of swapping two SSPs but the changes in the topology files are far from trivial. Feedback welcome. Pierre-Louis Bossart (2): topology: simplify sof-glk-codec.m4 topology: add sof-apl-da7219 support topology/Makefile.am | 1 + topology/sof-apl-da7219.m4 | 104 +++++++++++++++++++++++++++++++++++++++++++++ topology/sof-glk-codec.m4 | 30 ++++++------- 3 files changed, 120 insertions(+), 15 deletions(-) create mode 100644 topology/sof-apl-da7219.m4 -- 2.14.1

3 10

Re: [Sound-open-firmware] .m4 File Request Help Needed
by Ranjani Sridharan 05 Jul '18

05 Jul '18

On Wed, 2018-07-04 at 06:22 +0000, Nahulanthran, Sanjeev wrote: > Hi there, > > Managed to build the topology file that I needed. Thanks for your > help! Enabled some extra kernel logs to confirm that the tplg file > being used matches the one I have created. > > However, still obtaining this issue (dmesg logs attached) - > sof-nocodec sof-nocodec: ASoC: CPU DAI sof-audio not registered > sof-audio sof-audio: error: ipc error for 0x80010000 size 0xc > sof-audio sof-audio: error: failed to set DAI config for SSP2 > sof-audio sof-audio: error: DSP trace buffer overflow 16 bytes. Total > messages 699 Hi Sanjeev, Copying the list too. This looks like you might be missing some kernel patches. Could you please share the tree you're using for the kernel, firmware and topology? Thanks, Ranjani > > Right now using tplg base file "reef-apl-nocodec.tplg" (modified to > suit my needs, m4 also attached to email) from 1.1-stable branch of > SOFT and using firmware file "reef-apl.ri" from master branch of SOF. > > Can load the soundcards - shown using "aplay -l", however when > playback is initiated, ipc error is shown as stated above. What could > be the reason for this error? Tracing error code leads to function > tx_wait_done in driver code ipc.c (data received from DSP is invalid > I think). How can I resolve this? > > Hope you can extend some help regarding this. Thank you very much for > your time :) > > Thanks and Regards, > Sanjeev > > -----Original Message----- > From: Ranjani Sridharan [mailto:ranjani.sridharan@linux.intel.com] > Sent: Wednesday, July 4, 2018 5:31 AM > To: Nahulanthran, Sanjeev <sanjeev.nahulanthran(a)intel.com> > Subject: Re: [Sound-open-firmware] .m4 File Request Help Needed > > On Tue, 2018-07-03 at 08:01 +0000, Nahulanthran, Sanjeev wrote: > > Hi there, > > > > Somehow I am unable to bring up my soundcard except with this tplg > > file: > > test-ssp*-DSP_A-volume-s32le-s24le-48k-nocodec.tplg > > > > Would you happen to have the .m4 reference for this topology file > > so > > that I can modify it for my own needs and build? I understand how > > to > > configure the DAI section now :) thanks btw > > > > Somehow trying to configure the reference .m4 given to me was not > > successful. I need the .m4 to build this kind of tplg file (test- > > ssp*-DSP_A-volume-s32le-s24le-48k-nocodec.tplg), from there I can > > make > > my modifications as necessary. > > The easiest way is to just add a line to tplg-build.sh script like > this: > > simple_test nocodec volume "NoCodec-2" s24le SSP 2 s16le 25 16 > 2400000 > 24000000 I2S 0 SIMPLE_TESTS[@] > > The above line would create the tplg file you are looking for. Please > remember to modify the SSP port, container bits, valid bits, bclk and > mclk to suit your needs. > > Hope this helps. > > > > Thanks and Regards, > > Sanjeev > > > > -----Original Message----- > > From: Ranjani Sridharan [mailto:ranjani.sridharan@linux.intel.com] > > Sent: Sunday, July 1, 2018 4:55 AM > > To: Nahulanthran, Sanjeev <sanjeev.nahulanthran(a)intel.com>; sound- > > ope > > n-firmware(a)alsa-project.org > > Subject: Re: [Sound-open-firmware] .m4 File Request Help Needed > > > > On Thu, 2018-06-28 at 05:58 +0000, Nahulanthran, Sanjeev wrote: > > > Hi there, > > > > > > Writing this e-mail to request for SOF support in providing a > > > .m4 > > > file with 48kHz, TDM8, and MCLK enabled - used for Cirrus Logic > > > CS42435. > > > This is required as we need it as a reference to make our own > > > modifications to enable the same configuration for another device > > > (amplifier) that we have. > > > > Hi Sanjeev, > > > > Please take a look at the example m4 files provided in the > > topology/test folder in the SOF tools repo: > > https://github.com/thesofproject/soft > > > > For example to test playback, you can use test-playback.m4 and > > modify > > the DAI config to suit your needs. DAI_CONFIG is defined in the > > dai.m4 file and an example config is: > > > > DAI_CONFIG(TEST_DAI_TYPE, TEST_DAI_PORT, 0, TEST_DAI_LINK_NAME, > > SSP_CONFIG(TEST_SSP_MODE, > > SSP_CLOCK(mclk, TEST_SSP_MCLK, codec_slave), > > SSP_CLOCK(bclk, TEST_SSP_BCLK, codec_slave), > > SSP_CLOCK(fsync, 48000, codec_slave), > > SSP_TDM(2, TEST_SSP_PHY_BITS, 3, 3), > > SSP_CONFIG_DATA(TEST_DAI_TYPE, TEST_DAI_PORT, > > TEST_SSP_DATA_BITS))) > > > > As you can see, all the config parameters are arguments to the > > macro > > and quite self-explanatory. > > > > If you have more questions, please feel free to contact us. > > > > Thanks, > > Ranjani > > > > > > > > Thanks and Regards, > > > Sanjeev > > > Software Engineer > > > > > > > > > _______________________________________________ > > > Sound-open-firmware mailing list > > > Sound-open-firmware(a)alsa-project.org > > > http://mailman.alsa-project.org/mailman/listinfo/sound-open-firmw > > > ar > > > e

3 2

[Sound-open-firmware] .m4 File Request Help Needed
by Nahulanthran, Sanjeev 30 Jun '18

30 Jun '18

Hi there, Writing this e-mail to request for SOF support in providing a .m4 file with 48kHz, TDM8, and MCLK enabled - used for Cirrus Logic CS42435. This is required as we need it as a reference to make our own modifications to enable the same configuration for another device (amplifier) that we have. Thanks and Regards, Sanjeev Software Engineer

3 2

[Sound-open-firmware] Loading external library
by Daniel Baluta 27 Jun '18

27 Jun '18

Hi, Does SOF firmware/driver supports loading a library and executing code from it? e.g Loading binary for a codec and use it to decode a file. thanks, Daniel.

3 8

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