diff mbox series

[2/5,v2] clk: qcom: Elaborate on "active" clocks in the RPM clock bindings

Message ID 20170419091326.11226-2-linus.walleij@linaro.org
State Superseded
Headers show
Series [1/5,v2] clk: qcom: Update DT bindings for the MSM8660/APQ8060 RPMCC | expand

Commit Message

Linus Walleij April 19, 2017, 9:13 a.m. UTC
The concept of "active" clocks is just explained in a bried comment in the
device driver, let's explain it a bit more in the device tree bindings
so everyone understands this.

Cc: devicetree@vger.kernel.org
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

---
ChangeLog v1->v2:
- Reword slighty in accordance with Stephens feedback.
---
 Documentation/devicetree/bindings/clock/qcom,rpmcc.txt | 8 ++++++++
 1 file changed, 8 insertions(+)

-- 
2.9.3

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Rob Herring (Arm) April 28, 2017, 1:35 p.m. UTC | #1
On Wed, Apr 19, 2017 at 11:13:23AM +0200, Linus Walleij wrote:
> The concept of "active" clocks is just explained in a bried comment in the

> device driver, let's explain it a bit more in the device tree bindings

> so everyone understands this.

> 

> Cc: devicetree@vger.kernel.org

> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

> ---

> ChangeLog v1->v2:

> - Reword slighty in accordance with Stephens feedback.

> ---

>  Documentation/devicetree/bindings/clock/qcom,rpmcc.txt | 8 ++++++++

>  1 file changed, 8 insertions(+)


Despite my objections, you're just documenting what is already there.

Acked-by: Rob Herring <robh@kernel.org>

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Stephen Boyd June 1, 2017, 7:39 a.m. UTC | #2
On 04/28, Rob Herring wrote:
> On Wed, Apr 19, 2017 at 11:13:23AM +0200, Linus Walleij wrote:

> > The concept of "active" clocks is just explained in a bried comment in the

> > device driver, let's explain it a bit more in the device tree bindings

> > so everyone understands this.

> > 

> > Cc: devicetree@vger.kernel.org

> > Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

> > ---

> > ChangeLog v1->v2:

> > - Reword slighty in accordance with Stephens feedback.

> > ---

> >  Documentation/devicetree/bindings/clock/qcom,rpmcc.txt | 8 ++++++++

> >  1 file changed, 8 insertions(+)

> 

> Despite my objections, you're just documenting what is already there.

> 

> Acked-by: Rob Herring <robh@kernel.org>


Perhaps this can be put into the clk driver instead? Is that the
concern?

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/clock/qcom,rpmcc.txt b/Documentation/devicetree/bindings/clock/qcom,rpmcc.txt
index d470a0187035..833a89f06ae0 100644
--- a/Documentation/devicetree/bindings/clock/qcom,rpmcc.txt
+++ b/Documentation/devicetree/bindings/clock/qcom,rpmcc.txt
@@ -18,6 +18,14 @@  Required properties :
 
 - #clock-cells : shall contain 1
 
+The clock enumerators are defined in <dt-bindings/clock/qcom,rpmcc.h>
+and come in pairs: FOO_CLK followed by FOO_A_CLK. The latter clock
+is an "active" clock, which means that the consumer only care that the
+clock is available when the apps CPU subsystem is active, i.e. not
+suspended or in deep idle. If it is important that the clock keeps running
+during system suspend, you need to specify the non-active clock, the one
+not containing *_A_* in the enumerator name.
+
 Example:
 	smd {
 		compatible = "qcom,smd";