Hi, On Fri, 15 May 2020 at 14:00, Clément Péron <[email protected]> wrote: > > Hi Viresh, > > On Fri, 15 May 2020 at 09:57, Viresh Kumar <[email protected]> wrote: > > > > Hi, > > > > This series reintroduces the usage of regulator_enable/disable() to the > > OPP core after the previous attempt was reverted [1] shortly after getting > > applied. This time the regulator is enabled only after it is configured > > by the OPP core. > > > > Marek, Kamil and Clément: Can you guys please test this out and report > > if this doesn't work as expected ? > > I have reviewed the patch and it seems fine for my use case. > Unfortunately I can't test it until next week.
Ok, before the patch the regulator was released by regulator_late_cleanup() : [ 33.756849] vdd-gpu: disabling Now it works fine and the vdd-gpu is no more disabled. Tested-by: Clément Péron <[email protected]> Regards, Clement > > Acked-by: Clément Péron <[email protected]> > > Regards, > Clement > > > > > > -- > > viresh > > > > [1] > > https://lore.kernel.org/lkml/[email protected]/ > > > > Kamil Konieczny (1): > > opp: core: add regulators enable and disable > > > > Viresh Kumar (1): > > opp: Reorder the code for !target_freq case > > > > drivers/opp/core.c | 39 ++++++++++++++++++++++++++++++++++----- > > drivers/opp/opp.h | 2 ++ > > 2 files changed, 36 insertions(+), 5 deletions(-) > > > > -- > > 2.25.0.rc1.19.g042ed3e048af > >
