Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1469100

Re: [PATCH v2 09/10] clk: qcom: Add .is_enabled ops for clk-alpha-pll

From Stephen Boyd <sboyd@codeaurora.org>
Newsgroups linux.kernel
Subject Re: [PATCH v2 09/10] clk: qcom: Add .is_enabled ops for clk-alpha-pll
Date 2016-08-24 08:30 +0200
Message-ID <s9zRf-6Oo-9@gated-at.bofh.it> (permalink)
References <s4TQB-1tP-7@gated-at.bofh.it> <s4TQC-1tP-21@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


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@codeaurora.org>
> ---

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.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v2 09/10] clk: qcom: Add .is_enabled ops for clk-alpha-pll Rajendra Nayak <rnayak@codeaurora.org> - 2016-08-11 10:50 +0200
  Re: [PATCH v2 09/10] clk: qcom: Add .is_enabled ops for clk-alpha-pll Stephen Boyd <sboyd@codeaurora.org> - 2016-08-24 08:30 +0200
    Re: [PATCH v2 09/10] clk: qcom: Add .is_enabled ops for clk-alpha-pll Rajendra Nayak <rnayak@codeaurora.org> - 2016-08-25 11:20 +0200

csiph-web