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

From: Samuel Ortiz
Date: Mon Aug 01 2011 - 05:24:11 EST


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@xxxxxxxxxxxxxxxx> wrote:
> >
> > On Mon, 1 Aug 2011 01:15:24 +0100 Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote:
> > >
> > > On Fri, Jul 29, 2011 at 09:13:09PM -0700, Randy Dunlap wrote:
> > > > From: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
> > > >
> > > > aat2870-regulator.c needs to include linux/module.h to fix multiple
> > > > build errors.
> > >
> > > Acked-by: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
> > >
> > > 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 ?

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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/