Re: [PATCH 2/2] regulator/mc13783: various cleanups

[Liam Girdwood <lrg@slimlogic.co.uk>]

On Wed, 2009-11-11 at 15:16 +0100, Uwe Kleine-König wrote:
> Hello,
> 
> On Wed, Nov 11, 2009 at 02:11:48PM +0000, Liam Girdwood wrote:
> > On Tue, 2009-11-10 at 09:18 +0100, Uwe Kleine-König wrote:
> > > - define needed registers and bits in the driver
> > > - properly namespace functions and structs
> > > - fix locking as required by patch
> > >   "mfd/mc13783: near complete rewrite"
> > > - use platform_data as provided by "mfd/mc13783: near complete rewrite"
> > >   instead of accessing struct mc13783
> > > - struct mc13783_regulator_priv.desc is (and was) unused and so can go
> > >   away
> > > - use cpp magic to initialize mc13783_regulators
> > > - bring MODULE_LICENSE in sync with actual copyright
> > > - minor style fixes
> > > 
> > > This allows not including mc13783-private.h which I intend to remove
> > > soon.
> > > 
> > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> > > Cc: Sascha Hauer <s.hauer@pengutronix.de>
> > > Cc: Liam Girdwood <lrg@slimlogic.co.uk>
> > > Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>
> > > Cc: Samuel Ortiz <sameo@linux.intel.com>
> > > ---
> > 
> > I assume the new mc13783 function and type definitions are in mfd-next ?
> Yes, it's 3a69e7d9f6b1fb6c6db7f23e22351e3db7d55ebb in todays linux-next.
>  

Applied

Thanks.

Liam