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.
signature.asc
Description: Digital signature