Hi Ranjani Thank you for reviewing > > From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> > > > > We have 2 type of component functions > > snd_soc_component_xxx() is focusing to component itself, > > snd_soc_pcm_component_xxx() is focusing to rtd related component. > > > > Now we can update snd_soc_component_prepare() to > > snd_soc_pcm_component_prepare(). This patch do it. > > > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> > > --- (snip) > > - for_each_rtd_components(rtd, i, component) { > > - ret = snd_soc_component_prepare(component, substream); > > - if (ret < 0) { > > - dev_err(component->dev, > > - "ASoC: platform prepare error: %d\n", > > ret); > > - goto out; > > - } > > + ret = snd_soc_pcm_component_prepare(substream); > > + if (ret < 0) { > > + dev_err(rtd->dev, > > + "ASoC: platform prepare error: %d\n", ret); > Morimoto-san, > We should remove this. This will be a duplicate error message as > snd_soc_pcm_component_prepare() would already print the error before > returning. Ohh, yes. will fix in v2 Thank you for your help !! Best regards --- Kuninori Morimoto