On 08/24/2016 11:58 AM, Stephen Boyd wrote: > On 08/11, Rajendra Nayak wrote: >> This would be useful in subsequent patches when the .set_rate operation >> would need to identify if the PLL is actually enabled >> >> Signed-off-by: Rajendra Nayak <rnayak@xxxxxxxxxxxxxx> >> --- > > Hmmm I suspect I never implemented the is_enabled op because that > will happen to turn off clks during late init that shouldn't > otherwise be disabled because the framework now can see that some > PLL is enabled out of the bootloader. Is that happening now? We > really should fix the framework to make this not be a problem, > mostly by finishing off the clk handoff patches that Mike posted > a while back. But either way, I'm worried with these patches that > implement is_enabled ops. I did not see any, but will try and test some more to see if we indeed run into such issues -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html