linux-kernel - Re: [PATCH -next] regulator: aat2870-regulator.c needs module.h

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20110803102852.GD26275@sortiz-mobl>
Date:	Wed, 3 Aug 2011 12:28:52 +0200
From:	Samuel Ortiz <sameo@...ux.intel.com>
To:	Liam Girdwood <lrg@...com>
Cc:	Stephen Rothwell <sfr@...b.auug.org.au>,
	Mark Brown <broonie@...nsource.wolfsonmicro.com>,
	Randy Dunlap <rdunlap@...otime.net>,
	akpm <akpm@...ux-foundation.org>,
	"linux-next@...r.kernel.org" <linux-next@...r.kernel.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH -next] regulator: aat2870-regulator.c needs module.h

Hi Liam,

On Mon, Aug 01, 2011 at 07:09:46PM +0100, Liam Girdwood wrote:
> Hi Samuel,
> 
> On Mon, 2011-08-01 at 11:25 +0200, Samuel Ortiz wrote:
> > Hi Stephen, Liam,
> > 
> > On Mon, Aug 01, 2011 at 11:05:07AM +1000, Stephen Rothwell wrote:
> > > On Mon, 1 Aug 2011 10:33:04 +1000 Stephen Rothwell <sfr@...b.auug.org.au> wrote:
> > > >
> > > > On Mon, 1 Aug 2011 01:15:24 +0100 Mark Brown <broonie@...nsource.wolfsonmicro.com> wrote:
> > > > >
> > > > > On Fri, Jul 29, 2011 at 09:13:09PM -0700, Randy Dunlap wrote:
> > > > > > From: Randy Dunlap <rdunlap@...otime.net>
> > > > > > 
> > > > > > aat2870-regulator.c needs to include linux/module.h to fix multiple
> > > > > > build errors.
> > > > > 
> > > > > Acked-by: Mark Brown <broonie@...nsource.wolfsonmicro.com>
> > > > > 
> > > > > but if the breakage isn't in mainline yet to minimize the breaks
> > > > > bisection someone should figure out which tree introduced the change
> > > > > with module.h and either get the fix merged along with that.  If neither
> > > > > the module.h change nor the driver is in Linus' tree yet I rather
> > > > > suspect this should go via MFD as I seem to recall that the driver is
> > > > > going in via there.
> > > > 
> > > > It is indeed.  I have this patch set to apply to my tree today after the
> > > > mfd tree.  Hopefully Samuel will put it in his tree for tomorrow (and
> > > > before he asks Linus to pull.
> > > 
> > > Too late :-)
> > Sorry about that. I thought I would hear about any breakage after I pushed my
> > bits on Friday morning.
> > Anyway, Liam, are you going to push this fix through your tree ?
> > 
> 
> I can if required, however I've just done my pull request (sorry didn't
> see this in time). I have to do another pull request with misc fixes in
> rc1 time frame so I can add it there if OK. 
That's fine with me, thanks in advance.

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/