diff mbox series

arm64: dts: exynos: move XTCXO clock frequency to board in Exynos Auto v9

Message ID 20220503092631.174713-1-krzysztof.kozlowski@linaro.org (mailing list archive)
State Accepted
Commit f223095c6e1d20aac831e75016b58ca8adbbf122
Headers show
Series arm64: dts: exynos: move XTCXO clock frequency to board in Exynos Auto v9 | expand

Commit Message

Krzysztof Kozlowski May 3, 2022, 9:26 a.m. UTC
The external oscillator - XTCXO - is an input to the SoC.  It is defined
in the Exynos Auto v9 SoC DTSI, because all boards will provide it and
clock controller bindings expect it, however the actual frequency of the
clock should be determined by the board.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts | 4 ++++
 arch/arm64/boot/dts/exynos/exynosautov9.dtsi     | 1 -
 2 files changed, 4 insertions(+), 1 deletion(-)

Comments

Chanho Park May 3, 2022, 9:37 a.m. UTC | #1
> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Sent: Tuesday, May 3, 2022 6:27 PM
> To: Rob Herring <robh+dt@kernel.org>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt@linaro.org>; Alim Akhtar
<alim.akhtar@samsung.com>;
> devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
> samsung-soc@vger.kernel.org; linux-kernel@vger.kernel.org
> Cc: Chanho Park <chanho61.park@samsung.com>; Krzysztof Kozlowski
> <krzysztof.kozlowski@linaro.org>
> Subject: [PATCH] arm64: dts: exynos: move XTCXO clock frequency to board
> in Exynos Auto v9
> 
> The external oscillator - XTCXO - is an input to the SoC.  It is defined
> in the Exynos Auto v9 SoC DTSI, because all boards will provide it and
> clock controller bindings expect it, however the actual frequency of the
> clock should be determined by the board.

Thanks for the update.
Reviewed-by: Chanho Park <chanho61.park@samsung.com>

Best Regards,
Chanho Park
Krzysztof Kozlowski May 4, 2022, 8:30 a.m. UTC | #2
On Tue, 3 May 2022 11:26:31 +0200, Krzysztof Kozlowski wrote:
> The external oscillator - XTCXO - is an input to the SoC.  It is defined
> in the Exynos Auto v9 SoC DTSI, because all boards will provide it and
> clock controller bindings expect it, however the actual frequency of the
> clock should be determined by the board.
> 
> 

Applied, thanks!

[1/1] arm64: dts: exynos: move XTCXO clock frequency to board in Exynos Auto v9
      commit: f223095c6e1d20aac831e75016b58ca8adbbf122

Best regards,
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts b/arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts
index 57518cb5e8c4..17e568853eb6 100644
--- a/arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts
+++ b/arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts
@@ -58,3 +58,7 @@  &ufs_0 {
 &usi_0 {
 	status = "okay";
 };
+
+&xtcxo {
+	clock-frequency = <26000000>;
+};
diff --git a/arch/arm64/boot/dts/exynos/exynosautov9.dtsi b/arch/arm64/boot/dts/exynos/exynosautov9.dtsi
index 807d500d6022..68d087ed0459 100644
--- a/arch/arm64/boot/dts/exynos/exynosautov9.dtsi
+++ b/arch/arm64/boot/dts/exynos/exynosautov9.dtsi
@@ -153,7 +153,6 @@  fixed-rate-clocks {
 		xtcxo: clock {
 			compatible = "fixed-clock";
 			#clock-cells = <0>;
-			clock-frequency = <26000000>;
 			clock-output-names = "oscclk";
 		};