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

[Marek Szyprowski]

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