On Wed, Oct 26, 2016 at 04:48:29AM +0000, Kuninori Morimoto wrote: > > Hi Mark, Lars, Vinod > > > > Maybe somebody from the Intel side can look into fixing this. The affected > > > boards are cht_bsw_rt5672 and broadwell, which both access the cards > > > codec_dev_list field. > > > > This also exists in some customer SKL machines. > > > > I agree that this may not be best implementation so I can send a patch for > > this. > > > > As Lars suggested we can use DAI link init callback, but then I dont feel it > > is right to use rtd->codec to get codec pointer, again we will be looking > > into rtd internals. So would make sense to combine two suggestion and add an > > API: > > > > struct snd_soc_codec *snd_soc_get_codec(struct snd_soc_pcm_runtime *rtd) > > { > > return rtd->codec; > > } > > > > then we can use this is drivers. > > > > Let me know if all are in agreement, I can test this and send out.. > > I guess my patch series should based on Vinod's new patch > which doesn't use codec_dev_list ? > Or can I post separately ? > > Vinod, can you please Cc to me about your new patch ? Sure thing.. -- ~Vinod _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel