Hi Kuninori,
On 2019-06-25 10:25, Kuninori Morimoto wrote:
>>> I have no idea so far, but one capability is that if your system
>>> used / based dummy platform, and if your CPU driver didn't use/have
>>> snd_soc_set_runtime_hwparams() / snd_pcm_lib_ioctl,
>>> it might be happen, but this patch seems not related to it...
>> Reverting this patch fixes the issue, so it is definitely related somehow.
> Ahh... OK now I think I could understand your issue.
> Your CPU is using devm_snd_dmaengine_pcm_register() with same *dev.
> In such case, indeed this patch breaks sound card.
>
> Can I confirm it ?
> In local email, I asked printing component name list.
> Can you please replace it to below ?
> I think disappear component is "snd_dmaengine_pcm".
> If so, above my patch is the bug.
>
> ---------------------------
> diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c
> index 4cd77cd..4b1f8ae 100644
> --- a/sound/soc/soc-core.c
> +++ b/sound/soc/soc-core.c
> @@ -313,6 +313,7 @@ static int snd_soc_rtdcom_add(struct snd_soc_pcm_runtime *rtd,
> new_rtdcom->component = component;
> INIT_LIST_HEAD(&new_rtdcom->list);
>
> + printk("-----%s : %s\n", component->name, component->driver->name);
> list_add_tail(&new_rtdcom->list, &rtd->component_list);
>
> return 0;
> ---------------------------
>
> Thank you for your help !!
pure next-20190624:
[ 3.549712] -----3830000.i2s : samsung-i2s
[ 3.767732] -----max98095.7-0011 : (null)
[ 3.986370] -----hdmi-audio-codec.3.auto : (null)
next-20190624 + Revert "ASoC: samsung: snow: don't select unnecessary
Platform":
[ 3.508391] -----3830000.i2s : samsung-i2s
[ 3.726332] -----max98095.7-0011 : (null)
[ 3.944906] -----hdmi-audio-codec.3.auto : (null)
[ 4.376639] -----3830000.i2s : snd_dmaengine_pcm
Best regards
--
Marek Szyprowski, PhD
Samsung R&D Institute Poland
