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

Re: [Sound-open-firmware] [PATCH 2/3] ASoC: add function parameters to enable forced path pruning
by Guennadi Liakhovetski 11 Mar '20

11 Mar '20

Hi Mark, On Wed, Mar 11, 2020 at 12:25:31PM +0000, Mark Brown wrote: > On Wed, Mar 11, 2020 at 08:41:27AM +0100, Guennadi Liakhovetski wrote: > > On Tue, Mar 10, 2020 at 12:45:44PM +0000, Mark Brown wrote: > > > > So why doesn't DPCM recognize that the path is inactive and why is it > > > better to do this than fix whatever the issue is there? > > > Of course that would be better abd I'd much prefer that. Unfortunately I > > haven't been able to find a single scenario in which those paths would be > > exercised. As far as I understand path pruning should take place e.g. > > when a mixer modifies audio routing and as a result disables a certain > > pipeline, which is then pruned. If I could reproduce such a scenario I > > would be able to first check whether it's working, then see exactly how > > it is working and then see how best to add my use case to it. Since I > > wasn't able to find such a scenario, my only option was to preserve > > the current state and add my own path "on top." I'd be happy to try the > > other path too, I just need a use case, that I can reproduce. > > It's still not clear to me what the issue is here. If something is > making a modification to the graph which needs a recheck or update I'd > expect that these things happen along with that modification. I don't > understand what you're saying about not being able to reproduce > scenarios or adding things "on top". I mean, that I don't have a test-case to test dpcm_prune_paths() without my VirtIO support. That function currently exists in the kernel, but I don't have a test-case to verify its work, to see it called and actually perform the pruning. So I don't know how it is supposed to work. And because of that I cannot fix my VirtIO use case to use that function properly, without forcing it with an additional parameter. Thanks Guennadi

1 0

Re: [Sound-open-firmware] [PATCH 2/3] ASoC: add function parameters to enable forced path pruning
by Guennadi Liakhovetski 11 Mar '20

11 Mar '20

Hi Mark, On Tue, Mar 10, 2020 at 12:45:44PM +0000, Mark Brown wrote: > On Mon, Mar 09, 2020 at 06:07:48PM +0100, Guennadi Liakhovetski wrote: > > > soc_dpcm_runtime_update() to activate an audio path and update audio > > routing. When streaming is stopped, the vhost driver calls > > soc_dpcm_runtime_update() and snd_soc_runtime_deactivate(). The latter > > doesn't work at the moment, because the DPCM doesn't recognise the > > path as inactive. We address this by adding a "mode" parameter to > > soc_dpcm_runtime_update(). If virtualisation isn't used, the current > > behaviour isn't affected. > > So why doesn't DPCM recognize that the path is inactive and why is it > better to do this than fix whatever the issue is there? Of course that would be better abd I'd much prefer that. Unfortunately I haven't been able to find a single scenario in which those paths would be exercised. As far as I understand path pruning should take place e.g. when a mixer modifies audio routing and as a result disables a certain pipeline, which is then pruned. If I could reproduce such a scenario I would be able to first check whether it's working, then see exactly how it is working and then see how best to add my use case to it. Since I wasn't able to find such a scenario, my only option was to preserve the current state and add my own path "on top." I'd be happy to try the other path too, I just need a use case, that I can reproduce. Thanks Guennadi

1 0

Re: [Sound-open-firmware] [PATCH 0/3] [RESEND] ASoC: SOF: preparatory patches
by Pierre-Louis Bossart 10 Mar '20

10 Mar '20

On 3/10/20 7:50 AM, Mark Brown wrote: > On Mon, Mar 09, 2020 at 05:05:06PM -0500, Pierre-Louis Bossart wrote: > >> To get a better picture of the directions, reviewers are invited to take a >> look at the in-depth documentation written by Guennadi since the initial >> patches were shared. This documentation was reviewed by Liam and me and is >> really required to understand the concepts: > >> https://thesofproject.github.io/latest/developer_guides/virtualization/virt… > > How does this relate to the virtio audio spec that's currently under > review? It looks to be doing something much lower level than that. Mark, to avoid any ambiguity, what 'virtio audio spec that's currently under review' are you referring to? > I am concerned that this looks to be exposing DPCM as a virtio ABI, > we're trying to replace it as an internal API never mind ABI. Valid point indeed.

1 0

[Sound-open-firmware] [PATCH 0/3] [RESEND] ASoC: SOF: preparatory patches
by Guennadi Liakhovetski 10 Mar '20

10 Mar '20

This is the first set of patches for the SOF virtualisation work. We send these patches first because they touch the ASoC core. 2 of them are mostly cosmetic with no functional changes, but patch 2/3 might cause some discussions. Please review and comment. Thanks Guennadi Guennadi Liakhovetski (3): ASoC: (cosmetic) simplify dpcm_prune_paths() ASoC: add function parameters to enable forced path pruning ASoC: export DPCM runtime update functions include/sound/soc-dpcm.h | 30 ++++++++--- sound/soc/soc-compress.c | 2 +- sound/soc/soc-dapm.c | 8 +-- sound/soc/soc-pcm.c | 130 +++++++++++++++++++++++++++++------------------ 4 files changed, 109 insertions(+), 61 deletions(-) -- 1.9.3

2 7

Re: [Sound-open-firmware] Getting error while building firmware binaries (I am building SOF from scratch)
by Liam Girdwood 03 Mar '20

03 Mar '20

Hi, On Mon, 2020-03-02 at 21:54 +0530, Akriti Verma wrote: > > Hi, > I am stuck at this since a long time an cannot come to a possible > solution. > I was trying to build firmware binaries and i ha cloned sof.git by > using the command :- > git clone https://github.com/thesofproject/sof.git > > While running the './scripts/xtensa-build-all.sh -a' command, i get > certain > makefile errors as can be seen in the attached screenshot. > Have you built the compilers and installed it in your $PATH ? Can you confirm you have followed the instructions and are using a recent Linux distro. https://thesofproject.github.io/latest/getting_started/build-guide/build-fr… Thanks Liam > > I am unable to find a workaround for this problem and move further > with the building > > Thanks and regards > Akriti

1 0

[Sound-open-firmware] [Re]: Make error when building topology and tools
by Mohana Datta Yelugoti 02 Mar '20

02 Mar '20

Hi all, I am trying to building SOF from scratch by following through the official documentation guide [0]. I am getting errors when trying to build Topology and tools [1]. I am providing pastebin links to topology_log [2] and topology_error_log [3]. topology_log contains output to stdout and topology_error_log contains the output to stderr. I was able to build in docker successfully before. But, when i am trying to build from scratch, i encountered this error. Can someone guide me in solving this error. The system i am building runs on Ubuntu 19.19 eoan. My kernel version is 5.3.0-40-generic. Please feel free to ask for any other information, if i am missing any. Thank you for your time. Regards, ymdatta. References: ---------- [0]: https://thesofproject.github.io/latest/getting_started/build-guide/build-fr… [1]: https://thesofproject.github.io/latest/getting_started/build-guide/build-fr… [2]: https://paste.gnome.org/pffxkrwjm/cvhcko [3]: https://paste.gnome.org/pg9mefnet/crqsix

3 5