Re: [PATCH v4 1/4] ASoC: makes CPU/Codec channel connection map more generic

18 Oct 2023


      Hi Pierre-Louis
Thank you for your test.
...
...
/*
 * construct cpu channel mask by combining ch_mask of each
 * codec which maps to the cpu.


 * see


 *	soc.h :: [dai_link->ch_maps Image sample]

*/


for_each_rtd_codec_dais(rtd, j, codec_dai) {


	if (rtd->dai_link->codec_ch_maps[j].connected_cpu_id == i)


		ch_mask |= rtd->dai_link->codec_ch_maps[j].ch_mask;


if (rtd->dai_link->num_cpus >= rtd->dai_link->num_codecs) {


	/* .ch_map is from CPU */


	ch_mask = rtd->dai_link->ch_maps[i].ch_mask;


... and for a FE dailink there's no ch_maps so this results in a kernel
oops.
Hmm... this is strange...
New snd_soc_compensate_connection_map() will add default ch_maps for all
dai_link...
Oh, is it using topology or something which doesn't call
snd_soc_bind_card() ? If so could you please try to call
snd_soc_compensate_connection_map() ?
(I guess it is using soc_tplg_fe_link_create() ?)
If it could solve your issue, v5 will handle it.
Thank you for your help !!
Best regards
---
Kuninori Morimoto