lkml.org 
[lkml]   [2014]   [May]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/3] mfd-core: Don't register supplies from add_device, add register_supply_aliases()
On Mon, May 19, 2014 at 08:47:05AM +0100, Lee Jones wrote:
> > We cannot register supply alias in mfd_add_device before calling
> > platform_add_device, for 2 reasons:
> > 1) devm resources may not be registered before the (platform) drivers probe
> > method runs
> > 2) The platform-dev's name must be set before registering the aliases which
> > happens from platform_add_device.
> >
> > So stop registering supply aliases from mfd_add_device, and add a
> > mfd_register_supply_aliases helper functions for the cell's plaform driver
> > probe method to use.
> >
> > Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> > ---
> > drivers/mfd/mfd-core.c | 37 +++++++++++++++++++++----------------
> > include/linux/mfd/core.h | 6 +++++-
> > 2 files changed, 26 insertions(+), 17 deletions(-)
>
> Change looks reasonable to me, but I'd like to have Mark look over the
> changes. If he's okay with them I think it's best for this set to go
> through the MFD tree as a whole.

This should already be fixed by this patch:

mfd: core: Don't use devres functions before device is added

You had posted to say it was applied although I haven't checked
your tree for it directly.

Thanks,
Charles


\
 
 \ /
  Last update: 2014-05-19 10:21    [W:0.103 / U:0.456 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site