On 29-01-16, 17:48, Stephen Boyd wrote:
I'll look at this early next week.
Thanks.
Just a note for future work, I think we're going to need to add some sort of enable/disable into the OPP layer. At least in qcom designs, if a clock is off we don't want the voltage requirement for that clock to factor into the final voltage on the regulator. Furthermore, we want to disable the regulator with regulator_disable() if all the clocks are off.
Hmm, we need to discuss more on this, maybe in a HO sometime or at connect :)
This is also a problem with cpufreq-dt. The regulators and clocks are assumed to be enabled out of the bootloader, which may not even be true. Now that OPP layer is managing all the clocks and regulators here we're going to need to do something to make sure they're on and controllable.
Also, I fail to understand, how the clock of a 'online' CPU can be off?