lkml.org 
[lkml]   [2011]   [Aug]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH -next] regulator: aat2870-regulator.c needs module.h
From
Date
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@canb.auug.org.au> wrote:
> > >
> > > On Mon, 1 Aug 2011 01:15:24 +0100 Mark Brown <broonie@opensource.wolfsonmicro.com> wrote:
> > > >
> > > > On Fri, Jul 29, 2011 at 09:13:09PM -0700, Randy Dunlap wrote:
> > > > > From: Randy Dunlap <rdunlap@xenotime.net>
> > > > >
> > > > > aat2870-regulator.c needs to include linux/module.h to fix multiple
> > > > > build errors.
> > > >
> > > > Acked-by: Mark Brown <broonie@opensource.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.

Thanks

Liam



\
 
 \ /
  Last update: 2011-08-01 20:13    [W:0.061 / U:1.304 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site