Commit 26664c59 authored by Matthias Kaehlcke's avatar Matthias Kaehlcke Committed by Bjorn Andersson

arm64: dts: qcom: sc7180: Set 'polling-delay-passive' for thermal zones back to 250 ms

Commit 22337b91 ("arm64: dts: qcom: sc7180: Changed polling mode
in Thermal-zones node") sets both 'polling-delay' and
'polling-delay-passive' to zero with the rationale that TSENS interrupts
are enabled. A TSENS interrupt fires when the temperature of a thermal
zone reaches a trip point, which makes regular polling below the passive
trip point temperature unnecessary. However the situation is different
when passive cooling is active, regular polling is still needed to
trigger a periodic evaluation of the thermal zone by the thermal governor.

Change 'polling-delay-passive' back to the original value of 250 ms.
Commit 2315ae70 ("arm64: dts: qcom: sc7180: Add gpu cooling
support") recently changed the value for the GPU thermal zones from
zero to 100 ms, also set it to 250 ms for uniformity. If some zones
really need different values these can be changed in dedicated patches.
Reviewed-by: default avatarDouglas Anderson <dianders@chromium.org>
Fixes: 22337b91 ("arm64: dts: qcom: sc7180: Changed polling mode in Thermal-zones node")
Signed-off-by: default avatarMatthias Kaehlcke <mka@chromium.org>
Link: https://lore.kernel.org/r/20201111120334.1.Ifc04ea235c3c370e3b21ec3b4d5dead83cc403b4@changeidSigned-off-by: default avatarBjorn Andersson <bjorn.andersson@linaro.org>
parent 4e8692c2
......@@ -3522,7 +3522,7 @@ lpass_hm: clock-controller@63000000 {
thermal-zones {
cpu0-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 1>;
......@@ -3571,7 +3571,7 @@ map1 {
};
cpu1-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 2>;
......@@ -3620,7 +3620,7 @@ map1 {
};
cpu2-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 3>;
......@@ -3669,7 +3669,7 @@ map1 {
};
cpu3-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 4>;
......@@ -3718,7 +3718,7 @@ map1 {
};
cpu4-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 5>;
......@@ -3767,7 +3767,7 @@ map1 {
};
cpu5-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 6>;
......@@ -3816,7 +3816,7 @@ map1 {
};
cpu6-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 9>;
......@@ -3857,7 +3857,7 @@ map1 {
};
cpu7-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 10>;
......@@ -3898,7 +3898,7 @@ map1 {
};
cpu8-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 11>;
......@@ -3939,7 +3939,7 @@ map1 {
};
cpu9-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 12>;
......@@ -3980,7 +3980,7 @@ map1 {
};
aoss0-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 0>;
......@@ -4001,7 +4001,7 @@ aoss0_crit: aoss0_crit {
};
cpuss0-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 7>;
......@@ -4021,7 +4021,7 @@ cpuss0_crit: cluster0_crit {
};
cpuss1-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 8>;
......@@ -4041,7 +4041,7 @@ cpuss1_crit: cluster0_crit {
};
gpuss0-thermal {
polling-delay-passive = <100>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 13>;
......@@ -4069,7 +4069,7 @@ map0 {
};
gpuss1-thermal {
polling-delay-passive = <100>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens0 14>;
......@@ -4097,7 +4097,7 @@ map0 {
};
aoss1-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens1 0>;
......@@ -4118,7 +4118,7 @@ aoss1_crit: aoss1_crit {
};
cwlan-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens1 1>;
......@@ -4139,7 +4139,7 @@ cwlan_crit: cwlan_crit {
};
audio-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens1 2>;
......@@ -4160,7 +4160,7 @@ audio_crit: audio_crit {
};
ddr-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens1 3>;
......@@ -4181,7 +4181,7 @@ ddr_crit: ddr_crit {
};
q6-hvx-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens1 4>;
......@@ -4202,7 +4202,7 @@ q6_hvx_crit: q6_hvx_crit {
};
camera-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens1 5>;
......@@ -4223,7 +4223,7 @@ camera_crit: camera_crit {
};
mdm-core-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens1 6>;
......@@ -4244,7 +4244,7 @@ mdm_crit: mdm_crit {
};
mdm-dsp-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens1 7>;
......@@ -4265,7 +4265,7 @@ mdm_dsp_crit: mdm_dsp_crit {
};
npu-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens1 8>;
......@@ -4286,7 +4286,7 @@ npu_crit: npu_crit {
};
video-thermal {
polling-delay-passive = <0>;
polling-delay-passive = <250>;
polling-delay = <0>;
thermal-sensors = <&tsens1 9>;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment