devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 00/10] More 8180x dts fixes
@ 2023-12-30  0:05 Konrad Dybcio
  2023-12-30  0:05 ` [PATCH 01/10] dt-bindings: clock: gcc-sc8180x: Add the missing CX power domain Konrad Dybcio
                   ` (10 more replies)
  0 siblings, 11 replies; 13+ messages in thread
From: Konrad Dybcio @ 2023-12-30  0:05 UTC (permalink / raw)
  To: Bjorn Andersson, Michael Turquette, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vinod Koul
  Cc: Marijn Suijten, linux-arm-msm, linux-clk, devicetree,
	linux-kernel, Konrad Dybcio

SC8180X has got various random power plumbing issues, this series tries
to address that, and introduces RPMh sleep stats.

Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
Konrad Dybcio (10):
      dt-bindings: clock: gcc-sc8180x: Add the missing CX power domain
      arm64: dts: qcom: sc8180x: Hook up VDD_CX as GCC parent domain
      arm64: dts: qcom: sc8180x: Fix up big CPU idle state entry latency
      arm64: dts: qcom: sc8180x: Add missing CPU off state
      arm64: dts: qcom: sc8180x: Fix eDP PHY power-domains
      arm64: dts: qcom: sc8180x: Don't hold MDP core clock at FMAX
      arm64: dts: qcom: sc8180x: Require LOW_SVS vote for MMCX if DISPCC is on
      arm64: dts: qcom: sc8180x: Add missing CPU<->MDP_CFG path
      arm64: dts: qcom: sc8180x: Shrink aoss_qmp register space size
      arm64: dts: qcom: sc8180x: Add RPMh sleep stats

 .../bindings/clock/qcom,gcc-sc8180x.yaml           |  7 ++++
 arch/arm64/boot/dts/qcom/sc8180x.dtsi              | 43 ++++++++++++++++------
 2 files changed, 38 insertions(+), 12 deletions(-)
---
base-commit: 39676dfe52331dba909c617f213fdb21015c8d10
change-id: 20231230-topic-8180_more_fixes-81cf3c8a2637

Best regards,
-- 
Konrad Dybcio <konrad.dybcio@linaro.org>


^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH 01/10] dt-bindings: clock: gcc-sc8180x: Add the missing CX power domain
  2023-12-30  0:05 [PATCH 00/10] More 8180x dts fixes Konrad Dybcio
@ 2023-12-30  0:05 ` Konrad Dybcio
  2024-01-04  8:24   ` Krzysztof Kozlowski
  2023-12-30  0:05 ` [PATCH 02/10] arm64: dts: qcom: sc8180x: Hook up VDD_CX as GCC parent domain Konrad Dybcio
                   ` (9 subsequent siblings)
  10 siblings, 1 reply; 13+ messages in thread
From: Konrad Dybcio @ 2023-12-30  0:05 UTC (permalink / raw)
  To: Bjorn Andersson, Michael Turquette, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vinod Koul
  Cc: Marijn Suijten, linux-arm-msm, linux-clk, devicetree,
	linux-kernel, Konrad Dybcio

The GCC block is (mostly) powered by the VDD_CX rail. Allow specifying
it in power-domains.

Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 Documentation/devicetree/bindings/clock/qcom,gcc-sc8180x.yaml | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc-sc8180x.yaml b/Documentation/devicetree/bindings/clock/qcom,gcc-sc8180x.yaml
index 6c4846b34e4b..a1085ef4fd05 100644
--- a/Documentation/devicetree/bindings/clock/qcom,gcc-sc8180x.yaml
+++ b/Documentation/devicetree/bindings/clock/qcom,gcc-sc8180x.yaml
@@ -31,10 +31,15 @@ properties:
       - const: bi_tcxo_ao
       - const: sleep_clk
 
+  power-domains:
+    items:
+      - description: CX domain
+
 required:
   - compatible
   - clocks
   - clock-names
+  - power-domains
 
 allOf:
   - $ref: qcom,gcc.yaml#
