devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v1] ARM: dts: qcom: ipq4019: fix sleep clock
@ 2021-12-20 17:03 Christian Lamparter
  2022-02-01  5:19 ` (subset) " Bjorn Andersson
  0 siblings, 1 reply; 2+ messages in thread
From: Christian Lamparter @ 2021-12-20 17:03 UTC (permalink / raw)
  To: linux-arm-msm, linux-arm-kernel, devicetree
  Cc: Pavel Kubelun, Bjorn Andersson, Andy Gross, Rob Herring

From: Pavel Kubelun <be.dissent@gmail.com>

It seems like sleep_clk was copied from ipq806x.
Fix ipq40xx sleep_clk to the value QSDK defines.

Link: https://source.codeaurora.org/quic/qsdk/oss/kernel/linux-msm/commit/?id=d92ec59973484acc86dd24b67f10f8911b4b4b7d
Link: https://patchwork.kernel.org/comment/22721613/
Fixes: bec6ba4cdf2a ("qcom: ipq4019: Add basic board/dts support for IPQ4019 SoC")
Suggested-by: Bjorn Andersson <bjorn.andersson@linaro.org> (clock-output-names)
Signed-off-by: Pavel Kubelun <be.dissent@gmail.com>
Signed-off-by: Christian Lamparter <chunkeey@gmail.com> (removed clock rename)
---

@Bjorn,

as promised, here's the clock-output-names version.
I've tested it extensively with the lonely devices in the office.

Thank you.
---
 arch/arm/boot/dts/qcom-ipq4019.dtsi | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/qcom-ipq4019.dtsi b/arch/arm/boot/dts/qcom-ipq4019.dtsi
index 7a181352b306..632038d2f7b7 100644
--- a/arch/arm/boot/dts/qcom-ipq4019.dtsi
+++ b/arch/arm/boot/dts/qcom-ipq4019.dtsi
@@ -146,7 +146,8 @@ pmu {
 	clocks {
 		sleep_clk: sleep_clk {
 			compatible = "fixed-clock";
-			clock-frequency = <32768>;
+			clock-frequency = <32000>;
+			clock-output-names = "gcc_sleep_clk_src";
 			#clock-cells = <0>;
 		};
 
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: (subset) [PATCH v1] ARM: dts: qcom: ipq4019: fix sleep clock
  2021-12-20 17:03 [PATCH v1] ARM: dts: qcom: ipq4019: fix sleep clock Christian Lamparter
@ 2022-02-01  5:19 ` Bjorn Andersson
  0 siblings, 0 replies; 2+ messages in thread
From: Bjorn Andersson @ 2022-02-01  5:19 UTC (permalink / raw)
  To: devicetree, linux-arm-kernel, linux-arm-msm, Christian Lamparter
  Cc: Rob Herring, Pavel Kubelun, Andy Gross

On Mon, 20 Dec 2021 18:03:52 +0100, Christian Lamparter wrote:
> From: Pavel Kubelun <be.dissent@gmail.com>
> 
> It seems like sleep_clk was copied from ipq806x.
> Fix ipq40xx sleep_clk to the value QSDK defines.
> 
> 

Applied, thanks!

[1/1] ARM: dts: qcom: ipq4019: fix sleep clock
      commit: 3d7e7980993d2c1ae42d3d314040fc2de6a9c45f

Best regards,
-- 
Bjorn Andersson <bjorn.andersson@linaro.org>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-02-01  5:20 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-12-20 17:03 [PATCH v1] ARM: dts: qcom: ipq4019: fix sleep clock Christian Lamparter
2022-02-01  5:19 ` (subset) " Bjorn Andersson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).