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...