@@ -44,6 +49,7 @@ unevaluatedProperties: false
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
+    #include <dt-bindings/power/qcom-rpmpd.h>
     clock-controller@100000 {
       compatible = "qcom,gcc-sc8180x";
       reg = <0x00100000 0x1f0000>;
@@ -51,6 +57,7 @@ examples:
                <&rpmhcc RPMH_CXO_CLK_A>,
                <&sleep_clk>;
       clock-names = "bi_tcxo", "bi_tcxo_ao", "sleep_clk";
+      power-domains = <&rpmhpd SC8180X_CX>;
       #clock-cells = <1>;
       #reset-cells = <1>;
       #power-domain-cells = <1>;

-- 
2.43.0


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 02/10] arm64: dts: qcom: sc8180x: Hook up VDD_CX as GCC parent domain
  2023-12-30  0:05 [PATCH 00/10] More 8180x dts fixes Konrad Dybcio
  2023-12-30  0:05 ` [PATCH 01/10] dt-bindings: clock: gcc-sc8180x: Add the missing CX power domain Konrad Dybcio
@ 2023-12-30  0:05 ` Konrad Dybcio
  2023-12-30  0:05 ` [PATCH 03/10] arm64: dts: qcom: sc8180x: Fix up big CPU idle state entry latency Konrad Dybcio
                   ` (8 subsequent siblings)
  10 siblings, 0 replies; 13+ messages in thread
From: Konrad Dybcio @ 2023-12-30  0:05 UTC (permalink / raw)
  To: Bjorn Andersson, Michael Turquette, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vinod Koul
  Cc: Marijn Suijten, linux-arm-msm, linux-clk, devicetree,
	linux-kernel, Konrad Dybcio

Most of GCC is powered by the CX rail. Describe that relationship to
let the performance state requests trickle up the chain.

Fixes: 8575f197b077 ("arm64: dts: qcom: Introduce the SC8180x platform")
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 arch/arm64/boot/dts/qcom/sc8180x.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/qcom/sc8180x.dtsi b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
index 0430d99091e3..91fd805f17a1 100644
--- a/arch/arm64/boot/dts/qcom/sc8180x.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
@@ -782,6 +782,7 @@ gcc: clock-controller@100000 {
 			clock-names = "bi_tcxo",
 				      "bi_tcxo_ao",
 				      "sleep_clk";
+			power-domains = <&rpmhpd SC8180X_CX>;
 		};
 
 		qupv3_id_0: geniqup@8c0000 {

-- 
2.43.0


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 03/10] arm64: dts: qcom: sc8180x: Fix up big CPU idle state entry latency
  2023-12-30  0:05 [PATCH 00/10] More 8180x dts fixes Konrad Dybcio
  2023-12-30  0:05 ` [PATCH 01/10] dt-bindings: clock: gcc-sc8180x: Add the missing CX power domain Konrad Dybcio
  2023-12-30  0:05 ` [PATCH 02/10] arm64: dts: qcom: sc8180x: Hook up VDD_CX as GCC parent domain Konrad Dybcio
@ 2023-12-30  0:05 ` Konrad Dybcio
  2023-12-30  0:05 ` [PATCH 04/10] arm64: dts: qcom: sc8180x: Add missing CPU off state Konrad Dybcio
                   ` (7 subsequent siblings)
  10 siblings, 0 replies; 13+ messages in thread
From: Konrad Dybcio @ 2023-12-30  0:05 UTC (permalink / raw)
  To: Bjorn Andersson, Michael Turquette, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vinod Koul
  Cc: Marijn Suijten, linux-arm-msm, linux-clk, devicetree,
	linux-kernel, Konrad Dybcio

The entry latency was oddly low.. Turns out somebody forgot about a
second '1'! Fix it.

Fixes: 8575f197b077 ("arm64: dts: qcom: Introduce the SC8180x platform")
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 arch/arm64/boot/dts/qcom/sc8180x.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sc8180x.dtsi b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
index 91fd805f17a1..b84fe5f3b41c 100644
--- a/arch/arm64/boot/dts/qcom/sc8180x.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
@@ -290,7 +290,7 @@ LITTLE_CPU_SLEEP_0: cpu-sleep-0-0 {
 			BIG_CPU_SLEEP_0: cpu-sleep-1-0 {
 				compatible = "arm,idle-state";
 				arm,psci-suspend-param = <0x40000004>;
-				entry-latency-us = <241>;
+				entry-latency-us = <2411>;
 				exit-latency-us = <1461>;
 				min-residency-us = <4488>;
 				local-timer-stop;

-- 
2.43.0


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 04/10] arm64: dts: qcom: sc8180x: Add missing CPU off state
  2023-12-30  0:05 [PATCH 00/10] More 8180x dts fixes Konrad Dybcio
                   ` (2 preceding siblings ...)
  2023-12-30  0:05 ` [PATCH 03/10] arm64: dts: qcom: sc8180x: Fix up big CPU idle state entry latency Konrad Dybcio
@ 2023-12-30  0:05 ` Konrad Dybcio
  2023-12-30  0:05 ` [PATCH 05/10] arm64: dts: qcom: sc8180x: Fix eDP PHY power-domains Konrad Dybcio
                   ` (6 subsequent siblings)
  10 siblings, 0 replies; 13+ messages in thread
From: Konrad Dybcio @ 2023-12-30  0:05 UTC (permalink / raw)
  To: Bjorn Andersson, Michael Turquette, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vinod Koul
  Cc: Marijn Suijten, linux-arm-msm, linux-clk, devicetree,
	linux-kernel, Konrad Dybcio

The CPUs can be powered off without pulling the plug from the rest of
the system. Describe the idle state responsible for this.

Fixes: 8575f197b077 ("arm64: dts: qcom: Introduce the SC8180x platform")
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 arch/arm64/boot/dts/qcom/sc8180x.dtsi | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sc8180x.dtsi b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
index b84fe5f3b41c..8849469d0aa1 100644
--- a/arch/arm64/boot/dts/qcom/sc8180x.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
@@ -298,7 +298,15 @@ BIG_CPU_SLEEP_0: cpu-sleep-1-0 {
 		};
 
 		domain-idle-states {
-			CLUSTER_SLEEP_0: cluster-sleep-0 {
+			CLUSTER_SLEEP_APSS_OFF: cluster-sleep-0 {
+				compatible = "domain-idle-state";
+				arm,psci-suspend-param = <0x41000044>;
+				entry-latency-us = <3300>;
+				exit-latency-us = <3300>;
+				min-residency-us = <6000>;
+			};
+
+			CLUSTER_SLEEP_AOSS_SLEEP: cluster-sleep-1 {
 				compatible = "domain-idle-state";
 				arm,psci-suspend-param = <0x4100a344>;
 				entry-latency-us = <3263>;
@@ -582,7 +590,7 @@ CPU_PD7: power-domain-cpu7 {
 
 		CLUSTER_PD: power-domain-cpu-cluster0 {
 			#power-domain-cells = <0>;
-			domain-idle-states = <&CLUSTER_SLEEP_0>;
+			domain-idle-states = <&CLUSTER_SLEEP_APSS_OFF &CLUSTER_SLEEP_AOSS_SLEEP>;
 		};
 	};
 

-- 
2.43.0


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 05/10] arm64: dts: qcom: sc8180x: Fix eDP PHY power-domains
  2023-12-30  0:05 [PATCH 00/10] More 8180x dts fixes Konrad Dybcio
                   ` (3 preceding siblings ...)
  2023-12-30  0:05 ` [PATCH 04/10] arm64: dts: qcom: sc8180x: Add missing CPU off state Konrad Dybcio
@ 2023-12-30  0:05 ` Konrad Dybcio
  2023-12-30  0:05 ` [PATCH 06/10] arm64: dts: qcom: sc8180x: Don't hold MDP core clock at FMAX Konrad Dybcio
                   ` (5 subsequent siblings)
  10 siblings, 0 replies; 13+ messages in thread
From: Konrad Dybcio @ 2023-12-30  0:05 UTC (permalink / raw)
  To: Bjorn Andersson, Michael Turquette, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vinod Koul
  Cc: Marijn Suijten, linux-arm-msm, linux-clk, devicetree,
	linux-kernel, Konrad Dybcio

The (e)DP PHYs are powered by the MX line, not through the MDSS GDSC.
Fix that up.

Fixes: 494dec9b6f54 ("arm64: dts: qcom: sc8180x: Add display and gpu nodes")
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 arch/arm64/boot/dts/qcom/sc8180x.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sc8180x.dtsi b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
index 8849469d0aa1..8f7f5b74cdb9 100644
--- a/arch/arm64/boot/dts/qcom/sc8180x.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
@@ -3193,7 +3193,7 @@ edp_phy: phy@aec2a00 {
 				 <&dispcc DISP_CC_MDSS_AHB_CLK>;
 			clock-names = "aux", "cfg_ahb";
 
-			power-domains = <&dispcc MDSS_GDSC>;
+			power-domains = <&rpmhpd SC8180X_MX>;
 
 			#clock-cells = <1>;
 			#phy-cells = <0>;

-- 
2.43.0


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 06/10] arm64: dts: qcom: sc8180x: Don't hold MDP core clock at FMAX
  2023-12-30  0:05 [PATCH 00/10] More 8180x dts fixes Konrad Dybcio
                   ` (4 preceding siblings ...)
  2023-12-30  0:05 ` [PATCH 05/10] arm64: dts: qcom: sc8180x: Fix eDP PHY power-domains Konrad Dybcio
@ 2023-12-30  0:05 ` Konrad Dybcio
  2023-12-30  0:05 ` [PATCH 07/10] arm64: dts: qcom: sc8180x: Require LOW_SVS vote for MMCX if DISPCC is on Konrad Dybcio
                   ` (4 subsequent siblings)
  10 siblings, 0 replies; 13+ messages in thread
From: Konrad Dybcio @ 2023-12-30  0:05 UTC (permalink / raw)
  To: Bjorn Andersson, Michael Turquette, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vinod Koul
  Cc: Marijn Suijten, linux-arm-msm, linux-clk, devicetree,
	linux-kernel, Konrad Dybcio

There's an OPP table to handle this, drop the permanent vote.

Fixes: 494dec9b6f54 ("arm64: dts: qcom: sc8180x: Add display and gpu nodes")
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 arch/arm64/boot/dts/qcom/sc8180x.dtsi | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sc8180x.dtsi b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
index 8f7f5b74cdb9..3bb9d25b1dec 100644
--- a/arch/arm64/boot/dts/qcom/sc8180x.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
@@ -2732,10 +2732,8 @@ mdss_mdp: mdp@ae01000 {
 					      "rot",
 					      "lut";
 
-				assigned-clocks = <&dispcc DISP_CC_MDSS_MDP_CLK>,
-						  <&dispcc DISP_CC_MDSS_VSYNC_CLK>;
-				assigned-clock-rates = <460000000>,
-						       <19200000>;
+				assigned-clocks = <&dispcc DISP_CC_MDSS_VSYNC_CLK>;
+				assigned-clock-rates = <19200000>;
 
 				operating-points-v2 = <&mdp_opp_table>;
 				power-domains = <&rpmhpd SC8180X_MMCX>;

-- 
2.43.0


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 07/10] arm64: dts: qcom: sc8180x: Require LOW_SVS vote for MMCX if DISPCC is on
  2023-12-30  0:05 [PATCH 00/10] More 8180x dts fixes Konrad Dybcio
                   ` (5 preceding siblings ...)
  2023-12-30  0:05 ` [PATCH 06/10] arm64: dts: qcom: sc8180x: Don't hold MDP core clock at FMAX Konrad Dybcio
@ 2023-12-30  0:05 ` Konrad Dybcio
  2023-12-30  0:05 ` [PATCH 08/10] arm64: dts: qcom: sc8180x: Add missing CPU<->MDP_CFG path Konrad Dybcio
                   ` (3 subsequent siblings)
  10 siblings, 0 replies; 13+ messages in thread
From: Konrad Dybcio @ 2023-12-30  0:05 UTC (permalink / raw)
  To: Bjorn Andersson, Michael Turquette, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vinod Koul
  Cc: Marijn Suijten, linux-arm-msm, linux-clk, devicetree,
	linux-kernel, Konrad Dybcio

To ensure the PLLs are getting enough power, cast a vote with DISPCC so
that MMCX is at least at LOW_SVS.

Fixes: 494dec9b6f54 ("arm64: dts: qcom: sc8180x: Add display and gpu nodes")
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 arch/arm64/boot/dts/qcom/sc8180x.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/qcom/sc8180x.dtsi b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
index 3bb9d25b1dec..97c139d0399d 100644
--- a/arch/arm64/boot/dts/qcom/sc8180x.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
@@ -3217,6 +3217,7 @@ dispcc: clock-controller@af00000 {
 				      "edp_phy_pll_link_clk",
 				      "edp_phy_pll_vco_div_clk";
 			power-domains = <&rpmhpd SC8180X_MMCX>;
+			required-opps = <&rpmhpd_opp_low_svs>;
 			#clock-cells = <1>;
 			#reset-cells = <1>;
 			#power-domain-cells = <1>;

-- 
2.43.0


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 08/10] arm64: dts: qcom: sc8180x: Add missing CPU<->MDP_CFG path
  2023-12-30  0:05 [PATCH 00/10] More 8180x dts fixes Konrad Dybcio
                   ` (6 preceding siblings ...)
  2023-12-30  0:05 ` [PATCH 07/10] arm64: dts: qcom: sc8180x: Require LOW_SVS vote for MMCX if DISPCC is on Konrad Dybcio
@ 2023-12-30  0:05 ` Konrad Dybcio
  2023-12-30  0:05 ` [PATCH 09/10] arm64: dts: qcom: sc8180x: Shrink aoss_qmp register space size Konrad Dybcio
                   ` (2 subsequent siblings)
  10 siblings, 0 replies; 13+ messages in thread
From: Konrad Dybcio @ 2023-12-30  0:05 UTC (permalink / raw)
  To: Bjorn Andersson, Michael Turquette, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vinod Koul
  Cc: Marijn Suijten, linux-arm-msm, linux-clk, devicetree,
	linux-kernel, Konrad Dybcio

To guarantee the required resources are enabled, describe the
interconnect path between the MDSS and the CPU.

Fixes: 494dec9b6f54 ("arm64: dts: qcom: sc8180x: Add display and gpu nodes")
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 arch/arm64/boot/dts/qcom/sc8180x.dtsi | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sc8180x.dtsi b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
index 97c139d0399d..960058624a2f 100644
--- a/arch/arm64/boot/dts/qcom/sc8180x.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
@@ -2701,9 +2701,15 @@ mdss: mdss@ae00000 {
 			interrupt-controller;
 			#interrupt-cells = <1>;
 
-			interconnects = <&mmss_noc MASTER_MDP_PORT0 0 &mc_virt SLAVE_EBI_CH0 0>,
-					<&mmss_noc MASTER_MDP_PORT1 0 &mc_virt SLAVE_EBI_CH0 0>;
-			interconnect-names = "mdp0-mem", "mdp1-mem";
+			interconnects = <&mmss_noc MASTER_MDP_PORT0 QCOM_ICC_TAG_ALWAYS
+					 &mc_virt SLAVE_EBI_CH0 QCOM_ICC_TAG_ALWAYS>,
+					<&mmss_noc MASTER_MDP_PORT1 QCOM_ICC_TAG_ALWAYS
+					 &mc_virt SLAVE_EBI_CH0 QCOM_ICC_TAG_ALWAYS>,
+					<&gem_noc MASTER_AMPSS_M0 QCOM_ICC_TAG_ALWAYS
+					 &config_noc SLAVE_DISPLAY_CFG QCOM_ICC_TAG_ALWAYS>;
+			interconnect-names = "mdp0-mem",
+					     "mdp1-mem",
+					     "cpu-cfg";
 
 			iommus = <&apps_smmu 0x800 0x420>;
 

-- 
2.43.0


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 09/10] arm64: dts: qcom: sc8180x: Shrink aoss_qmp register space size
  2023-12-30  0:05 [PATCH 00/10] More 8180x dts fixes Konrad Dybcio
                   ` (7 preceding siblings ...)
  2023-12-30  0:05 ` [PATCH 08/10] arm64: dts: qcom: sc8180x: Add missing CPU<->MDP_CFG path Konrad Dybcio
@ 2023-12-30  0:05 ` Konrad Dybcio
  2023-12-30  0:05 ` [PATCH 10/10] arm64: dts: qcom: sc8180x: Add RPMh sleep stats Konrad Dybcio
  2024-01-27 22:34 ` [PATCH 00/10] More 8180x dts fixes Bjorn Andersson
  10 siblings, 0 replies; 13+ messages in thread
From: Konrad Dybcio @ 2023-12-30  0:05 UTC (permalink / raw)
  To: Bjorn Andersson, Michael Turquette, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vinod Koul
  Cc: Marijn Suijten, linux-arm-msm, linux-clk, devicetree,
	linux-kernel, Konrad Dybcio

The AOSS_QMP region is overallocated, bleeding into space that's supposed
to be used by other peripherals. Fix it.

Fixes: 8575f197b077 ("arm64: dts: qcom: Introduce the SC8180x platform")
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 arch/arm64/boot/dts/qcom/sc8180x.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sc8180x.dtsi b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
index 960058624a2f..c0dd44f14674 100644
--- a/arch/arm64/boot/dts/qcom/sc8180x.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
@@ -3262,7 +3262,7 @@ tsens1: thermal-sensor@c265000 {
 
 		aoss_qmp: power-controller@c300000 {
 			compatible = "qcom,sc8180x-aoss-qmp", "qcom,aoss-qmp";
-			reg = <0x0 0x0c300000 0x0 0x100000>;
+			reg = <0x0 0x0c300000 0x0 0x400>;
 			interrupts = <GIC_SPI 389 IRQ_TYPE_EDGE_RISING>;
 			mboxes = <&apss_shared 0>;
 

-- 
2.43.0


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 10/10] arm64: dts: qcom: sc8180x: Add RPMh sleep stats
  2023-12-30  0:05 [PATCH 00/10] More 8180x dts fixes Konrad Dybcio
                   ` (8 preceding siblings ...)
  2023-12-30  0:05 ` [PATCH 09/10] arm64: dts: qcom: sc8180x: Shrink aoss_qmp register space size Konrad Dybcio
@ 2023-12-30  0:05 ` Konrad Dybcio
  2024-01-27 22:34 ` [PATCH 00/10] More 8180x dts fixes Bjorn Andersson
  10 siblings, 0 replies; 13+ messages in thread
From: Konrad Dybcio @ 2023-12-30  0:05 UTC (permalink / raw)
  To: Bjorn Andersson, Michael Turquette, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vinod Koul
  Cc: Marijn Suijten, linux-arm-msm, linux-clk, devicetree,
	linux-kernel, Konrad Dybcio

Add the sleep stats node to enable peeking at the power collapse reports
coming from the AOSS.

Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 arch/arm64/boot/dts/qcom/sc8180x.dtsi | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sc8180x.dtsi b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
index c0dd44f14674..013723be4bb7 100644
--- a/arch/arm64/boot/dts/qcom/sc8180x.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
@@ -3270,6 +3270,11 @@ aoss_qmp: power-controller@c300000 {
 			#power-domain-cells = <1>;
 		};
 
+		sram@c3f0000 {
+			compatible = "qcom,rpmh-stats";
+			reg = <0x0 0x0c3f0000 0x0 0x400>;
+		};
+
 		spmi_bus: spmi@c440000 {
 			compatible = "qcom,spmi-pmic-arb";
 			reg = <0x0 0x0c440000 0x0 0x0001100>,

-- 
2.43.0


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* Re: [PATCH 01/10] dt-bindings: clock: gcc-sc8180x: Add the missing CX power domain
  2023-12-30  0:05 ` [PATCH 01/10] dt-bindings: clock: gcc-sc8180x: Add the missing CX power domain Konrad Dybcio
@ 2024-01-04  8:24   ` Krzysztof Kozlowski
  0 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2024-01-04  8:24 UTC (permalink / raw)
  To: Konrad Dybcio, Bjorn Andersson, Michael Turquette, Stephen Boyd,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, Vinod Koul
  Cc: Marijn Suijten, linux-arm-msm, linux-clk, devicetree,
	linux-kernel

On 30/12/2023 01:05, Konrad Dybcio wrote:
> The GCC block is (mostly) powered by the VDD_CX rail. Allow specifying
> it in power-domains.
> 
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 00/10] More 8180x dts fixes
  2023-12-30  0:05 [PATCH 00/10] More 8180x dts fixes Konrad Dybcio
                   ` (9 preceding siblings ...)
  2023-12-30  0:05 ` [PATCH 10/10] arm64: dts: qcom: sc8180x: Add RPMh sleep stats Konrad Dybcio
@ 2024-01-27 22:34 ` Bjorn Andersson
  10 siblings, 0 replies; 13+ messages in thread
From: Bjorn Andersson @ 2024-01-27 22:34 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Vinod Koul, Konrad Dybcio
  Cc: Marijn Suijten, linux-arm-msm, linux-clk, devicetree,
	linux-kernel


On Sat, 30 Dec 2023 01:05:01 +0100, Konrad Dybcio wrote:
> SC8180X has got various random power plumbing issues, this series tries
> to address that, and introduces RPMh sleep stats.
> 
> 

Applied, thanks!

[01/10] dt-bindings: clock: gcc-sc8180x: Add the missing CX power domain
        commit: af5b3a595954bdf5a7fb92bd6594fccd241f77b2
[02/10] arm64: dts: qcom: sc8180x: Hook up VDD_CX as GCC parent domain
        commit: 3c58b96df110a80e78fa36ef928f1e6c375008e3
[03/10] arm64: dts: qcom: sc8180x: Fix up big CPU idle state entry latency
        commit: 266a3a92044b89c392b3e9cfcc328d4167c18294
[04/10] arm64: dts: qcom: sc8180x: Add missing CPU off state
        commit: 07b600dfdfea65d58dd80ea25becd8cff69bfafc
[05/10] arm64: dts: qcom: sc8180x: Fix eDP PHY power-domains
        commit: 24e98cb3d5e2c86565680e00008a794b4eac0040
[06/10] arm64: dts: qcom: sc8180x: Don't hold MDP core clock at FMAX
        commit: 309b5774f45aafd002efdb2656673542419abd6f
[07/10] arm64: dts: qcom: sc8180x: Require LOW_SVS vote for MMCX if DISPCC is on
        commit: 6d9fb9e4c473cdfd2adca019b46d8e482105cae7
[08/10] arm64: dts: qcom: sc8180x: Add missing CPU<->MDP_CFG path
        commit: f0cd5a0ebd419bd151ed79baf5f044da797521ac
[09/10] arm64: dts: qcom: sc8180x: Shrink aoss_qmp register space size
        commit: dcad0590d1ea4278a55c30dd2903611a96111601
[10/10] arm64: dts: qcom: sc8180x: Add RPMh sleep stats
        commit: 3168e86af8d1c346edf69b2114b3948ff03b2848

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2024-01-27 22:34 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-12-30  0:05 [PATCH 00/10] More 8180x dts fixes Konrad Dybcio
2023-12-30  0:05 ` [PATCH 01/10] dt-bindings: clock: gcc-sc8180x: Add the missing CX power domain Konrad Dybcio
2024-01-04  8:24   ` Krzysztof Kozlowski
2023-12-30  0:05 ` [PATCH 02/10] arm64: dts: qcom: sc8180x: Hook up VDD_CX as GCC parent domain Konrad Dybcio
2023-12-30  0:05 ` [PATCH 03/10] arm64: dts: qcom: sc8180x: Fix up big CPU idle state entry latency Konrad Dybcio
2023-12-30  0:05 ` [PATCH 04/10] arm64: dts: qcom: sc8180x: Add missing CPU off state Konrad Dybcio
2023-12-30  0:05 ` [PATCH 05/10] arm64: dts: qcom: sc8180x: Fix eDP PHY power-domains Konrad Dybcio
2023-12-30  0:05 ` [PATCH 06/10] arm64: dts: qcom: sc8180x: Don't hold MDP core clock at FMAX Konrad Dybcio
2023-12-30  0:05 ` [PATCH 07/10] arm64: dts: qcom: sc8180x: Require LOW_SVS vote for MMCX if DISPCC is on Konrad Dybcio
2023-12-30  0:05 ` [PATCH 08/10] arm64: dts: qcom: sc8180x: Add missing CPU<->MDP_CFG path Konrad Dybcio
2023-12-30  0:05 ` [PATCH 09/10] arm64: dts: qcom: sc8180x: Shrink aoss_qmp register space size Konrad Dybcio
2023-12-30  0:05 ` [PATCH 10/10] arm64: dts: qcom: sc8180x: Add RPMh sleep stats Konrad Dybcio
2024-01-27 22:34 ` [PATCH 00/10] More 8180x dts fixes Bjorn Andersson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).