Re: [PATCH 1/4] arm64: dts: sdm845: Add OPP table for all qup devices

[Matthias Kaehlcke]

On Tue, Jun 30, 2020 at 02:15:09PM +0530, Rajendra Nayak wrote:
> qup has a requirement to vote on the performance state of the CX domain
> in sdm845 devices. Add OPP tables for these and also add power-domains
> property for all qup instances for uart and spi.
> i2c does not support scaling and uses a fixed clock.
> 
> Signed-off-by: Rajendra Nayak 
> Signed-off-by: Stephen Boyd 

Reviewed-by: Matthias Kaehlcke 

[PATCH 1/4] arm64: dts: sdm845: Add OPP table for all qup devices

[Rajendra Nayak]

qup has a requirement to vote on the performance state of the CX domain
in sdm845 devices. Add OPP tables for these and also add power-domains
property for all qup instances for uart and spi.
i2c does not support scaling and uses a fixed clock.

Signed-off-by: Rajendra Nayak 
Signed-off-by: Stephen Boyd 
---
 arch/arm64/boot/dts/qcom/sdm845.dtsi | 83 
 1 file changed, 83 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi 
b/arch/arm64/boot/dts/qcom/sdm845.dtsi
index 8eb5a31..db1f82b 100644
--- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
@@ -805,6 +805,25 @@
clock-names = "core";
};
 
+   qup_opp_table: qup-opp-table {
+   compatible = "operating-points-v2";
+
+   opp-1920 {
+   opp-hz = /bits/ 64 <1920>;
+   required-opps = <&rpmhpd_opp_min_svs>;
+   };
+
+   opp-7500 {
+   opp-hz = /bits/ 64 <7500>;
+   required-opps = <&rpmhpd_opp_low_svs>;
+   };
+
+   opp-1 {
+   opp-hz = /bits/ 64 <1>;
+   required-opps = <&rpmhpd_opp_svs>;
+   };
+   };
+
qupv3_id_0: geniqup@8c {
compatible = "qcom,geni-se-qup";
reg = <0 0x008c 0 0x6000>;
@@ -839,6 +858,8 @@
interrupts = ;
#address-cells = <1>;
#size-cells = <0>;
+   power-domains = <&rpmhpd SDM845_CX>;
+   operating-points-v2 = <&qup_opp_table>;
status = "disabled";
};
 
@@ -850,6 +871,8 @@
pinctrl-names = "default";
pinctrl-0 = <&qup_uart0_default>;
interrupts = ;
+   power-domains = <&rpmhpd SDM845_CX>;
+   operating-points-v2 = <&qup_opp_table>;
status = "disabled";
};
 
@@ -876,6 +899,8 @@
interrupts = ;
#address-cells = <1>;
#size-cells = <0>;
+   power-domains = <&rpmhpd SDM845_CX>;
+   operating-points-v2 = <&qup_opp_table>;
status = "disabled";
};
 
@@ -887,6 +912,8 @@
pinctrl-names = "default";
pinctrl-0 = <&qup_uart1_default>;
interrupts = ;
+   power-domains = <&rpmhpd SDM845_CX>;
+   operating-points-v2 = <&qup_opp_table>;
status = "disabled";
};
 
@@ -913,6 +940,8 @@
interrupts = ;
#address-cells = <1>;
#size-cells = <0>;
+   power-domains = <&rpmhpd SDM845_CX>;
+   operating-points-v2 = <&qup_opp_table>;
status = "disabled";
};
 
@@ -924,6 +953,8 @@
pinctrl-names = "default";
pinctrl-0 = <&qup_uart2_default>;
interrupts = ;
+   power-domains = <&rpmhpd SDM845_CX>;
+   operating-points-v2 = <&qup_opp_table>;
status = "disabled";
};
 
@@ -950,6 +981,8 @@
interrupts = ;
#address-cells = <1>;
#size-cells = <0>;
+   power-domains = <&rpmhpd SDM845_CX>;
+   operating-points-v2 = <&qup_opp_table>;
status = "disabled";
};
 
@@ -961,6 +994,8 @@
pinctrl-names = "default";
pinctrl-0 = <&qup_uart3_default>;
interrupts = ;
+   power-domains = <&rpmhpd SDM845_CX>;
+   operating-points-v2 = <&qup_opp_table>;
status = "disabled";
};
 
@@ -987,6 +1022,8 @@
interrupts = ;
#address-cells = <1>;