On 15.02.2023 08:03, Manivannan Sadhasivam wrote: > Qualcomm platforms making use of CPUFreq HW Engine (EPSS/OSM) supply clocks > to the CPU cores. But this relationship is not represented in DTS so far. > > So let's make cpufreq node as the clock provider and CPU nodes as the > consumers. The clock index for each CPU node is based on the frequency > domain index. > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx> > --- Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx> Konrad > arch/arm64/boot/dts/qcom/sm8350.dtsi | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/sm8350.dtsi b/arch/arm64/boot/dts/qcom/sm8350.dtsi > index 0a422637b61f..1b423c42ec0d 100644 > --- a/arch/arm64/boot/dts/qcom/sm8350.dtsi > +++ b/arch/arm64/boot/dts/qcom/sm8350.dtsi > @@ -48,6 +48,7 @@ CPU0: cpu@0 { > device_type = "cpu"; > compatible = "qcom,kryo685"; > reg = <0x0 0x0>; > + clocks = <&cpufreq_hw 0>; > enable-method = "psci"; > next-level-cache = <&L2_0>; > qcom,freq-domain = <&cpufreq_hw 0>; > @@ -69,6 +70,7 @@ CPU1: cpu@100 { > device_type = "cpu"; > compatible = "qcom,kryo685"; > reg = <0x0 0x100>; > + clocks = <&cpufreq_hw 0>; > enable-method = "psci"; > next-level-cache = <&L2_100>; > qcom,freq-domain = <&cpufreq_hw 0>; > @@ -86,6 +88,7 @@ CPU2: cpu@200 { > device_type = "cpu"; > compatible = "qcom,kryo685"; > reg = <0x0 0x200>; > + clocks = <&cpufreq_hw 0>; > enable-method = "psci"; > next-level-cache = <&L2_200>; > qcom,freq-domain = <&cpufreq_hw 0>; > @@ -103,6 +106,7 @@ CPU3: cpu@300 { > device_type = "cpu"; > compatible = "qcom,kryo685"; > reg = <0x0 0x300>; > + clocks = <&cpufreq_hw 0>; > enable-method = "psci"; > next-level-cache = <&L2_300>; > qcom,freq-domain = <&cpufreq_hw 0>; > @@ -120,6 +124,7 @@ CPU4: cpu@400 { > device_type = "cpu"; > compatible = "qcom,kryo685"; > reg = <0x0 0x400>; > + clocks = <&cpufreq_hw 1>; > enable-method = "psci"; > next-level-cache = <&L2_400>; > qcom,freq-domain = <&cpufreq_hw 1>; > @@ -137,6 +142,7 @@ CPU5: cpu@500 { > device_type = "cpu"; > compatible = "qcom,kryo685"; > reg = <0x0 0x500>; > + clocks = <&cpufreq_hw 1>; > enable-method = "psci"; > next-level-cache = <&L2_500>; > qcom,freq-domain = <&cpufreq_hw 1>; > @@ -155,6 +161,7 @@ CPU6: cpu@600 { > device_type = "cpu"; > compatible = "qcom,kryo685"; > reg = <0x0 0x600>; > + clocks = <&cpufreq_hw 1>; > enable-method = "psci"; > next-level-cache = <&L2_600>; > qcom,freq-domain = <&cpufreq_hw 1>; > @@ -172,6 +179,7 @@ CPU7: cpu@700 { > device_type = "cpu"; > compatible = "qcom,kryo685"; > reg = <0x0 0x700>; > + clocks = <&cpufreq_hw 2>; > enable-method = "psci"; > next-level-cache = <&L2_700>; > qcom,freq-domain = <&cpufreq_hw 2>; > @@ -2283,6 +2291,7 @@ cpufreq_hw: cpufreq@18591000 { > clock-names = "xo", "alternate"; > > #freq-domain-cells = <1>; > + #clock-cells = <1>; > }; > > ufs_mem_hc: ufshc@1d84000 {