Skip to content

Conversation

@kv2019i
Copy link
Collaborator

@kv2019i kv2019i commented Jan 26, 2023

Multiple pipelines had incorrect num_audio_formats leading to invalid format structures ending up to topology.

Example FW trace:

[ 1601.624195] sof-audio-pci-intel-tgl 0000:00:1f.3: Get input audio formats for copier.module.8.2 [ 1601.624197] sof-audio-pci-intel-tgl 0000:00:1f.3:  #0: 48000KHz, 32bit (ch_map 0xffffff10 ch_cfg 1 interleaving_style 0 fmt_cfg 0x2002) [ 1601.624201] sof-audio-pci-intel-tgl 0000:00:1f.3:  #1: 0KHz, 0bit (ch_map 0x0 ch_cfg 0 interleaving_style 0 fmt_cfg 0x0)

Signed-off-by: Kai Vehmanen kai.vehmanen@linux.intel.com

@kv2019i
Copy link
Collaborator Author

kv2019i commented Jan 26, 2023

Wow, capture cases fail with nocodec:
https://sof-ci.01.org/sofpr/PR6994/build3540/devicetest/index.html

So apparently some piece of code is relying on invalid formats to be defined in tplg.

Multiple pipelines had incorrect num_audio_formats leading to invalid
format structures ending up to topology.

Example FW trace:

[ 1601.624195] sof-audio-pci-intel-tgl 0000:00:1f.3: Get input audio formats for copier.module.8.2
[ 1601.624197] sof-audio-pci-intel-tgl 0000:00:1f.3:  #0: 48000KHz, 32bit (ch_map 0xffffff10 ch_cfg 1 interleaving_style 0 fmt_cfg 0x2002)
[ 1601.624201] sof-audio-pci-intel-tgl 0000:00:1f.3:  thesofproject#1: 0KHz, 0bit (ch_map 0x0 ch_cfg 0 interleaving_style 0 fmt_cfg 0x0)

Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
@kv2019i kv2019i force-pushed the 202301-tplg-numaudiofmt-fix branch from a0edbb7 to f0d9670 Compare January 26, 2023 16:08
@kv2019i
Copy link
Collaborator Author

kv2019i commented Jan 26, 2023

V2:

  • fixed a bug in the patch w.r.t. gain-capture and gain-playback

@kv2019i
Copy link
Collaborator Author

kv2019i commented Jan 26, 2023

Ok, now looks better, proceeding with merge.

@kv2019i kv2019i merged commit 48afcef into thesofproject:main Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants