| Message ID | 20230215070400.5901-3-manivannan.sadhasivam@linaro.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 782ad94df85c4dfa7ab70b5bc9a2882c367844d3 |
| Headers | show |
| Series | arm64: dts: qcom: Supply clock from cpufreq node to CPUs | expand |
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@linaro.org> > --- Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org> Konrad > arch/arm64/boot/dts/qcom/sc7280.dtsi | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi > index bdcb74925313..d9b6e028cdac 100644 > --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi > +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi > @@ -168,6 +168,7 @@ CPU0: cpu@0 { > device_type = "cpu"; > compatible = "qcom,kryo"; > reg = <0x0 0x0>; > + clocks = <&cpufreq_hw 0>; > enable-method = "psci"; > cpu-idle-states = <&LITTLE_CPU_SLEEP_0 > &LITTLE_CPU_SLEEP_1 > @@ -193,6 +194,7 @@ CPU1: cpu@100 { > device_type = "cpu"; > compatible = "qcom,kryo"; > reg = <0x0 0x100>; > + clocks = <&cpufreq_hw 0>; > enable-method = "psci"; > cpu-idle-states = <&LITTLE_CPU_SLEEP_0 > &LITTLE_CPU_SLEEP_1 > @@ -214,6 +216,7 @@ CPU2: cpu@200 { > device_type = "cpu"; > compatible = "qcom,kryo"; > reg = <0x0 0x200>; > + clocks = <&cpufreq_hw 0>; > enable-method = "psci"; > cpu-idle-states = <&LITTLE_CPU_SLEEP_0 > &LITTLE_CPU_SLEEP_1 > @@ -235,6 +238,7 @@ CPU3: cpu@300 { > device_type = "cpu"; > compatible = "qcom,kryo"; > reg = <0x0 0x300>; > + clocks = <&cpufreq_hw 0>; > enable-method = "psci"; > cpu-idle-states = <&LITTLE_CPU_SLEEP_0 > &LITTLE_CPU_SLEEP_1 > @@ -256,6 +260,7 @@ CPU4: cpu@400 { > device_type = "cpu"; > compatible = "qcom,kryo"; > reg = <0x0 0x400>; > + clocks = <&cpufreq_hw 1>; > enable-method = "psci"; > cpu-idle-states = <&BIG_CPU_SLEEP_0 > &BIG_CPU_SLEEP_1 > @@ -277,6 +282,7 @@ CPU5: cpu@500 { > device_type = "cpu"; > compatible = "qcom,kryo"; > reg = <0x0 0x500>; > + clocks = <&cpufreq_hw 1>; > enable-method = "psci"; > cpu-idle-states = <&BIG_CPU_SLEEP_0 > &BIG_CPU_SLEEP_1 > @@ -298,6 +304,7 @@ CPU6: cpu@600 { > device_type = "cpu"; > compatible = "qcom,kryo"; > reg = <0x0 0x600>; > + clocks = <&cpufreq_hw 1>; > enable-method = "psci"; > cpu-idle-states = <&BIG_CPU_SLEEP_0 > &BIG_CPU_SLEEP_1 > @@ -319,6 +326,7 @@ CPU7: cpu@700 { > device_type = "cpu"; > compatible = "qcom,kryo"; > reg = <0x0 0x700>; > + clocks = <&cpufreq_hw 2>; > enable-method = "psci"; > cpu-idle-states = <&BIG_CPU_SLEEP_0 > &BIG_CPU_SLEEP_1 > @@ -5337,6 +5345,7 @@ cpufreq_hw: cpufreq@18591000 { > clocks = <&rpmhcc RPMH_CXO_CLK>, <&gcc GCC_GPLL0>; > clock-names = "xo", "alternate"; > #freq-domain-cells = <1>; > + #clock-cells = <1>; > }; > }; >
diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi index bdcb74925313..d9b6e028cdac 100644 --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi @@ -168,6 +168,7 @@ CPU0: cpu@0 { device_type = "cpu"; compatible = "qcom,kryo"; reg = <0x0 0x0>; + clocks = <&cpufreq_hw 0>; enable-method = "psci"; cpu-idle-states = <&LITTLE_CPU_SLEEP_0 &LITTLE_CPU_SLEEP_1 @@ -193,6 +194,7 @@ CPU1: cpu@100 { device_type = "cpu"; compatible = "qcom,kryo"; reg = <0x0 0x100>; + clocks = <&cpufreq_hw 0>; enable-method = "psci"; cpu-idle-states = <&LITTLE_CPU_SLEEP_0 &LITTLE_CPU_SLEEP_1 @@ -214,6 +216,7 @@ CPU2: cpu@200 { device_type = "cpu"; compatible = "qcom,kryo"; reg = <0x0 0x200>; + clocks = <&cpufreq_hw 0>; enable-method = "psci"; cpu-idle-states = <&LITTLE_CPU_SLEEP_0 &LITTLE_CPU_SLEEP_1 @@ -235,6 +238,7 @@ CPU3: cpu@300 { device_type = "cpu"; compatible = "qcom,kryo"; reg = <0x0 0x300>; + clocks = <&cpufreq_hw 0>; enable-method = "psci"; cpu-idle-states = <&LITTLE_CPU_SLEEP_0 &LITTLE_CPU_SLEEP_1 @@ -256,6 +260,7 @@ CPU4: cpu@400 { device_type = "cpu"; compatible = "qcom,kryo"; reg = <0x0 0x400>; + clocks = <&cpufreq_hw 1>; enable-method = "psci"; cpu-idle-states = <&BIG_CPU_SLEEP_0 &BIG_CPU_SLEEP_1 @@ -277,6 +282,7 @@ CPU5: cpu@500 { device_type = "cpu"; compatible = "qcom,kryo"; reg = <0x0 0x500>; + clocks = <&cpufreq_hw 1>; enable-method = "psci"; cpu-idle-states = <&BIG_CPU_SLEEP_0 &BIG_CPU_SLEEP_1 @@ -298,6 +304,7 @@ CPU6: cpu@600 { device_type = "cpu"; compatible = "qcom,kryo"; reg = <0x0 0x600>; + clocks = <&cpufreq_hw 1>; enable-method = "psci"; cpu-idle-states = <&BIG_CPU_SLEEP_0 &BIG_CPU_SLEEP_1 @@ -319,6 +326,7 @@ CPU7: cpu@700 { device_type = "cpu"; compatible = "qcom,kryo"; reg = <0x0 0x700>; + clocks = <&cpufreq_hw 2>; enable-method = "psci"; cpu-idle-states = <&BIG_CPU_SLEEP_0 &BIG_CPU_SLEEP_1 @@ -5337,6 +5345,7 @@ cpufreq_hw: cpufreq@18591000 { clocks = <&rpmhcc RPMH_CXO_CLK>, <&gcc GCC_GPLL0>; clock-names = "xo", "alternate"; #freq-domain-cells = <1>; + #clock-cells = <1>; }; };
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@linaro.org> --- arch/arm64/boot/dts/qcom/sc7280.dtsi | 9 +++++++++ 1 file changed, 9 insertions(+)