In the bindings that landed for the gpucc we require that the XO clock
(one possible parent of the gpucc) be listed. The code doesn't use
this yet--this is just to allow us to move toward the day when it does
use it. What the code does do today is to hardcode the parent name to
"bi_tcxo". That's all well and good.
...but the example in the bindings shows this clock mapping to the
clock "xo_board". On the current sdm845.dtsi file the "xo_board"
clock is a fixed clock with an output name of "xo_board". The clock
with the name "bi_tcxo" is actually provided by the RPMh Clock
Controller. Presumably that's the one that was wanted.
Let's update the example to make this clearer.
Fixes: e431c92188a9 ("dt-bindings: clock: Introduce QCOM Graphics clock
bindings")
Signed-off-by: Douglas Anderson
---
Documentation/devicetree/bindings/clock/qcom,gpucc.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/clock/qcom,gpucc.txt
b/Documentation/devicetree/bindings/clock/qcom,gpucc.txt
index 9d0358cc08b4..4e5215ef1acd 100644
--- a/Documentation/devicetree/bindings/clock/qcom,gpucc.txt
+++ b/Documentation/devicetree/bindings/clock/qcom,gpucc.txt
@@ -17,6 +17,6 @@ Example:
#clock-cells = <1>;
#reset-cells = <1>;
#power-domain-cells = <1>;
- clocks = <&xo_board>;
+ clocks = <&rpmhcc RPMH_CXO_CLK>;
clock-names = "xo";
};
--
2.20.0.rc0.387.gc7a69e6b6c-goog
