Re: [PATCH 44/54 v2] ASoC: switch over to use snd_soc_register_component() on sh4 siu

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Stephen

Thank you for your checking


> >  static struct snd_soc_dai_link migor_dai = {
> >  	.name = "wm8978",
> >  	.stream_name = "WM8978",
> > -	.cpu_dai_name = "siu-i2s-dai",
> > +	.cpu_dai_name = "siu-pcm-audio",
> 
> I'm not sure what that change is for. The need for it should be
> described in the commit description.
> 
> > diff --git a/sound/soc/sh/siu_dai.c b/sound/soc/sh/siu_dai.c
> 
> > @@ -783,7 +787,8 @@ static int siu_probe(struct platform_device *pdev)
> >  	dev_set_drvdata(&pdev->dev, info);
> >  
> >  	/* register using ARRAY version so we can keep dai name */
> > -	ret = snd_soc_register_dais(&pdev->dev, &siu_i2s_dai, 1);
> > +	ret = snd_soc_register_component(&pdev->dev, &siu_i2s_component,
> > +					 &siu_i2s_dai, 1);
> 
> That changes from snd_soc_register_dais() to snd_soc_register_dai().
> Will that be a problem?

I think .cpu_dai_name will be "siu-i2s-dai" if it used dais(),
and it will be "siu-pcm-audio" if it used dai()


Best regards
---
Kuninori Morimoto
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel


[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux