Re: [PM8921 MFD V5 0/6] pm8921 core and subdevices

[Samuel Ortiz <sameo@linux.intel.com>]

Hi Uwe,

On Tue, Apr 26, 2011 at 05:37:37PM +0200, Uwe Kleine-König wrote:
> Hi Samuel,
> 
> On Tue, Apr 26, 2011 at 05:07:47PM +0200, Samuel Ortiz wrote:
> > > - mfd_cell does not have platfrom_data anymore - use mfd data instead.
> > I'm currently moving that away from the mfd_cell structure, in my for-next
> > branch. mfd_cell now has a platform_data pointer back, and you should use
> > mfd_get_cell() if you really need to reach the mfd_cell pointer (which you
> > apparently don't). 
> > Could you please generate patchs 3 and 4 against this branch ?
> I just saw commit
> 
> 	5828fdb (mfd: Use mfd cell platform_data for mc13xxx cells platform bits)
> 
> in next. It has a Cc: for me, but I didn't got it by mail.
> 
> Did you send a mail and it didn't reach me? Or maybe you just forgot to
> send the mail out?
I haven't sent the patchset to lkml yet.


> Anyhow, Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> for
> 5828fdb, 
Thanks a lot.

> but I wonder if it's worth to add a new member to struct
> platform_device for that to work.
This has been discussed at length already, and it's probably the best solution
for now to clearly separate mfd cell access from the platform data bits.

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/