SM8650 is one of the platforms where board-level clocks (XO, sleep) definitions are split between the SoC dtsi file and the board file. This is not optimal, as the clocks are a part of the SoC + PMICs design. Frequencies are common for the whole set of devices using the same SoC. Remove the split and move frequencies to the SoC DTSI file. Suggested-by: Bjorn Andersson <andersson@xxxxxxxxxx> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> --- arch/arm64/boot/dts/qcom/sm8650-hdk.dts | 8 -------- arch/arm64/boot/dts/qcom/sm8650-mtp.dts | 8 -------- arch/arm64/boot/dts/qcom/sm8650-qrd.dts | 8 -------- arch/arm64/boot/dts/qcom/sm8650.dtsi | 2 ++ 4 files changed, 2 insertions(+), 24 deletions(-) diff --git a/arch/arm64/boot/dts/qcom/sm8650-hdk.dts b/arch/arm64/boot/dts/qcom/sm8650-hdk.dts index d0912735b54e5090f9f213c2c9341e03effbbbff..f509689af1c2e0dd6ea85edebe490538ff43964c 100644 --- a/arch/arm64/boot/dts/qcom/sm8650-hdk.dts +++ b/arch/arm64/boot/dts/qcom/sm8650-hdk.dts @@ -1112,10 +1112,6 @@ &sdhc_2 { status = "okay"; }; -&sleep_clk { - clock-frequency = <32764>; -}; - &swr0 { status = "okay"; @@ -1337,7 +1333,3 @@ &usb_dp_qmpphy { &usb_dp_qmpphy_out { remote-endpoint = <&pmic_glink_ss_in>; }; - -&xo_board { - clock-frequency = <76800000>; -}; diff --git a/arch/arm64/boot/dts/qcom/sm8650-mtp.dts b/arch/arm64/boot/dts/qcom/sm8650-mtp.dts index 76ef43c10f77d8329ccf0a05c9d590a46372315f..9a7f98d922af887657b4b56b5e271cc4b590db2f 100644 --- a/arch/arm64/boot/dts/qcom/sm8650-mtp.dts +++ b/arch/arm64/boot/dts/qcom/sm8650-mtp.dts @@ -729,10 +729,6 @@ &sdhc_2 { status = "okay"; }; -&sleep_clk { - clock-frequency = <32764>; -}; - &swr0 { status = "okay"; @@ -884,7 +880,3 @@ &usb_dp_qmpphy { &usb_dp_qmpphy_out { remote-endpoint = <&pmic_glink_ss_in>; }; - -&xo_board { - clock-frequency = <76800000>; -}; diff --git a/arch/arm64/boot/dts/qcom/sm8650-qrd.dts b/arch/arm64/boot/dts/qcom/sm8650-qrd.dts index 71033fba21b56bc63620dca3e453c14191739675..4943127e27a15af72f6d3f281550b1d8ac0ab77c 100644 --- a/arch/arm64/boot/dts/qcom/sm8650-qrd.dts +++ b/arch/arm64/boot/dts/qcom/sm8650-qrd.dts @@ -1040,10 +1040,6 @@ &remoteproc_mpss { status = "okay"; }; -&sleep_clk { - clock-frequency = <32764>; -}; - &spi4 { status = "okay"; @@ -1320,7 +1316,3 @@ &usb_dp_qmpphy { &usb_dp_qmpphy_out { remote-endpoint = <&redriver_ss_in>; }; - -&xo_board { - clock-frequency = <76800000>; -}; diff --git a/arch/arm64/boot/dts/qcom/sm8650.dtsi b/arch/arm64/boot/dts/qcom/sm8650.dtsi index 25e47505adcb790d09f1d2726386438487255824..cda249f767e5f1766134c62a8e5f312ed17e17d5 100644 --- a/arch/arm64/boot/dts/qcom/sm8650.dtsi +++ b/arch/arm64/boot/dts/qcom/sm8650.dtsi @@ -38,11 +38,13 @@ clocks { xo_board: xo-board { compatible = "fixed-clock"; #clock-cells = <0>; + clock-frequency = <76800000>; }; sleep_clk: sleep-clk { compatible = "fixed-clock"; #clock-cells = <0>; + clock-frequency = <32764>; }; bi_tcxo_div2: bi-tcxo-div2-clk { -- 2.39.5