[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20171205193912.GB11039@Asurada-Nvidia>
Date: Tue, 5 Dec 2017 11:39:13 -0800
From: Nicolin Chen <nicoleotsuka@...il.com>
To: Mark Brown <broonie@...nel.org>
Cc: linux-kernel@...r.kernel.org, linuxppc-dev@...ts.ozlabs.org,
alsa-devel@...a-project.org, fabio.estevam@....com, timur@...i.org,
mail@...iej.szmigiero.name, caleb@...me.org, lgirdwood@...il.com
Subject: Re: [PATCH 01/10] ASoC: fsl_ssi: Remove unused struct device
On Tue, Dec 05, 2017 at 01:04:46PM +0000, Mark Brown wrote:
> On Mon, Dec 04, 2017 at 12:46:34PM -0800, Nicolin Chen wrote:
> > struct device is defined in system level header files any way.
> > As long as fsl_ssi.h is included after those header files, it
> > should be safe to remove this line.
>
> It's better practice to either directly include those headers or have a
> forward declaration like this, it means there's no way things can go
> wrong.
I see. I will drop this patch then.
Powered by blists - more mailing lists