Skip to content

Commit dbc64a5

Browse files
valpackettbarni2000
authored andcommitted
arm64: dts: qcom: msm8917: fix cpu idle states
Like on MSM8953, we need to use force-psci-domains on this SoCs. Also, we cannot just do cluster power collapse like that (it freezes): since MSM8917 is a single-cluster SoC, cluster PC is system PC. Signed-off-by: Val Packett <val@packett.cool>
1 parent 55e1817 commit dbc64a5

File tree

1 file changed

+22
-6
lines changed

1 file changed

+22
-6
lines changed

arch/arm64/boot/dts/qcom/msm8917.dtsi

Lines changed: 22 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -113,12 +113,27 @@
113113
};
114114

115115
domain-idle-states {
116-
cluster_sleep_0: cluster-sleep-0 {
116+
/* Seems strange, but downstream msm8917-pm.dtsi specifies
117+
GHDS as faster than retention, unlike for other SoCs */
118+
119+
cluster_gdhs: cluster-gdhs {
120+
compatible = "domain-idle-state";
121+
idle-state-name = "cluster-gdhs";
122+
arm,psci-suspend-param = <0x41000043>;
123+
entry-latency-us = <150>;
124+
exit-latency-us = <280>;
125+
min-residency-us = <560>;
126+
local-timer-stop;
127+
};
128+
129+
cluster_ret: cluster-retention {
117130
compatible = "domain-idle-state";
118-
arm,psci-suspend-param = <0x41000053>;
119-
entry-latency-us = <700>;
120-
exit-latency-us = <1000>;
121-
min-residency-us = <6500>;
131+
idle-state-name = "cluster-ret";
132+
arm,psci-suspend-param = <0x41000033>;
133+
entry-latency-us = <120>;
134+
exit-latency-us = <650>;
135+
min-residency-us = <1300>;
136+
local-timer-stop;
122137
};
123138
};
124139

@@ -185,10 +200,11 @@
185200
psci {
186201
compatible = "arm,psci-1.0";
187202
method = "smc";
203+
force-psci-domains;
188204

189205
cluster_pd: power-domain-cluster {
190206
#power-domain-cells = <0>;
191-
domain-idle-states = <&cluster_sleep_0>;
207+
domain-idle-states = <&cluster_gdhs>, <&cluster_ret>;
192208
};
193209

194210
cpu_pd0: power-domain-cpu0 {

0 commit comments

Comments
 (0)