Re: Commit 8f7206d69a ("ASoC: imx-wm8962: Remove global variables") looks strange

[Daniel Baluta <daniel.baluta@nxp.com>]

On Lu, 2018-02-26 at 11:34 -0300, Fabio Estevam wrote:
> Hi Daniel,
> 
> On Mon, Feb 26, 2018 at 11:21 AM, Daniel Baluta <daniel.baluta@nxp.com> wrote:
> > 
> > Hi Fabio,
> > 
> > Leonard (Cc'ed) noticed that commit in $subject looks very strange.
> > 
> > For example:
> > 
> > @@ -139,7 +140,7 @@ static int imx_wm8962_late_probe(struct snd_soc_card *card)
> >  {
> >         struct snd_soc_pcm_runtime *rtd;
> >         struct snd_soc_dai *codec_dai;
> > -       struct imx_priv *priv = &card_priv;
> > +       struct imx_priv *priv = snd_soc_card_get_drvdata(card);
> >         struct imx_wm8962_data *data = snd_soc_card_get_drvdata(card);
> > 
> > So, now data and priv point to the same memory area but they are completely
> > different structures!
> > 
> > Unless we are missing something this is obviously a bug. I will try to have a look at
> > the end of the week, but any comments are welcome.
> > 
> > I think the fix would be to have one global priv structure and to unify imx_wm8962_data
> > with imx_priv.
> Could you please look in mainline? The structs were re arranged there
> and I do not see this problem.
> 
> Please confirm.

Hmm,

Looking into latest:

git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git

for-next branch and the commit in subject is the last on this file.

Is there any other more updated tree/branch? Can you point the commit hash that did the re-arrangement.

Daniel.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel