[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20191119141046.GC30789@kadam>
Date: Tue, 19 Nov 2019 17:10:46 +0300
From: Dan Carpenter <dan.carpenter@...cle.com>
To: vishnu <vravulap@....com>
Cc: Ravulapati Vishnu vardhan rao
<Vishnuvardhanrao.Ravulapati@....com>, Alexander.Deucher@....com,
djkurtz@...gle.com, Akshu.Agrawal@....com,
Liam Girdwood <lgirdwood@...il.com>,
Mark Brown <broonie@...nel.org>,
Jaroslav Kysela <perex@...ex.cz>,
Takashi Iwai <tiwai@...e.com>,
Vijendar Mukunda <Vijendar.Mukunda@....com>,
YueHaibing <yuehaibing@...wei.com>,
"Gustavo A. R. Silva" <gustavo@...eddedor.com>,
Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>,
"moderated list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM..."
<alsa-devel@...a-project.org>,
open list <linux-kernel@...r.kernel.org>
Subject: Re: [RESEND PATCH v9 6/6] ASoC: amd: Added ACP3x system resume and
runtime pm
On Tue, Nov 19, 2019 at 07:33:08PM +0530, vishnu wrote:
> > > > static void snd_acp3x_remove(struct pci_dev *pci)
> > > > {
> > > > - struct acp3x_dev_data *adata = pci_get_drvdata(pci);
> > >
> > > This was fine. Leave it as-is.
> > >
>
> Actually I was reported by kbuild robot tool about ISO mixed forbids of
> initialization so I did this.
>
You misunderstood the warning. It is about putting declarations after
the start of code. (Initializers don't count as code in this context).
regards,
dan carpenter
Powered by blists - more mailing lists