Re: [PATCH] regulator: core: Use postcore_initcall(regulator_init_complete)

[Mark Brown <broonie@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1189 bytes --]

On Mon, Nov 25, 2013 at 07:57:35PM -0200, Fabio Estevam wrote:

> Calling regulator_init_complete via late_initcall can be too 'late' for the 
> regulator to be available for the pwm backlight driver.

> Call regulator_init_complete with postcore_initcall level instead.

That's too early for some of the other functionality in this function
though - I'd be particularly worried about the code that goes aroud
turning off unused supplies running before drivers have a chance to load
either breaking things or causing user visible glitches (and of course
the goal here is to get have this run before that happens...).

I seem to remember that there's also some one initcall per object
limitation so if we are going to try to dance this dance purely in the
regulator code it'd probably need to be split out into another object as
well as another function.  The other options that were suggested were to
either have the DT code push out the announcement or change the callers
to check of_have_populated_dt() as well as looking at the flag, of this
lot changing the callers is probably the cleanest.  I'll probably do
that tomorrow if a patch doesn't turn up before then.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]