18 Oct
2023
18 Oct
'23
1:16 a.m.
On 10/17/23 18:03, Kuninori Morimoto wrote:
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.
Sorry, not following what the suggestion is.
Yes all our solutions are based on the topology, and I don't really understand the benefit of a ch_map for an FE? the codec_dai is a dummy one...