[alsa-devel] [PATCH resend 25/47] ASoC: samsung: snow: don't select unnecessary Platform

[Kuninori Morimoto]

Hi Marek

Thank you for your test.
And so sorry to bother you.
I will fixup this issue.

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