* [PATCH 01/31] arm64: dts: qcom: ipq6018-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 02/31] arm64: dts: qcom: ipq8074-*: " Konrad Dybcio
` (31 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/ipq6018.dtsi | 14 --------------
1 file changed, 14 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
index 4e29adea570a..cd0ee133964d 100644
--- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
+++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
@@ -912,8 +912,6 @@ pcie0: pcie@20000000 {
thermal-zones {
nss-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 4>;
trips {
@@ -926,8 +924,6 @@ nss-top-critical {
};
nss-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 5>;
trips {
@@ -940,8 +936,6 @@ nss-critical {
};
wcss-phya0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 7>;
trips {
@@ -954,8 +948,6 @@ wcss-phya0-critical {
};
wcss-phya1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 8>;
trips {
@@ -968,8 +960,6 @@ wcss-phya1-critical {
};
cpu-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 13>;
trips {
@@ -998,8 +988,6 @@ map0 {
};
lpass-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 14>;
trips {
@@ -1012,8 +1000,6 @@ lpass-critical {
};
ddrss-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 15>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 02/31] arm64: dts: qcom: ipq8074-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
2024-03-19 16:13 ` [PATCH 01/31] arm64: dts: qcom: ipq6018-*: Remove thermal zone polling delays Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 03/31] arm64: dts: qcom: ipq9574-*: " Konrad Dybcio
` (30 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/ipq8074.dtsi | 36 -----------------------------------
1 file changed, 36 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/ipq8074.dtsi b/arch/arm64/boot/dts/qcom/ipq8074.dtsi
index e5b89753aa5c..75641175dbfb 100644
--- a/arch/arm64/boot/dts/qcom/ipq8074.dtsi
+++ b/arch/arm64/boot/dts/qcom/ipq8074.dtsi
@@ -942,9 +942,6 @@ timer {
thermal-zones {
nss-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 4>;
trips {
@@ -957,9 +954,6 @@ nss-top-crit {
};
nss0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 5>;
trips {
@@ -972,9 +966,6 @@ nss-0-crit {
};
nss1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 6>;
trips {
@@ -987,9 +978,6 @@ nss-1-crit {
};
wcss-phya0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 7>;
trips {
@@ -1002,9 +990,6 @@ wcss-phya0-crit {
};
wcss-phya1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 8>;
trips {
@@ -1017,9 +1002,6 @@ wcss-phya1-crit {
};
cpu0_thermal: cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 9>;
trips {
@@ -1032,9 +1014,6 @@ cpu0-crit {
};
cpu1_thermal: cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 10>;
trips {
@@ -1047,9 +1026,6 @@ cpu1-crit {
};
cpu2_thermal: cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 11>;
trips {
@@ -1062,9 +1038,6 @@ cpu2-crit {
};
cpu3_thermal: cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 12>;
trips {
@@ -1077,9 +1050,6 @@ cpu3-crit {
};
cluster_thermal: cluster-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 13>;
trips {
@@ -1092,9 +1062,6 @@ cluster-crit {
};
wcss-phyb0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 14>;
trips {
@@ -1107,9 +1074,6 @@ wcss-phyb0-crit {
};
wcss-phyb1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 15>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 03/31] arm64: dts: qcom: ipq9574-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
2024-03-19 16:13 ` [PATCH 01/31] arm64: dts: qcom: ipq6018-*: Remove thermal zone polling delays Konrad Dybcio
2024-03-19 16:13 ` [PATCH 02/31] arm64: dts: qcom: ipq8074-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-20 10:03 ` Varadarajan Narayanan
2024-03-19 16:13 ` [PATCH 04/31] arm64: dts: qcom: msm8916-*: " Konrad Dybcio
` (29 subsequent siblings)
32 siblings, 1 reply; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/ipq9574.dtsi | 26 --------------------------
1 file changed, 26 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/ipq9574.dtsi b/arch/arm64/boot/dts/qcom/ipq9574.dtsi
index 7f2e5cbf3bbb..98c5623f4391 100644
--- a/arch/arm64/boot/dts/qcom/ipq9574.dtsi
+++ b/arch/arm64/boot/dts/qcom/ipq9574.dtsi
@@ -749,8 +749,6 @@ frame@b128000 {
thermal-zones {
nss-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens 3>;
trips {
@@ -763,8 +761,6 @@ nss-top-critical {
};
ubi-0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens 4>;
trips {
@@ -777,8 +773,6 @@ ubi_0-critical {
};
ubi-1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens 5>;
trips {
@@ -791,8 +785,6 @@ ubi_1-critical {
};
ubi-2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens 6>;
trips {
@@ -805,8 +797,6 @@ ubi_2-critical {
};
ubi-3-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens 7>;
trips {
@@ -819,8 +809,6 @@ ubi_3-critical {
};
cpuss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens 8>;
trips {
@@ -833,8 +821,6 @@ cpu-critical {
};
cpuss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens 9>;
trips {
@@ -847,8 +833,6 @@ cpu-critical {
};
cpu0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens 10>;
trips {
@@ -877,8 +861,6 @@ map0 {
};
cpu1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens 11>;
trips {
@@ -907,8 +889,6 @@ map0 {
};
cpu2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens 12>;
trips {
@@ -937,8 +917,6 @@ map0 {
};
cpu3-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens 13>;
trips {
@@ -967,8 +945,6 @@ map0 {
};
wcss-phyb-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens 14>;
trips {
@@ -981,8 +957,6 @@ wcss_phyb-critical {
};
top-glue-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens 15>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* Re: [PATCH 03/31] arm64: dts: qcom: ipq9574-*: Remove thermal zone polling delays
2024-03-19 16:13 ` [PATCH 03/31] arm64: dts: qcom: ipq9574-*: " Konrad Dybcio
@ 2024-03-20 10:03 ` Varadarajan Narayanan
0 siblings, 0 replies; 35+ messages in thread
From: Varadarajan Narayanan @ 2024-03-20 10:03 UTC (permalink / raw)
To: Konrad Dybcio
Cc: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel
On Tue, Mar 19, 2024 at 05:13:33PM +0100, Konrad Dybcio wrote:
> All of the thermal zone suppliers are interrupt-driven, remove the
> bogus and unnecessary polling that only wastes CPU time.
>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/ipq9574.dtsi | 26 --------------------------
> 1 file changed, 26 deletions(-)
Reviewed-by: Varadarajan Narayanan <quic_varada@quicinc.com>
Thanks
Varada
>
> diff --git a/arch/arm64/boot/dts/qcom/ipq9574.dtsi b/arch/arm64/boot/dts/qcom/ipq9574.dtsi
> index 7f2e5cbf3bbb..98c5623f4391 100644
> --- a/arch/arm64/boot/dts/qcom/ipq9574.dtsi
> +++ b/arch/arm64/boot/dts/qcom/ipq9574.dtsi
> @@ -749,8 +749,6 @@ frame@b128000 {
>
> thermal-zones {
> nss-top-thermal {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> thermal-sensors = <&tsens 3>;
>
> trips {
> @@ -763,8 +761,6 @@ nss-top-critical {
> };
>
> ubi-0-thermal {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> thermal-sensors = <&tsens 4>;
>
> trips {
> @@ -777,8 +773,6 @@ ubi_0-critical {
> };
>
> ubi-1-thermal {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> thermal-sensors = <&tsens 5>;
>
> trips {
> @@ -791,8 +785,6 @@ ubi_1-critical {
> };
>
> ubi-2-thermal {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> thermal-sensors = <&tsens 6>;
>
> trips {
> @@ -805,8 +797,6 @@ ubi_2-critical {
> };
>
> ubi-3-thermal {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> thermal-sensors = <&tsens 7>;
>
> trips {
> @@ -819,8 +809,6 @@ ubi_3-critical {
> };
>
> cpuss0-thermal {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> thermal-sensors = <&tsens 8>;
>
> trips {
> @@ -833,8 +821,6 @@ cpu-critical {
> };
>
> cpuss1-thermal {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> thermal-sensors = <&tsens 9>;
>
> trips {
> @@ -847,8 +833,6 @@ cpu-critical {
> };
>
> cpu0-thermal {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> thermal-sensors = <&tsens 10>;
>
> trips {
> @@ -877,8 +861,6 @@ map0 {
> };
>
> cpu1-thermal {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> thermal-sensors = <&tsens 11>;
>
> trips {
> @@ -907,8 +889,6 @@ map0 {
> };
>
> cpu2-thermal {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> thermal-sensors = <&tsens 12>;
>
> trips {
> @@ -937,8 +917,6 @@ map0 {
> };
>
> cpu3-thermal {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> thermal-sensors = <&tsens 13>;
>
> trips {
> @@ -967,8 +945,6 @@ map0 {
> };
>
> wcss-phyb-thermal {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> thermal-sensors = <&tsens 14>;
>
> trips {
> @@ -981,8 +957,6 @@ wcss_phyb-critical {
> };
>
> top-glue-thermal {
> - polling-delay-passive = <0>;
> - polling-delay = <0>;
> thermal-sensors = <&tsens 15>;
>
> trips {
>
> --
> 2.40.1
>
>
^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH 04/31] arm64: dts: qcom: msm8916-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (2 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 03/31] arm64: dts: qcom: ipq9574-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 05/31] arm64: dts: qcom: msm8939-*: " Konrad Dybcio
` (28 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8916.dtsi | 15 ---------------
1 file changed, 15 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi
index cedff4166bfb..e95800f8258f 100644
--- a/arch/arm64/boot/dts/qcom/msm8916.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi
@@ -2625,9 +2625,6 @@ cpu3_saw: power-manager@b0b9000 {
thermal-zones {
cpu0-1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 5>;
trips {
@@ -2655,9 +2652,6 @@ map0 {
};
cpu2-3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 4>;
trips {
@@ -2685,9 +2679,6 @@ map0 {
};
gpu-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 2>;
cooling-maps {
@@ -2712,9 +2703,6 @@ gpu_crit: gpu-crit {
};
camera-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 1>;
trips {
@@ -2727,9 +2715,6 @@ cam_alert0: trip-point0 {
};
modem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 0>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 05/31] arm64: dts: qcom: msm8939-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (3 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 04/31] arm64: dts: qcom: msm8916-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 06/31] arm64: dts: qcom: msm8953-*: " Konrad Dybcio
` (27 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8939.dtsi | 27 ---------------------------
1 file changed, 27 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8939.dtsi b/arch/arm64/boot/dts/qcom/msm8939.dtsi
index dd45975682b2..43c0335b2be2 100644
--- a/arch/arm64/boot/dts/qcom/msm8939.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8939.dtsi
@@ -2298,9 +2298,6 @@ apcs2: mailbox@b1d1000 {
thermal_zones: thermal-zones {
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 5>;
trips {
@@ -2329,9 +2326,6 @@ map0 {
};
cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 6>;
trips {
@@ -2360,9 +2354,6 @@ map0 {
};
cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 7>;
trips {
@@ -2391,9 +2382,6 @@ map0 {
};
cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 8>;
trips {
@@ -2422,9 +2410,6 @@ map0 {
};
cpu4567-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 9>;
trips {
@@ -2453,9 +2438,6 @@ map0 {
};
gpu-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 3>;
cooling-maps {
@@ -2481,9 +2463,6 @@ gpu_crit: gpu-crit {
};
modem1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 0>;
trips {
@@ -2496,9 +2475,6 @@ modem1_alert0: trip-point0 {
};
modem2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 2>;
trips {
@@ -2511,9 +2487,6 @@ modem2_alert0: trip-point0 {
};
camera-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 1>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 06/31] arm64: dts: qcom: msm8953-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (4 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 05/31] arm64: dts: qcom: msm8939-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 07/31] arm64: dts: qcom: msm8976-*: " Konrad Dybcio
` (26 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8953.dtsi | 18 ------------------
arch/arm64/boot/dts/qcom/pm8953.dtsi | 3 ---
2 files changed, 21 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8953.dtsi b/arch/arm64/boot/dts/qcom/msm8953.dtsi
index f1011bb641c6..73e52cc54236 100644
--- a/arch/arm64/boot/dts/qcom/msm8953.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8953.dtsi
@@ -1953,8 +1953,6 @@ q6routing: routing {
thermal-zones {
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens0 9>;
trips {
cpu0_alert: trip-point0 {
@@ -1976,8 +1974,6 @@ map0 {
};
};
cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens0 10>;
trips {
cpu1_alert: trip-point0 {
@@ -1999,8 +1995,6 @@ map0 {
};
};
cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens0 11>;
trips {
cpu2_alert: trip-point0 {
@@ -2022,8 +2016,6 @@ map0 {
};
};
cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens0 12>;
trips {
cpu3_alert: trip-point0 {
@@ -2045,8 +2037,6 @@ map0 {
};
};
cpu4-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens0 4>;
trips {
cpu4_alert: trip-point0 {
@@ -2068,8 +2058,6 @@ map0 {
};
};
cpu5-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens0 5>;
trips {
cpu5_alert: trip-point0 {
@@ -2091,8 +2079,6 @@ map0 {
};
};
cpu6-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens0 6>;
trips {
cpu6_alert: trip-point0 {
@@ -2114,8 +2100,6 @@ map0 {
};
};
cpu7-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens0 7>;
trips {
cpu7_alert: trip-point0 {
@@ -2138,8 +2122,6 @@ map0 {
};
gpu-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens0 15>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm8953.dtsi b/arch/arm64/boot/dts/qcom/pm8953.dtsi
index 1067e141be6c..64258505f9ba 100644
--- a/arch/arm64/boot/dts/qcom/pm8953.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8953.dtsi
@@ -9,9 +9,6 @@
/ {
thermal-zones {
pm8953-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8953_temp>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 07/31] arm64: dts: qcom: msm8976-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (5 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 06/31] arm64: dts: qcom: msm8953-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 08/31] arm64: dts: qcom: msm8996-*: " Konrad Dybcio
` (25 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8976.dtsi | 26 --------------------------
1 file changed, 26 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8976.dtsi b/arch/arm64/boot/dts/qcom/msm8976.dtsi
index d2bb1ada361a..09cb8cbceb6e 100644
--- a/arch/arm64/boot/dts/qcom/msm8976.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8976.dtsi
@@ -1123,9 +1123,6 @@ pil-reloc@94c {
thermal-zones {
aoss0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 0>;
trips {
@@ -1138,9 +1135,6 @@ aoss0_alert0: trip-point0 {
};
modem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 1>;
trips {
modem_alert0: trip-point0 {
@@ -1152,9 +1146,6 @@ modem_alert0: trip-point0 {
};
qdsp-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 2>;
trips {
qdsp_alert0: trip-point0 {
@@ -1166,9 +1157,6 @@ qdsp_alert0: trip-point0 {
};
cam-isp-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 3>;
trips {
cam_isp_alert0: trip-point0 {
@@ -1180,8 +1168,6 @@ cam_isp_alert0: trip-point0 {
};
cpu4-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 4>;
trips {
@@ -1204,8 +1190,6 @@ cpu4_crit: cpu-crit {
};
cpu5-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 5>;
trips {
@@ -1228,8 +1212,6 @@ cpu5_crit: cpu-crit {
};
cpu6-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 6>;
trips {
@@ -1252,8 +1234,6 @@ cpu6_crit: cpu-crit {
};
cpu7-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 7>;
trips {
@@ -1276,8 +1256,6 @@ cpu7_crit: cpu-crit {
};
big-l2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 8>;
trips {
@@ -1300,8 +1278,6 @@ l2_crit: l2-crit {
};
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 9>;
trips {
@@ -1324,8 +1300,6 @@ cpu0_crit: cpu-crit {
};
gpu-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
thermal-sensors = <&tsens 10>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 08/31] arm64: dts: qcom: msm8996-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (6 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 07/31] arm64: dts: qcom: msm8976-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 09/31] arm64: dts: qcom: msm8998-*: " Konrad Dybcio
` (24 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8996.dtsi | 42 -----------------------------------
arch/arm64/boot/dts/qcom/pm8994.dtsi | 3 ---
2 files changed, 45 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
index 1601e46549e7..f9bdf09a91d1 100644
--- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
@@ -3623,9 +3623,6 @@ sound: sound {
thermal-zones {
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 3>;
trips {
@@ -3644,9 +3641,6 @@ cpu0_crit: cpu-crit {
};
cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 5>;
trips {
@@ -3665,9 +3659,6 @@ cpu1_crit: cpu-crit {
};
cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -3686,9 +3677,6 @@ cpu2_crit: cpu-crit {
};
cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 10>;
trips {
@@ -3707,9 +3695,6 @@ cpu3_crit: cpu-crit {
};
gpu-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 6>;
trips {
@@ -3729,9 +3714,6 @@ map0 {
};
gpu-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 7>;
trips {
@@ -3751,9 +3733,6 @@ map0 {
};
m4m-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 1>;
trips {
@@ -3766,9 +3745,6 @@ m4m_alert0: trip-point0 {
};
l3-or-venus-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 2>;
trips {
@@ -3781,9 +3757,6 @@ l3_or_venus_alert0: trip-point0 {
};
cluster0-l2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -3796,9 +3769,6 @@ cluster0_l2_alert0: trip-point0 {
};
cluster1-l2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 12>;
trips {
@@ -3811,9 +3781,6 @@ cluster1_l2_alert0: trip-point0 {
};
camera-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 1>;
trips {
@@ -3826,9 +3793,6 @@ camera_alert0: trip-point0 {
};
q6-dsp-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 2>;
trips {
@@ -3841,9 +3805,6 @@ q6_dsp_alert0: trip-point0 {
};
mem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 3>;
trips {
@@ -3856,9 +3817,6 @@ mem_alert0: trip-point0 {
};
modemtx-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 4>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm8994.dtsi b/arch/arm64/boot/dts/qcom/pm8994.dtsi
index d44a95caf04a..68629dc1bf69 100644
--- a/arch/arm64/boot/dts/qcom/pm8994.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8994.dtsi
@@ -7,9 +7,6 @@
/ {
thermal-zones {
pm8994-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&pm8994_temp>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 09/31] arm64: dts: qcom: msm8998-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (7 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 08/31] arm64: dts: qcom: msm8996-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 10/31] arm64: dts: qcom: pm7550ba: " Konrad Dybcio
` (23 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8998.dtsi | 57 -----------------------------------
arch/arm64/boot/dts/qcom/pm8998.dtsi | 3 --
2 files changed, 60 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8998.dtsi b/arch/arm64/boot/dts/qcom/msm8998.dtsi
index 4dfe2d09ac28..8f5a92f1b863 100644
--- a/arch/arm64/boot/dts/qcom/msm8998.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8998.dtsi
@@ -487,9 +487,6 @@ slpi_smp2p_in: slave-kernel {
thermal-zones {
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 1>;
trips {
@@ -508,9 +505,6 @@ cpu0_crit: cpu-crit {
};
cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 2>;
trips {
@@ -529,9 +523,6 @@ cpu1_crit: cpu-crit {
};
cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 3>;
trips {
@@ -550,9 +541,6 @@ cpu2_crit: cpu-crit {
};
cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 4>;
trips {
@@ -571,9 +559,6 @@ cpu3_crit: cpu-crit {
};
cpu4-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -592,9 +577,6 @@ cpu4_crit: cpu-crit {
};
cpu5-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -613,9 +595,6 @@ cpu5_crit: cpu-crit {
};
cpu6-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 9>;
trips {
@@ -634,9 +613,6 @@ cpu6_crit: cpu-crit {
};
cpu7-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 10>;
trips {
@@ -655,9 +631,6 @@ cpu7_crit: cpu-crit {
};
gpu-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 12>;
trips {
@@ -670,9 +643,6 @@ gpu1_alert0: trip-point0 {
};
gpu-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 13>;
trips {
@@ -685,9 +655,6 @@ gpu2_alert0: trip-point0 {
};
clust0-mhm-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 5>;
trips {
@@ -700,9 +667,6 @@ cluster0_mhm_alert0: trip-point0 {
};
clust1-mhm-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 6>;
trips {
@@ -715,9 +679,6 @@ cluster1_mhm_alert0: trip-point0 {
};
cluster1-l2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 11>;
trips {
@@ -730,9 +691,6 @@ cluster1_l2_alert0: trip-point0 {
};
modem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 1>;
trips {
@@ -745,9 +703,6 @@ modem_alert0: trip-point0 {
};
mem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 2>;
trips {
@@ -760,9 +715,6 @@ mem_alert0: trip-point0 {
};
wlan-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 3>;
trips {
@@ -775,9 +727,6 @@ wlan_alert0: trip-point0 {
};
q6-dsp-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 4>;
trips {
@@ -790,9 +739,6 @@ q6_dsp_alert0: trip-point0 {
};
camera-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 5>;
trips {
@@ -805,9 +751,6 @@ camera_alert0: trip-point0 {
};
multimedia-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 6>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm8998.dtsi b/arch/arm64/boot/dts/qcom/pm8998.dtsi
index 3f82715392c6..157feef07b03 100644
--- a/arch/arm64/boot/dts/qcom/pm8998.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8998.dtsi
@@ -10,9 +10,6 @@
/ {
thermal-zones {
pm8998-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&pm8998_temp>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 10/31] arm64: dts: qcom: pm7550ba: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (8 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 09/31] arm64: dts: qcom: msm8998-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 11/31] arm64: dts: qcom: pms405: " Konrad Dybcio
` (22 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/pm7550ba.dtsi | 3 ---
1 file changed, 3 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/pm7550ba.dtsi b/arch/arm64/boot/dts/qcom/pm7550ba.dtsi
index 8b00ece987d1..05fcd8591367 100644
--- a/arch/arm64/boot/dts/qcom/pm7550ba.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm7550ba.dtsi
@@ -9,9 +9,6 @@
/ {
thermal-zones {
pm7550ba-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm7550ba_temp_alarm>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 11/31] arm64: dts: qcom: pms405: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (9 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 10/31] arm64: dts: qcom: pm7550ba: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 12/31] arm64: dts: qcom: pmx75: " Konrad Dybcio
` (21 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/pms405.dtsi | 3 ---
1 file changed, 3 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/pms405.dtsi b/arch/arm64/boot/dts/qcom/pms405.dtsi
index 461ad97032f7..afa3bb469870 100644
--- a/arch/arm64/boot/dts/qcom/pms405.dtsi
+++ b/arch/arm64/boot/dts/qcom/pms405.dtsi
@@ -11,9 +11,6 @@
/ {
thermal-zones {
pms405-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&pms405_temp>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 12/31] arm64: dts: qcom: pmx75: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (10 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 11/31] arm64: dts: qcom: pms405: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 13/31] arm64: dts: qcom: qcm2290-*: " Konrad Dybcio
` (20 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/pmx75.dtsi | 3 ---
1 file changed, 3 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/pmx75.dtsi b/arch/arm64/boot/dts/qcom/pmx75.dtsi
index 373e45f63dff..84ea94e3b394 100644
--- a/arch/arm64/boot/dts/qcom/pmx75.dtsi
+++ b/arch/arm64/boot/dts/qcom/pmx75.dtsi
@@ -9,9 +9,6 @@
/ {
thermal-zones {
pmx75-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pmx75_temp_alarm>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 13/31] arm64: dts: qcom: qcm2290-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (11 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 12/31] arm64: dts: qcom: pmx75: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 14/31] arm64: dts: qcom: qcs404-*: " Konrad Dybcio
` (19 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/qcm2290.dtsi | 30 ------------------------------
1 file changed, 30 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/qcm2290.dtsi b/arch/arm64/boot/dts/qcom/qcm2290.dtsi
index 89beac833d43..c439ad5a16ae 100644
--- a/arch/arm64/boot/dts/qcom/qcm2290.dtsi
+++ b/arch/arm64/boot/dts/qcom/qcm2290.dtsi
@@ -1870,9 +1870,6 @@ cpufreq_hw: cpufreq@f521000 {
thermal-zones {
mapss-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 0>;
trips {
@@ -1897,9 +1894,6 @@ mapss_crit: mapss-crit {
};
video-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 1>;
trips {
@@ -1924,9 +1918,6 @@ video_crit: video-crit {
};
wlan-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 2>;
trips {
@@ -1951,9 +1942,6 @@ wlan_crit: wlan-crit {
};
cpuss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 3>;
trips {
@@ -1978,9 +1966,6 @@ cpuss0_crit: cpuss0-crit {
};
cpuss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 4>;
trips {
@@ -2005,9 +1990,6 @@ cpuss1_crit: cpuss1-crit {
};
mdm0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 5>;
trips {
@@ -2032,9 +2014,6 @@ mdm0_crit: mdm0-crit {
};
mdm1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 6>;
trips {
@@ -2059,9 +2038,6 @@ mdm1_crit: mdm1-crit {
};
gpu-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -2086,9 +2062,6 @@ gpu_crit: gpu-crit {
};
hm-center-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -2113,9 +2086,6 @@ hm_center_crit: hm-center-crit {
};
camera-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 9>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 14/31] arm64: dts: qcom: qcs404-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (12 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 13/31] arm64: dts: qcom: qcm2290-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 15/31] arm64: dts: qcom: sa8775p-*: " Konrad Dybcio
` (18 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/qcs404.dtsi | 30 ------------------------------
1 file changed, 30 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/qcs404.dtsi b/arch/arm64/boot/dts/qcom/qcs404.dtsi
index a05d0234f7fc..ad109b38199e 100644
--- a/arch/arm64/boot/dts/qcom/qcs404.dtsi
+++ b/arch/arm64/boot/dts/qcom/qcs404.dtsi
@@ -1589,9 +1589,6 @@ wcss_smp2p_in: slave-kernel {
thermal-zones {
aoss-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 0>;
trips {
@@ -1604,9 +1601,6 @@ aoss_alert0: trip-point0 {
};
q6-hvx-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 1>;
trips {
@@ -1619,9 +1613,6 @@ q6_hvx_alert0: trip-point0 {
};
lpass-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 2>;
trips {
@@ -1634,9 +1625,6 @@ lpass_alert0: trip-point0 {
};
wlan-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 3>;
trips {
@@ -1649,9 +1637,6 @@ wlan_alert0: trip-point0 {
};
cluster-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 4>;
trips {
@@ -1683,9 +1668,6 @@ map0 {
};
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 5>;
trips {
@@ -1717,9 +1699,6 @@ map0 {
};
cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 6>;
trips {
@@ -1751,9 +1730,6 @@ map0 {
};
cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 7>;
trips {
@@ -1785,9 +1761,6 @@ map0 {
};
cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 8>;
trips {
@@ -1819,9 +1792,6 @@ map0 {
};
gpu-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 9>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 15/31] arm64: dts: qcom: sa8775p-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (13 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 14/31] arm64: dts: qcom: qcs404-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 16/31] arm64: dts: qcom: sc7180-*: " Konrad Dybcio
` (17 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi | 8 --
arch/arm64/boot/dts/qcom/sa8775p.dtsi | 150 ----------------------------
2 files changed, 158 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi b/arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi
index eaa43f022a65..f23a99729d9b 100644
--- a/arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi
+++ b/arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi
@@ -9,8 +9,6 @@
/ {
thermal-zones {
pmm8654au_0_thermal: pm8775-0-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
thermal-sensors = <&pmm8654au_0_temp_alarm>;
trips {
@@ -29,8 +27,6 @@ trip1 {
};
pmm8654au_1_thermal: pm8775-1-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
thermal-sensors = <&pmm8654au_1_temp_alarm>;
trips {
@@ -49,8 +45,6 @@ trip1 {
};
pmm8654au_2_thermal: pm8775-2-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
thermal-sensors = <&pmm8654au_2_temp_alarm>;
trips {
@@ -69,8 +63,6 @@ trip1 {
};
pmm8654au_3_thermal: pm8775-3-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
thermal-sensors = <&pmm8654au_3_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sa8775p.dtsi b/arch/arm64/boot/dts/qcom/sa8775p.dtsi
index 231cea1f0fa8..45e6443c9dc9 100644
--- a/arch/arm64/boot/dts/qcom/sa8775p.dtsi
+++ b/arch/arm64/boot/dts/qcom/sa8775p.dtsi
@@ -2550,9 +2550,6 @@ ethernet0: ethernet@23040000 {
thermal-zones {
aoss-0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 0>;
trips {
@@ -2571,9 +2568,6 @@ trip-point1 {
};
cpu-0-0-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 1>;
trips {
@@ -2592,9 +2586,6 @@ trip-point1 {
};
cpu-0-1-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 2>;
trips {
@@ -2613,9 +2604,6 @@ trip-point1 {
};
cpu-0-2-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 3>;
trips {
@@ -2634,9 +2622,6 @@ trip-point1 {
};
cpu-0-3-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 4>;
trips {
@@ -2655,9 +2640,6 @@ trip-point1 {
};
gpuss-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 5>;
trips {
@@ -2676,9 +2658,6 @@ trip-point1 {
};
gpuss-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 6>;
trips {
@@ -2697,9 +2676,6 @@ trip-point1 {
};
gpuss-2-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -2718,9 +2694,6 @@ trip-point1 {
};
audio-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -2739,9 +2712,6 @@ trip-point1 {
};
camss-0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 9>;
trips {
@@ -2760,9 +2730,6 @@ trip-point1 {
};
pcie-0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 10>;
trips {
@@ -2781,9 +2748,6 @@ trip-point1 {
};
cpuss-0-0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 11>;
trips {
@@ -2802,9 +2766,6 @@ trip-point1 {
};
aoss-1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 0>;
trips {
@@ -2823,9 +2784,6 @@ trip-point1 {
};
cpu-0-0-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 1>;
trips {
@@ -2844,9 +2802,6 @@ trip-point1 {
};
cpu-0-1-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 2>;
trips {
@@ -2865,9 +2820,6 @@ trip-point1 {
};
cpu-0-2-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 3>;
trips {
@@ -2886,9 +2838,6 @@ trip-point1 {
};
cpu-0-3-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 4>;
trips {
@@ -2907,9 +2856,6 @@ trip-point1 {
};
gpuss-3-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 5>;
trips {
@@ -2928,9 +2874,6 @@ trip-point1 {
};
gpuss-4-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 6>;
trips {
@@ -2949,9 +2892,6 @@ trip-point1 {
};
gpuss-5-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 7>;
trips {
@@ -2970,9 +2910,6 @@ trip-point1 {
};
video-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 8>;
trips {
@@ -2991,9 +2928,6 @@ trip-point1 {
};
camss-1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 9>;
trips {
@@ -3012,9 +2946,6 @@ trip-point1 {
};
pcie-1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 10>;
trips {
@@ -3033,9 +2964,6 @@ trip-point1 {
};
cpuss-0-1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 11>;
trips {
@@ -3054,9 +2982,6 @@ trip-point1 {
};
aoss-2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 0>;
trips {
@@ -3075,9 +3000,6 @@ trip-point1 {
};
cpu-1-0-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 1>;
trips {
@@ -3096,9 +3018,6 @@ trip-point1 {
};
cpu-1-1-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 2>;
trips {
@@ -3117,9 +3036,6 @@ trip-point1 {
};
cpu-1-2-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 3>;
trips {
@@ -3138,9 +3054,6 @@ trip-point1 {
};
cpu-1-3-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 4>;
trips {
@@ -3159,9 +3072,6 @@ trip-point1 {
};
nsp-0-0-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 5>;
trips {
@@ -3180,9 +3090,6 @@ trip-point1 {
};
nsp-0-1-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 6>;
trips {
@@ -3201,9 +3108,6 @@ trip-point1 {
};
nsp-0-2-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 7>;
trips {
@@ -3222,9 +3126,6 @@ trip-point1 {
};
nsp-1-0-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 8>;
trips {
@@ -3243,9 +3144,6 @@ trip-point1 {
};
nsp-1-1-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 9>;
trips {
@@ -3264,9 +3162,6 @@ trip-point1 {
};
nsp-1-2-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 10>;
trips {
@@ -3285,9 +3180,6 @@ trip-point1 {
};
ddrss-0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 11>;
trips {
@@ -3306,9 +3198,6 @@ trip-point1 {
};
cpuss-1-0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 12>;
trips {
@@ -3327,9 +3216,6 @@ trip-point1 {
};
aoss-3-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens3 0>;
trips {
@@ -3348,9 +3234,6 @@ trip-point1 {
};
cpu-1-0-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens3 1>;
trips {
@@ -3369,9 +3252,6 @@ trip-point1 {
};
cpu-1-1-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens3 2>;
trips {
@@ -3390,9 +3270,6 @@ trip-point1 {
};
cpu-1-2-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens3 3>;
trips {
@@ -3411,9 +3288,6 @@ trip-point1 {
};
cpu-1-3-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens3 4>;
trips {
@@ -3432,9 +3306,6 @@ trip-point1 {
};
nsp-0-0-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens3 5>;
trips {
@@ -3453,9 +3324,6 @@ trip-point1 {
};
nsp-0-1-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens3 6>;
trips {
@@ -3474,9 +3342,6 @@ trip-point1 {
};
nsp-0-2-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens3 7>;
trips {
@@ -3495,9 +3360,6 @@ trip-point1 {
};
nsp-1-0-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens3 8>;
trips {
@@ -3516,9 +3378,6 @@ trip-point1 {
};
nsp-1-1-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens3 9>;
trips {
@@ -3537,9 +3396,6 @@ trip-point1 {
};
nsp-1-2-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens3 10>;
trips {
@@ -3558,9 +3414,6 @@ trip-point1 {
};
ddrss-1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens3 11>;
trips {
@@ -3579,9 +3432,6 @@ trip-point1 {
};
cpuss-1-1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens3 12>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 16/31] arm64: dts: qcom: sc7180-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (14 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 15/31] arm64: dts: qcom: sa8775p-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 17/31] arm64: dts: qcom: sc7280-*: " Konrad Dybcio
` (16 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/pm6150.dtsi | 2 -
arch/arm64/boot/dts/qcom/pm6150l.dtsi | 3 -
.../arm64/boot/dts/qcom/sc7180-trogdor-coachz.dtsi | 3 -
.../boot/dts/qcom/sc7180-trogdor-homestar.dtsi | 3 -
.../arm64/boot/dts/qcom/sc7180-trogdor-pompom.dtsi | 3 -
.../boot/dts/qcom/sc7180-trogdor-wormdingler.dtsi | 3 -
arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 3 -
arch/arm64/boot/dts/qcom/sc7180.dtsi | 75 ----------------------
8 files changed, 95 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/pm6150.dtsi b/arch/arm64/boot/dts/qcom/pm6150.dtsi
index 11158c2bd524..78acad023e62 100644
--- a/arch/arm64/boot/dts/qcom/pm6150.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm6150.dtsi
@@ -12,8 +12,6 @@
/ {
thermal-zones {
pm6150_thermal: pm6150-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
thermal-sensors = <&pm6150_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm6150l.dtsi b/arch/arm64/boot/dts/qcom/pm6150l.dtsi
index d13a1ab7c20b..a20e9b9993b2 100644
--- a/arch/arm64/boot/dts/qcom/pm6150l.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm6150l.dtsi
@@ -10,9 +10,6 @@
/ {
thermal-zones {
pm6150l-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&pm6150l_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz.dtsi
index 7765c8f64905..80fa40cdebd5 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz.dtsi
@@ -24,9 +24,6 @@ adau7002: audio-codec-1 {
thermal-zones {
skin_temp_thermal: skin-temp-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&pm6150_adc_tm 1>;
sustainable-power = <965>;
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar.dtsi
index 2ba3bbf3b9ad..385eba9e142d 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar.dtsi
@@ -44,9 +44,6 @@ pp3300_touch: pp3300-touch-regulator {
thermal-zones {
skin_temp_thermal: skin-temp-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&pm6150_adc_tm 1>;
sustainable-power = <965>;
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom.dtsi
index 067813f5f437..026d0845d78b 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom.dtsi
@@ -14,9 +14,6 @@
/ {
thermal-zones {
5v-choke-thermal {
- polling-delay-passive = <0>;
- polling-delay = <250>;
-
thermal-sensors = <&pm6150_adc_tm 1>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler.dtsi
index 305ad127246e..1bf313a3c59c 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler.dtsi
@@ -49,9 +49,6 @@ v1p8_mipi: v1p8-mipi-regulator {
thermal-zones {
skin_temp_thermal: skin-temp-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&pm6150_adc_tm 1>;
sustainable-power = <574>;
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
index f3a6da8b2890..8a8a5ca0e568 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
@@ -21,9 +21,6 @@
/ {
thermal-zones {
charger_thermal: charger-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&pm6150_adc_tm 0>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
index 2b481e20ae38..54e0ab1c8580 100644
--- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
@@ -4031,9 +4031,6 @@ lpass_hm: clock-controller@63000000 {
thermal-zones {
cpu0_thermal: cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 1>;
sustainable-power = <1052>;
@@ -4080,9 +4077,6 @@ map1 {
};
cpu1_thermal: cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 2>;
sustainable-power = <1052>;
@@ -4129,9 +4123,6 @@ map1 {
};
cpu2_thermal: cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 3>;
sustainable-power = <1052>;
@@ -4178,9 +4169,6 @@ map1 {
};
cpu3_thermal: cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 4>;
sustainable-power = <1052>;
@@ -4227,9 +4215,6 @@ map1 {
};
cpu4_thermal: cpu4-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 5>;
sustainable-power = <1052>;
@@ -4276,9 +4261,6 @@ map1 {
};
cpu5_thermal: cpu5-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 6>;
sustainable-power = <1052>;
@@ -4325,9 +4307,6 @@ map1 {
};
cpu6_thermal: cpu6-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 9>;
sustainable-power = <1425>;
@@ -4366,9 +4345,6 @@ map1 {
};
cpu7_thermal: cpu7-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 10>;
sustainable-power = <1425>;
@@ -4407,9 +4383,6 @@ map1 {
};
cpu8_thermal: cpu8-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 11>;
sustainable-power = <1425>;
@@ -4448,9 +4421,6 @@ map1 {
};
cpu9_thermal: cpu9-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 12>;
sustainable-power = <1425>;
@@ -4489,9 +4459,6 @@ map1 {
};
aoss0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 0>;
trips {
@@ -4510,9 +4477,6 @@ aoss0_crit: aoss0-crit {
};
cpuss0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -4530,9 +4494,6 @@ cpuss0_crit: cluster0-crit {
};
cpuss1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -4550,9 +4511,6 @@ cpuss1_crit: cluster0-crit {
};
gpuss0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 13>;
trips {
@@ -4578,9 +4536,6 @@ map0 {
};
gpuss1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 14>;
trips {
@@ -4606,9 +4561,6 @@ map0 {
};
aoss1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 0>;
trips {
@@ -4627,9 +4579,6 @@ aoss1_crit: aoss1-crit {
};
cwlan-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 1>;
trips {
@@ -4648,9 +4597,6 @@ cwlan_crit: cwlan-crit {
};
audio-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 2>;
trips {
@@ -4669,9 +4615,6 @@ audio_crit: audio-crit {
};
ddr-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 3>;
trips {
@@ -4690,9 +4633,6 @@ ddr_crit: ddr-crit {
};
q6-hvx-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 4>;
trips {
@@ -4711,9 +4651,6 @@ q6_hvx_crit: q6-hvx-crit {
};
camera-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 5>;
trips {
@@ -4732,9 +4669,6 @@ camera_crit: camera-crit {
};
mdm-core-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 6>;
trips {
@@ -4753,9 +4687,6 @@ mdm_crit: mdm-crit {
};
mdm-dsp-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 7>;
trips {
@@ -4774,9 +4705,6 @@ mdm_dsp_crit: mdm-dsp-crit {
};
npu-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 8>;
trips {
@@ -4795,9 +4723,6 @@ npu_crit: npu-crit {
};
video-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 9>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 17/31] arm64: dts: qcom: sc7280-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (15 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 16/31] arm64: dts: qcom: sc7180-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 18/31] arm64: dts: qcom: sc8180x-*: " Konrad Dybcio
` (15 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/pm7250b.dtsi | 3 -
arch/arm64/boot/dts/qcom/pm7325.dtsi | 2 -
arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts | 14 ----
arch/arm64/boot/dts/qcom/sc7280.dtsi | 81 ----------------------
4 files changed, 100 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/pm7250b.dtsi b/arch/arm64/boot/dts/qcom/pm7250b.dtsi
index 3bf7cf5d1700..48eb8d9a5ac3 100644
--- a/arch/arm64/boot/dts/qcom/pm7250b.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm7250b.dtsi
@@ -10,9 +10,6 @@
/ {
thermal-zones {
pm7250b-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm7250b_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm7325.dtsi b/arch/arm64/boot/dts/qcom/pm7325.dtsi
index d1c5476af5ee..a25f6f65e416 100644
--- a/arch/arm64/boot/dts/qcom/pm7325.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm7325.dtsi
@@ -34,8 +34,6 @@ pm7325_gpios: gpio@8800 {
&thermal_zones {
pm7325_thermal: pm7325-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
thermal-sensors = <&pm7325_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts b/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts
index 4ff9fc24e50e..e4882b94d103 100644
--- a/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts
+++ b/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts
@@ -164,8 +164,6 @@ afvdd_2p8: regulator-afvdd-2p8 {
thermal-zones {
camera-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pmk8350_adc_tm 2>;
trips {
@@ -178,8 +176,6 @@ active-config0 {
};
chg-skin-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm7250b_adc_tm 0>;
trips {
@@ -192,8 +188,6 @@ active-config0 {
};
conn-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm7250b_adc_tm 1>;
trips {
@@ -206,8 +200,6 @@ active-config0 {
};
quiet-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pmk8350_adc_tm 1>;
trips {
@@ -220,8 +212,6 @@ active-config0 {
};
rear-cam-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pmk8350_adc_tm 4>;
trips {
@@ -234,8 +224,6 @@ active-config0 {
};
sdm-skin-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pmk8350_adc_tm 3>;
trips {
@@ -248,8 +236,6 @@ active-config0 {
};
xo-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pmk8350_adc_tm 0>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
index 7e7f0f0fb41b..ef60a511a69f 100644
--- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
@@ -5969,9 +5969,6 @@ cpufreq_hw: cpufreq@18591000 {
thermal_zones: thermal-zones {
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 1>;
trips {
@@ -6013,9 +6010,6 @@ map1 {
};
cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 2>;
trips {
@@ -6057,9 +6051,6 @@ map1 {
};
cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 3>;
trips {
@@ -6101,9 +6092,6 @@ map1 {
};
cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 4>;
trips {
@@ -6145,9 +6133,6 @@ map1 {
};
cpu4-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -6189,9 +6174,6 @@ map1 {
};
cpu5-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -6233,9 +6215,6 @@ map1 {
};
cpu6-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 9>;
trips {
@@ -6277,9 +6256,6 @@ map1 {
};
cpu7-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 10>;
trips {
@@ -6321,9 +6297,6 @@ map1 {
};
cpu8-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 11>;
trips {
@@ -6365,9 +6338,6 @@ map1 {
};
cpu9-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 12>;
trips {
@@ -6409,9 +6379,6 @@ map1 {
};
cpu10-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 13>;
trips {
@@ -6453,9 +6420,6 @@ map1 {
};
cpu11-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 14>;
trips {
@@ -6497,9 +6461,6 @@ map1 {
};
aoss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 0>;
trips {
@@ -6518,9 +6479,6 @@ aoss0_crit: aoss0-crit {
};
aoss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 0>;
trips {
@@ -6539,9 +6497,6 @@ aoss1_crit: aoss1-crit {
};
cpuss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 5>;
trips {
@@ -6559,9 +6514,6 @@ cpuss0_crit: cluster0-crit {
};
cpuss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 6>;
trips {
@@ -6579,9 +6531,6 @@ cpuss1_crit: cluster0-crit {
};
gpuss0-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 1>;
trips {
@@ -6607,9 +6556,6 @@ map0 {
};
gpuss1-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 2>;
trips {
@@ -6635,9 +6581,6 @@ map0 {
};
nspss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 3>;
trips {
@@ -6656,9 +6599,6 @@ nspss0_crit: nspss0-crit {
};
nspss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 4>;
trips {
@@ -6677,9 +6617,6 @@ nspss1_crit: nspss1-crit {
};
video-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 5>;
trips {
@@ -6698,9 +6635,6 @@ video_crit: video-crit {
};
ddr-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 6>;
trips {
@@ -6719,9 +6653,6 @@ ddr_crit: ddr-crit {
};
mdmss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 7>;
trips {
@@ -6740,9 +6671,6 @@ mdmss0_crit: mdmss0-crit {
};
mdmss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 8>;
trips {
@@ -6761,9 +6689,6 @@ mdmss1_crit: mdmss1-crit {
};
mdmss2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 9>;
trips {
@@ -6782,9 +6707,6 @@ mdmss2_crit: mdmss2-crit {
};
mdmss3-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 10>;
trips {
@@ -6803,9 +6725,6 @@ mdmss3_crit: mdmss3-crit {
};
camera0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 11>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 18/31] arm64: dts: qcom: sc8180x-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (16 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 17/31] arm64: dts: qcom: sc7280-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 19/31] arm64: dts: qcom: sc8280xp-*: " Konrad Dybcio
` (14 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/sc8180x-pmics.dtsi | 6 ---
arch/arm64/boot/dts/qcom/sc8180x.dtsi | 78 -----------------------------
2 files changed, 84 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sc8180x-pmics.dtsi b/arch/arm64/boot/dts/qcom/sc8180x-pmics.dtsi
index ddc84282f142..e88e7fd832fe 100644
--- a/arch/arm64/boot/dts/qcom/sc8180x-pmics.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8180x-pmics.dtsi
@@ -12,9 +12,6 @@
/ {
thermal-zones {
pmc8180-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pmc8180_temp>;
trips {
@@ -39,9 +36,6 @@ trip2 {
};
pmc8180c-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pmc8180c_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sc8180x.dtsi b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
index 32afc78d5b76..010afc39895e 100644
--- a/arch/arm64/boot/dts/qcom/sc8180x.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
@@ -3729,9 +3729,6 @@ wifi: wifi@18800000 {
thermal-zones {
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 1>;
trips {
@@ -3744,9 +3741,6 @@ cpu-crit {
};
cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 2>;
trips {
@@ -3759,9 +3753,6 @@ cpu-crit {
};
cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 3>;
trips {
@@ -3774,9 +3765,6 @@ cpu-crit {
};
cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 4>;
trips {
@@ -3789,9 +3777,6 @@ cpu-crit {
};
cpu4-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -3804,9 +3789,6 @@ cpu-crit {
};
cpu5-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -3819,9 +3801,6 @@ cpu-crit {
};
cpu6-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 9>;
trips {
@@ -3834,9 +3813,6 @@ cpu-crit {
};
cpu7-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 10>;
trips {
@@ -3849,9 +3825,6 @@ cpu-crit {
};
cpu4-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 11>;
trips {
@@ -3864,9 +3837,6 @@ cpu-crit {
};
cpu5-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 12>;
trips {
@@ -3879,9 +3849,6 @@ cpu-crit {
};
cpu6-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 13>;
trips {
@@ -3894,9 +3861,6 @@ cpu-crit {
};
cpu7-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 14>;
trips {
@@ -3909,9 +3873,6 @@ cpu-crit {
};
aoss0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 0>;
trips {
@@ -3924,9 +3885,6 @@ trip-point0 {
};
cluster0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 5>;
trips {
@@ -3939,9 +3897,6 @@ cluster-crit {
};
cluster1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 6>;
trips {
@@ -3954,9 +3909,6 @@ cluster-crit {
};
gpu-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 15>;
cooling-maps {
@@ -3976,9 +3928,6 @@ gpu_top_alert0: trip-point0 {
};
aoss1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 0>;
trips {
@@ -3991,9 +3940,6 @@ trip-point0 {
};
wlan-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 1>;
trips {
@@ -4006,9 +3952,6 @@ trip-point0 {
};
video-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 2>;
trips {
@@ -4021,9 +3964,6 @@ trip-point0 {
};
mem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 3>;
trips {
@@ -4036,9 +3976,6 @@ trip-point0 {
};
q6-hvx-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 4>;
trips {
@@ -4051,9 +3988,6 @@ trip-point0 {
};
camera-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 5>;
trips {
@@ -4066,9 +4000,6 @@ trip-point0 {
};
compute-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 6>;
trips {
@@ -4081,9 +4012,6 @@ trip-point0 {
};
mdm-dsp-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 7>;
trips {
@@ -4096,9 +4024,6 @@ trip-point0 {
};
npu-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 8>;
trips {
@@ -4111,9 +4036,6 @@ trip-point0 {
};
gpu-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 11>;
cooling-maps {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 19/31] arm64: dts: qcom: sc8280xp-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (17 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 18/31] arm64: dts: qcom: sc8180x-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 20/31] arm64: dts: qcom: sdm660-*: " Konrad Dybcio
` (13 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
.../dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts | 2 --
arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi | 4 ---
arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 33 ----------------------
3 files changed, 39 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts b/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts
index 15ae94c1602d..cd6a585359f7 100644
--- a/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts
+++ b/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts
@@ -296,8 +296,6 @@ linux,cma {
thermal-zones {
skin-temp-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
thermal-sensors = <&pmk8280_adc_tm 5>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi
index 945de77911de..4283d59c556d 100644
--- a/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi
@@ -13,8 +13,6 @@
/ {
thermal-zones {
pm8280_1_thermal: pm8280-1-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
thermal-sensors = <&pm8280_1_temp_alarm>;
trips {
@@ -33,8 +31,6 @@ trip1 {
};
pm8280_2_thermal: pm8280-2-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
thermal-sensors = <&pm8280_2_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi
index a5b194813079..00fd4a187297 100644
--- a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi
@@ -5648,9 +5648,6 @@ sound: sound {
thermal-zones {
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 1>;
trips {
@@ -5663,9 +5660,6 @@ cpu-crit {
};
cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 2>;
trips {
@@ -5678,9 +5672,6 @@ cpu-crit {
};
cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 3>;
trips {
@@ -5693,9 +5684,6 @@ cpu-crit {
};
cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 4>;
trips {
@@ -5708,9 +5696,6 @@ cpu-crit {
};
cpu4-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 5>;
trips {
@@ -5723,9 +5708,6 @@ cpu-crit {
};
cpu5-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 6>;
trips {
@@ -5738,9 +5720,6 @@ cpu-crit {
};
cpu6-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -5753,9 +5732,6 @@ cpu-crit {
};
cpu7-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -5768,9 +5744,6 @@ cpu-crit {
};
cluster0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 9>;
trips {
@@ -5783,9 +5756,6 @@ cpu-crit {
};
gpu-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens2 2>;
trips {
@@ -5798,9 +5768,6 @@ gpu-crit {
};
mem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 15>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 20/31] arm64: dts: qcom: sdm660-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (18 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 19/31] arm64: dts: qcom: sc8280xp-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 21/31] arm64: dts: qcom: sdm845-*: " Konrad Dybcio
` (12 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/pm660.dtsi | 3 ---
arch/arm64/boot/dts/qcom/pm660l.dtsi | 3 ---
arch/arm64/boot/dts/qcom/sdm630.dtsi | 27 ---------------------------
3 files changed, 33 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/pm660.dtsi b/arch/arm64/boot/dts/qcom/pm660.dtsi
index 98dc04962fe3..24efd1bf271c 100644
--- a/arch/arm64/boot/dts/qcom/pm660.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm660.dtsi
@@ -12,9 +12,6 @@
/ {
thermal-zones {
pm660-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&pm660_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm660l.dtsi b/arch/arm64/boot/dts/qcom/pm660l.dtsi
index 6fdbf507c262..46165a2a3694 100644
--- a/arch/arm64/boot/dts/qcom/pm660l.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm660l.dtsi
@@ -12,9 +12,6 @@
/ {
thermal-zones {
pm660l-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&pm660l_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
index f5921b80ef94..9851bc73222b 100644
--- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
@@ -2421,9 +2421,6 @@ sound: sound {
thermal-zones {
aoss-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 0>;
trips {
@@ -2436,9 +2433,6 @@ aoss_alert0: trip-point0 {
};
cpuss0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 1>;
trips {
@@ -2451,9 +2445,6 @@ cpuss0_alert0: trip-point0 {
};
cpuss1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 2>;
trips {
@@ -2466,9 +2457,6 @@ cpuss1_alert0: trip-point0 {
};
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 3>;
trips {
@@ -2487,9 +2475,6 @@ cpu0_crit: cpu-crit {
};
cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 4>;
trips {
@@ -2508,9 +2493,6 @@ cpu1_crit: cpu-crit {
};
cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 5>;
trips {
@@ -2529,9 +2511,6 @@ cpu2_crit: cpu-crit {
};
cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 6>;
trips {
@@ -2556,9 +2535,6 @@ cpu3_crit: cpu-crit {
*/
pwr-cluster-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 7>;
trips {
@@ -2577,9 +2553,6 @@ pwr_cluster_crit: cpu-crit {
};
gpu-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens 8>;
cooling-maps {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 21/31] arm64: dts: qcom: sdm845-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (19 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 20/31] arm64: dts: qcom: sdm660-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 22/31] arm64: dts: qcom: sm6115-*: " Konrad Dybcio
` (11 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/sdm845-mtp.dts | 12 -------
arch/arm64/boot/dts/qcom/sdm845.dtsi | 63 ---------------------------------
2 files changed, 75 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
index 76bfa786612c..2391f842c903 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
+++ b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
@@ -51,9 +51,6 @@ vreg_s4a_1p8: pm8998-smps4 {
thermal-zones {
xo_thermal: xo-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8998_adc_tm 1>;
trips {
@@ -66,9 +63,6 @@ trip-point {
};
msm_thermal: msm-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8998_adc_tm 2>;
trips {
@@ -81,9 +75,6 @@ trip-point {
};
pa_thermal: pa-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8998_adc_tm 3>;
trips {
@@ -96,9 +87,6 @@ trip-point {
};
quiet_thermal: quiet-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8998_adc_tm 4>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
index 2f20be99ee7e..ef05e3506d54 100644
--- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
@@ -5337,9 +5337,6 @@ sound: sound {
thermal-zones {
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 1>;
trips {
@@ -5364,9 +5361,6 @@ cpu0_crit: cpu-crit {
};
cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 2>;
trips {
@@ -5391,9 +5385,6 @@ cpu1_crit: cpu-crit {
};
cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 3>;
trips {
@@ -5418,9 +5409,6 @@ cpu2_crit: cpu-crit {
};
cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 4>;
trips {
@@ -5445,9 +5433,6 @@ cpu3_crit: cpu-crit {
};
cpu4-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -5472,9 +5457,6 @@ cpu4_crit: cpu-crit {
};
cpu5-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -5499,9 +5481,6 @@ cpu5_crit: cpu-crit {
};
cpu6-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 9>;
trips {
@@ -5526,9 +5505,6 @@ cpu6_crit: cpu-crit {
};
cpu7-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 10>;
trips {
@@ -5553,9 +5529,6 @@ cpu7_crit: cpu-crit {
};
aoss0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 0>;
trips {
@@ -5568,9 +5541,6 @@ aoss0_alert0: trip-point0 {
};
cluster0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 5>;
trips {
@@ -5588,9 +5558,6 @@ cluster0_crit: cluster0-crit {
};
cluster1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 6>;
trips {
@@ -5608,9 +5575,6 @@ cluster1_crit: cluster1-crit {
};
gpu-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 11>;
cooling-maps {
@@ -5630,9 +5594,6 @@ gpu_top_alert0: trip-point0 {
};
gpu-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 12>;
cooling-maps {
@@ -5652,9 +5613,6 @@ gpu_bottom_alert0: trip-point0 {
};
aoss1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 0>;
trips {
@@ -5667,9 +5625,6 @@ aoss1_alert0: trip-point0 {
};
q6-modem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 1>;
trips {
@@ -5682,9 +5637,6 @@ q6_modem_alert0: trip-point0 {
};
mem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 2>;
trips {
@@ -5697,9 +5649,6 @@ mem_alert0: trip-point0 {
};
wlan-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 3>;
trips {
@@ -5712,9 +5661,6 @@ wlan_alert0: trip-point0 {
};
q6-hvx-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 4>;
trips {
@@ -5727,9 +5673,6 @@ q6_hvx_alert0: trip-point0 {
};
camera-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 5>;
trips {
@@ -5742,9 +5685,6 @@ camera_alert0: trip-point0 {
};
video-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 6>;
trips {
@@ -5757,9 +5697,6 @@ video_alert0: trip-point0 {
};
modem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 7>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 22/31] arm64: dts: qcom: sm6115-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (20 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 21/31] arm64: dts: qcom: sdm845-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 23/31] arm64: dts: qcom: sm6125-*: " Konrad Dybcio
` (10 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/pmi632.dtsi | 3 ---
arch/arm64/boot/dts/qcom/sm6115.dtsi | 32 --------------------------------
2 files changed, 35 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/pmi632.dtsi b/arch/arm64/boot/dts/qcom/pmi632.dtsi
index 94d53b1cf6c8..e1658993d6c1 100644
--- a/arch/arm64/boot/dts/qcom/pmi632.dtsi
+++ b/arch/arm64/boot/dts/qcom/pmi632.dtsi
@@ -10,9 +10,6 @@
/ {
thermal-zones {
pmi632-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pmi632_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sm6115.dtsi b/arch/arm64/boot/dts/qcom/sm6115.dtsi
index aca0a87092e4..0a0bb5310849 100644
--- a/arch/arm64/boot/dts/qcom/sm6115.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm6115.dtsi
@@ -3011,8 +3011,6 @@ cpufreq_hw: cpufreq@f521000 {
thermal-zones {
mapss-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 0>;
trips {
@@ -3031,8 +3029,6 @@ trip-point1 {
};
cdsp-hvx-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 1>;
trips {
@@ -3051,8 +3047,6 @@ trip-point1 {
};
wlan-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 2>;
trips {
@@ -3071,8 +3065,6 @@ trip-point1 {
};
camera-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 3>;
trips {
@@ -3091,8 +3083,6 @@ trip-point1 {
};
video-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 4>;
trips {
@@ -3111,8 +3101,6 @@ trip-point1 {
};
modem1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 5>;
trips {
@@ -3131,8 +3119,6 @@ trip-point1 {
};
cpu4-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 6>;
trips {
@@ -3157,8 +3143,6 @@ cpu4_crit: cpu-crit {
};
cpu5-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 7>;
trips {
@@ -3183,8 +3167,6 @@ cpu5_crit: cpu-crit {
};
cpu6-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 8>;
trips {
@@ -3209,8 +3191,6 @@ cpu6_crit: cpu-crit {
};
cpu7-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 9>;
trips {
@@ -3235,8 +3215,6 @@ cpu7_crit: cpu-crit {
};
cpu45-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 10>;
trips {
@@ -3261,8 +3239,6 @@ cpu45_crit: cpu-crit {
};
cpu67-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 11>;
trips {
@@ -3287,8 +3263,6 @@ cpu67_crit: cpu-crit {
};
cpu0123-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 12>;
trips {
@@ -3313,8 +3287,6 @@ cpu0123_crit: cpu-crit {
};
modem0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 13>;
trips {
@@ -3333,8 +3305,6 @@ trip-point1 {
};
display-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 14>;
trips {
@@ -3353,8 +3323,6 @@ trip-point1 {
};
gpu-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 15>;
cooling-maps {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 23/31] arm64: dts: qcom: sm6125-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (21 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 22/31] arm64: dts: qcom: sm6115-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 24/31] arm64: dts: qcom: sm6350-*: " Konrad Dybcio
` (9 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/pm6125.dtsi | 3 ---
arch/arm64/boot/dts/qcom/sm6125-sony-xperia-seine-pdx201.dts | 8 --------
arch/arm64/boot/dts/qcom/sm6125-xiaomi-laurel-sprout.dts | 6 ------
3 files changed, 17 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/pm6125.dtsi b/arch/arm64/boot/dts/qcom/pm6125.dtsi
index 99369a0cdb61..b92f357511de 100644
--- a/arch/arm64/boot/dts/qcom/pm6125.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm6125.dtsi
@@ -9,9 +9,6 @@
/ {
thermal-zones {
pm6125-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm6125_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sm6125-sony-xperia-seine-pdx201.dts b/arch/arm64/boot/dts/qcom/sm6125-sony-xperia-seine-pdx201.dts
index 08046f866f60..dcd05f303b78 100644
--- a/arch/arm64/boot/dts/qcom/sm6125-sony-xperia-seine-pdx201.dts
+++ b/arch/arm64/boot/dts/qcom/sm6125-sony-xperia-seine-pdx201.dts
@@ -90,8 +90,6 @@ cmdline_mem: memory@ffd00000 {
thermal-zones {
rf-pa0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm6125_adc_tm 0>;
trips {
@@ -104,8 +102,6 @@ active-config0 {
};
quiet-thermal {
- polling-delay-passive = <0>;
- polling-delay = <5000>;
thermal-sensors = <&pm6125_adc_tm 1>;
trips {
@@ -118,8 +114,6 @@ active-config0 {
};
xo-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm6125_adc_tm 2>;
trips {
@@ -132,8 +126,6 @@ active-config0 {
};
rf-pa1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm6125_adc_tm 3>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sm6125-xiaomi-laurel-sprout.dts b/arch/arm64/boot/dts/qcom/sm6125-xiaomi-laurel-sprout.dts
index a49d3ebb1931..994fb0412fcb 100644
--- a/arch/arm64/boot/dts/qcom/sm6125-xiaomi-laurel-sprout.dts
+++ b/arch/arm64/boot/dts/qcom/sm6125-xiaomi-laurel-sprout.dts
@@ -84,8 +84,6 @@ key-volume-up {
thermal-zones {
rf-pa0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm6125_adc_tm 0>;
trips {
@@ -98,8 +96,6 @@ active-config0 {
};
quiet-thermal {
- polling-delay-passive = <0>;
- polling-delay = <5000>;
thermal-sensors = <&pm6125_adc_tm 1>;
trips {
@@ -112,8 +108,6 @@ active-config0 {
};
xo-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm6125_adc_tm 2>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 24/31] arm64: dts: qcom: sm6350-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (22 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 23/31] arm64: dts: qcom: sm6125-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 25/31] arm64: dts: qcom: sm6375-*: " Konrad Dybcio
` (8 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/pm6350.dtsi | 3 -
arch/arm64/boot/dts/qcom/sm6350.dtsi | 81 -----------------------
arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts | 4 --
3 files changed, 88 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/pm6350.dtsi b/arch/arm64/boot/dts/qcom/pm6350.dtsi
index 3a2a841e83f1..be118da8838b 100644
--- a/arch/arm64/boot/dts/qcom/pm6350.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm6350.dtsi
@@ -9,9 +9,6 @@
/ {
thermal-zones {
pm6350-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm6350_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sm6350.dtsi b/arch/arm64/boot/dts/qcom/sm6350.dtsi
index 24bcec3366ef..da17057028af 100644
--- a/arch/arm64/boot/dts/qcom/sm6350.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm6350.dtsi
@@ -2712,9 +2712,6 @@ wifi: wifi@18800000 {
thermal-zones {
aoss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 0>;
trips {
@@ -2727,9 +2724,6 @@ aoss0-crit {
};
aoss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 0>;
trips {
@@ -2742,9 +2736,6 @@ aoss1-crit {
};
audio-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 2>;
trips {
@@ -2757,9 +2748,6 @@ audio-crit {
};
camera-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 5>;
trips {
@@ -2772,9 +2760,6 @@ camera-crit {
};
cpu0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 1>;
trips {
@@ -2800,9 +2785,6 @@ map0 {
};
cpu1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 2>;
trips {
@@ -2828,9 +2810,6 @@ map0 {
};
cpu2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 3>;
trips {
@@ -2856,9 +2835,6 @@ map0 {
};
cpu3-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 4>;
trips {
@@ -2884,9 +2860,6 @@ map0 {
};
cpu4-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 5>;
trips {
@@ -2912,9 +2885,6 @@ map0 {
};
cpu5-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 6>;
trips {
@@ -2940,9 +2910,6 @@ map0 {
};
cpu6-left-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 9>;
trips {
@@ -2968,9 +2935,6 @@ map0 {
};
cpu6-right-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 10>;
trips {
@@ -2996,9 +2960,6 @@ map0 {
};
cpu7-left-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 11>;
trips {
@@ -3024,9 +2985,6 @@ map0 {
};
cpu7-right-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 12>;
trips {
@@ -3052,9 +3010,6 @@ map0 {
};
cpuss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -3067,9 +3022,6 @@ cpuss0-crit {
};
cpuss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -3082,9 +3034,6 @@ cpuss1-crit {
};
cwlan-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 1>;
trips {
@@ -3097,9 +3046,6 @@ cwlan-crit {
};
ddr-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 3>;
trips {
@@ -3112,9 +3058,6 @@ ddr-crit {
};
gpuss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 13>;
trips {
@@ -3140,9 +3083,6 @@ map0 {
};
gpuss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 14>;
trips {
@@ -3168,9 +3108,6 @@ map0 {
};
modem-core0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 6>;
trips {
@@ -3183,9 +3120,6 @@ modem-core0-crit {
};
modem-core1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 7>;
trips {
@@ -3198,9 +3132,6 @@ modem-core1-crit {
};
modem-scl-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 9>;
trips {
@@ -3213,9 +3144,6 @@ modem-scl-crit {
};
modem-vec-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 8>;
trips {
@@ -3228,9 +3156,6 @@ modem-vec-crit {
};
npu-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 10>;
trips {
@@ -3243,9 +3168,6 @@ npu-crit {
};
q6-hvx-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 4>;
trips {
@@ -3258,9 +3180,6 @@ q6-hvx-crit {
};
video-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 11>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts b/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
index bc67e8c1fe4d..c50fa7f3a5fd 100644
--- a/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
+++ b/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
@@ -94,8 +94,6 @@ memory@efe01000 {
thermal-zones {
chg-skin-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm7250b_adc_tm 0>;
trips {
@@ -108,8 +106,6 @@ active-config0 {
};
conn-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm7250b_adc_tm 1>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 25/31] arm64: dts: qcom: sm6375-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (23 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 24/31] arm64: dts: qcom: sm6350-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 26/31] arm64: dts: qcom: sm8150-*: " Konrad Dybcio
` (7 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/sm6375.dtsi | 78 ------------------------------------
1 file changed, 78 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sm6375.dtsi b/arch/arm64/boot/dts/qcom/sm6375.dtsi
index 4386f8a9c636..16880789f071 100644
--- a/arch/arm64/boot/dts/qcom/sm6375.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm6375.dtsi
@@ -1837,9 +1837,6 @@ cpufreq_hw: cpufreq@fd91000 {
thermal-zones {
mapss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 0>;
trips {
@@ -1864,9 +1861,6 @@ mapss0_crit: mapss-crit {
};
cpu0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 1>;
trips {
@@ -1891,9 +1885,6 @@ cpu0_crit: cpu-crit {
};
cpu1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 2>;
trips {
@@ -1918,9 +1909,6 @@ cpu1_crit: cpu-crit {
};
cpu2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 3>;
trips {
@@ -1945,9 +1933,6 @@ cpu2_crit: cpu-crit {
};
cpu3-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 4>;
trips {
@@ -1972,9 +1957,6 @@ cpu3_crit: cpu-crit {
};
cpu4-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 5>;
trips {
@@ -1999,9 +1981,6 @@ cpu4_crit: cpu-crit {
};
cpu5-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 6>;
trips {
@@ -2026,9 +2005,6 @@ cpu5_crit: cpu-crit {
};
cluster0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -2053,9 +2029,6 @@ cluster0_crit: cpu-crit {
};
cluster1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -2080,9 +2053,6 @@ cluster1_crit: cpu-crit {
};
cpu6-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 9>;
trips {
@@ -2107,9 +2077,6 @@ cpu6_crit: cpu-crit {
};
cpu7-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 10>;
trips {
@@ -2134,9 +2101,6 @@ cpu7_crit: cpu-crit {
};
cpu-unk0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 11>;
trips {
@@ -2161,9 +2125,6 @@ cpu_unk0_crit: cpu-crit {
};
cpu-unk1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 12>;
trips {
@@ -2188,9 +2149,6 @@ cpu_unk1_crit: cpu-crit {
};
gpuss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 13>;
trips {
@@ -2215,9 +2173,6 @@ gpuss0_crit: gpu-crit {
};
gpuss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens0 14>;
trips {
@@ -2242,9 +2197,6 @@ gpuss1_crit: gpu-crit {
};
mapss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 0>;
trips {
@@ -2269,9 +2221,6 @@ mapss1_crit: mapss-crit {
};
cwlan-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 1>;
trips {
@@ -2296,9 +2245,6 @@ cwlan_crit: cwlan-crit {
};
audio-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 2>;
trips {
@@ -2323,9 +2269,6 @@ audio_crit: audio-crit {
};
ddr-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 3>;
trips {
@@ -2350,9 +2293,6 @@ ddr_crit: ddr-crit {
};
q6hvx-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 4>;
trips {
@@ -2377,9 +2317,6 @@ q6hvx_crit: q6hvx-crit {
};
camera-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 5>;
trips {
@@ -2404,9 +2341,6 @@ camera_crit: camera-crit {
};
mdm-core0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 6>;
trips {
@@ -2431,9 +2365,6 @@ mdm_core0_crit: mdm-core0-crit {
};
mdm-core1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 7>;
trips {
@@ -2458,9 +2389,6 @@ mdm_core1_crit: mdm-core1-crit {
};
mdm-vec-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 8>;
trips {
@@ -2485,9 +2413,6 @@ mdm_vec_crit: mdm-vec-crit {
};
msm-scl-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 9>;
trips {
@@ -2512,9 +2437,6 @@ msm_scl_crit: msm-scl-crit {
};
video-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
-
thermal-sensors = <&tsens1 10>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 26/31] arm64: dts: qcom: sm8150-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (24 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 25/31] arm64: dts: qcom: sm6375-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 27/31] arm64: dts: qcom: sm8250-*: " Konrad Dybcio
` (6 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/pm8150.dtsi | 3 --
arch/arm64/boot/dts/qcom/pm8150b.dtsi | 3 --
arch/arm64/boot/dts/qcom/pm8150l.dtsi | 3 --
arch/arm64/boot/dts/qcom/pmm8155au_1.dtsi | 3 --
arch/arm64/boot/dts/qcom/pmm8155au_2.dtsi | 3 --
arch/arm64/boot/dts/qcom/sm8150.dtsi | 84 -------------------------------
6 files changed, 99 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/pm8150.dtsi b/arch/arm64/boot/dts/qcom/pm8150.dtsi
index 3ba3ba5d8fce..72343fc80a47 100644
--- a/arch/arm64/boot/dts/qcom/pm8150.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8150.dtsi
@@ -12,9 +12,6 @@
/ {
thermal-zones {
pm8150-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8150_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm8150b.dtsi b/arch/arm64/boot/dts/qcom/pm8150b.dtsi
index 1aee3270ce7b..f75e9817de85 100644
--- a/arch/arm64/boot/dts/qcom/pm8150b.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8150b.dtsi
@@ -11,9 +11,6 @@
/ {
thermal-zones {
pm8150b-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8150b_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm8150l.dtsi b/arch/arm64/boot/dts/qcom/pm8150l.dtsi
index ac08a09c64c2..b9e0c8af5c48 100644
--- a/arch/arm64/boot/dts/qcom/pm8150l.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8150l.dtsi
@@ -11,9 +11,6 @@
/ {
thermal-zones {
pm8150l-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8150l_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pmm8155au_1.dtsi b/arch/arm64/boot/dts/qcom/pmm8155au_1.dtsi
index dbd4b91dfe06..a25927eba785 100644
--- a/arch/arm64/boot/dts/qcom/pmm8155au_1.dtsi
+++ b/arch/arm64/boot/dts/qcom/pmm8155au_1.dtsi
@@ -11,9 +11,6 @@
/ {
thermal-zones {
pmm8155au-1-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pmm8155au_1_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pmm8155au_2.dtsi b/arch/arm64/boot/dts/qcom/pmm8155au_2.dtsi
index 1cee20ac2c9c..6c370d0e4938 100644
--- a/arch/arm64/boot/dts/qcom/pmm8155au_2.dtsi
+++ b/arch/arm64/boot/dts/qcom/pmm8155au_2.dtsi
@@ -10,9 +10,6 @@
/ {
thermal-zones {
pmm8155au-2-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pmm8155au_2_temp>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sm8150.dtsi b/arch/arm64/boot/dts/qcom/sm8150.dtsi
index a35c0852b5a1..a2ee7bfe03e1 100644
--- a/arch/arm64/boot/dts/qcom/sm8150.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8150.dtsi
@@ -4556,9 +4556,6 @@ timer {
thermal-zones {
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 1>;
trips {
@@ -4600,9 +4597,6 @@ map1 {
};
cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 2>;
trips {
@@ -4644,9 +4638,6 @@ map1 {
};
cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 3>;
trips {
@@ -4688,9 +4679,6 @@ map1 {
};
cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 4>;
trips {
@@ -4732,9 +4720,6 @@ map1 {
};
cpu4-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -4776,9 +4761,6 @@ map1 {
};
cpu5-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -4820,9 +4802,6 @@ map1 {
};
cpu6-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 9>;
trips {
@@ -4864,9 +4843,6 @@ map1 {
};
cpu7-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 10>;
trips {
@@ -4908,9 +4884,6 @@ map1 {
};
cpu4-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 11>;
trips {
@@ -4952,9 +4925,6 @@ map1 {
};
cpu5-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 12>;
trips {
@@ -4996,9 +4966,6 @@ map1 {
};
cpu6-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 13>;
trips {
@@ -5040,9 +5007,6 @@ map1 {
};
cpu7-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 14>;
trips {
@@ -5084,9 +5048,6 @@ map1 {
};
aoss0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 0>;
trips {
@@ -5099,9 +5060,6 @@ aoss0_alert0: trip-point0 {
};
cluster0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 5>;
trips {
@@ -5119,9 +5077,6 @@ cluster0_crit: cluster0-crit {
};
cluster1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 6>;
trips {
@@ -5139,9 +5094,6 @@ cluster1_crit: cluster1-crit {
};
gpu-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 15>;
cooling-maps {
@@ -5161,9 +5113,6 @@ gpu_top_alert0: trip-point0 {
};
aoss1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 0>;
trips {
@@ -5176,9 +5125,6 @@ aoss1_alert0: trip-point0 {
};
wlan-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 1>;
trips {
@@ -5191,9 +5137,6 @@ wlan_alert0: trip-point0 {
};
video-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 2>;
trips {
@@ -5206,9 +5149,6 @@ video_alert0: trip-point0 {
};
mem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 3>;
trips {
@@ -5221,9 +5161,6 @@ mem_alert0: trip-point0 {
};
q6-hvx-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 4>;
trips {
@@ -5236,9 +5173,6 @@ q6_hvx_alert0: trip-point0 {
};
camera-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 5>;
trips {
@@ -5251,9 +5185,6 @@ camera_alert0: trip-point0 {
};
compute-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 6>;
trips {
@@ -5266,9 +5197,6 @@ compute_alert0: trip-point0 {
};
modem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 7>;
trips {
@@ -5281,9 +5209,6 @@ modem_alert0: trip-point0 {
};
npu-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 8>;
trips {
@@ -5296,9 +5221,6 @@ npu_alert0: trip-point0 {
};
modem-vec-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 9>;
trips {
@@ -5311,9 +5233,6 @@ modem_vec_alert0: trip-point0 {
};
modem-scl-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 10>;
trips {
@@ -5326,9 +5245,6 @@ modem_scl_alert0: trip-point0 {
};
gpu-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 11>;
cooling-maps {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 27/31] arm64: dts: qcom: sm8250-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (25 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 26/31] arm64: dts: qcom: sm8150-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 28/31] arm64: dts: qcom: sm8350-*: " Konrad Dybcio
` (5 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 10 -----
arch/arm64/boot/dts/qcom/sm8250-mtp.dts | 14 ------
arch/arm64/boot/dts/qcom/sm8250.dtsi | 75 --------------------------------
3 files changed, 99 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
index cd0db4f31d4a..ce13fdfb0797 100644
--- a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
+++ b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
@@ -110,8 +110,6 @@ lt9611_3v3: lt9611-3v3 {
thermal-zones {
conn-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm8150b_adc_tm 0>;
trips {
@@ -124,8 +122,6 @@ active-config0 {
};
pm8150l-pcb-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm8150l_adc_tm 1>;
trips {
@@ -138,8 +134,6 @@ active-config0 {
};
skin-msm-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm8150l_adc_tm 0>;
trips {
@@ -152,8 +146,6 @@ active-config0 {
};
wifi-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm8150_adc_tm 1>;
trips {
@@ -166,8 +158,6 @@ active-config0 {
};
xo-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm8150_adc_tm 0>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sm8250-mtp.dts b/arch/arm64/boot/dts/qcom/sm8250-mtp.dts
index 7ef99038cb37..21b2ca1def83 100644
--- a/arch/arm64/boot/dts/qcom/sm8250-mtp.dts
+++ b/arch/arm64/boot/dts/qcom/sm8250-mtp.dts
@@ -53,8 +53,6 @@ chosen {
thermal-zones {
camera-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm8150l_adc_tm 0>;
trips {
@@ -67,8 +65,6 @@ active-config0 {
};
conn-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm8150b_adc_tm 0>;
trips {
@@ -81,8 +77,6 @@ active-config0 {
};
mmw-pa1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm8150_adc_tm 2>;
trips {
@@ -95,8 +89,6 @@ active-config0 {
};
mmw-pa2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm8150l_adc_tm 2>;
trips {
@@ -109,8 +101,6 @@ active-config0 {
};
skin-msm-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm8150l_adc_tm 1>;
trips {
@@ -123,8 +113,6 @@ active-config0 {
};
skin-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm8150_adc_tm 1>;
trips {
@@ -137,8 +125,6 @@ active-config0 {
};
xo-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pm8150_adc_tm 0>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi
index 39bd8f0eba1e..c8d4aaacb00a 100644
--- a/arch/arm64/boot/dts/qcom/sm8250.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi
@@ -6244,9 +6244,6 @@ timer {
thermal-zones {
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 1>;
trips {
@@ -6288,9 +6285,6 @@ map1 {
};
cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 2>;
trips {
@@ -6332,9 +6326,6 @@ map1 {
};
cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 3>;
trips {
@@ -6376,9 +6367,6 @@ map1 {
};
cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 4>;
trips {
@@ -6420,9 +6408,6 @@ map1 {
};
cpu4-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -6464,9 +6449,6 @@ map1 {
};
cpu5-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -6508,9 +6490,6 @@ map1 {
};
cpu6-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 9>;
trips {
@@ -6552,9 +6531,6 @@ map1 {
};
cpu7-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 10>;
trips {
@@ -6596,9 +6572,6 @@ map1 {
};
cpu4-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 11>;
trips {
@@ -6640,9 +6613,6 @@ map1 {
};
cpu5-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 12>;
trips {
@@ -6684,9 +6654,6 @@ map1 {
};
cpu6-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 13>;
trips {
@@ -6728,9 +6695,6 @@ map1 {
};
cpu7-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 14>;
trips {
@@ -6772,9 +6736,6 @@ map1 {
};
aoss0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 0>;
trips {
@@ -6787,9 +6748,6 @@ aoss0_alert0: trip-point0 {
};
cluster0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 5>;
trips {
@@ -6807,9 +6765,6 @@ cluster0_crit: cluster0-crit {
};
cluster1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 6>;
trips {
@@ -6827,9 +6782,6 @@ cluster1_crit: cluster1-crit {
};
gpu-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 15>;
cooling-maps {
@@ -6849,9 +6801,6 @@ gpu_top_alert0: trip-point0 {
};
aoss1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 0>;
trips {
@@ -6864,9 +6813,6 @@ aoss1_alert0: trip-point0 {
};
wlan-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 1>;
trips {
@@ -6879,9 +6825,6 @@ wlan_alert0: trip-point0 {
};
video-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 2>;
trips {
@@ -6894,9 +6837,6 @@ video_alert0: trip-point0 {
};
mem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 3>;
trips {
@@ -6909,9 +6849,6 @@ mem_alert0: trip-point0 {
};
q6-hvx-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 4>;
trips {
@@ -6924,9 +6861,6 @@ q6_hvx_alert0: trip-point0 {
};
camera-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 5>;
trips {
@@ -6939,9 +6873,6 @@ camera_alert0: trip-point0 {
};
compute-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 6>;
trips {
@@ -6954,9 +6885,6 @@ compute_alert0: trip-point0 {
};
npu-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 7>;
trips {
@@ -6969,9 +6897,6 @@ npu_alert0: trip-point0 {
};
gpu-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 8>;
cooling-maps {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 28/31] arm64: dts: qcom: sm8350-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (26 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 27/31] arm64: dts: qcom: sm8250-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:13 ` [PATCH 29/31] arm64: dts: qcom: sm8450-*: " Konrad Dybcio
` (4 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/pm8350.dtsi | 2 -
arch/arm64/boot/dts/qcom/pm8350b.dtsi | 2 -
arch/arm64/boot/dts/qcom/pm8350c.dtsi | 2 -
arch/arm64/boot/dts/qcom/pmr735a.dtsi | 2 -
arch/arm64/boot/dts/qcom/pmr735b.dtsi | 2 -
arch/arm64/boot/dts/qcom/sm8350.dtsi | 87 -----------------------------------
6 files changed, 97 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/pm8350.dtsi b/arch/arm64/boot/dts/qcom/pm8350.dtsi
index 9ed9ba23e81e..20cd4d28b46b 100644
--- a/arch/arm64/boot/dts/qcom/pm8350.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8350.dtsi
@@ -9,8 +9,6 @@
/ {
thermal-zones {
pm8350_thermal: pm8350-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
thermal-sensors = <&pm8350_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm8350b.dtsi b/arch/arm64/boot/dts/qcom/pm8350b.dtsi
index 05c105898892..058f7421ea49 100644
--- a/arch/arm64/boot/dts/qcom/pm8350b.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8350b.dtsi
@@ -9,8 +9,6 @@
/ {
thermal-zones {
pm8350b_thermal: pm8350b-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
thermal-sensors = <&pm8350b_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm8350c.dtsi b/arch/arm64/boot/dts/qcom/pm8350c.dtsi
index aa74e21fe0dc..bc5555b14ca7 100644
--- a/arch/arm64/boot/dts/qcom/pm8350c.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8350c.dtsi
@@ -47,8 +47,6 @@ pm8350c_pwm: pwm {
/ {
thermal-zones {
pm8350c_thermal: pm8350c-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
thermal-sensors = <&pm8350c_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pmr735a.dtsi b/arch/arm64/boot/dts/qcom/pmr735a.dtsi
index febda50779f9..809bee0e164f 100644
--- a/arch/arm64/boot/dts/qcom/pmr735a.dtsi
+++ b/arch/arm64/boot/dts/qcom/pmr735a.dtsi
@@ -35,8 +35,6 @@ pmr735a_gpios: gpio@8800 {
/ {
thermal-zones {
pmr735a_thermal: pmr735a-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
thermal-sensors = <&pmr735a_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pmr735b.dtsi b/arch/arm64/boot/dts/qcom/pmr735b.dtsi
index f7473e247322..9a71b77c7fc4 100644
--- a/arch/arm64/boot/dts/qcom/pmr735b.dtsi
+++ b/arch/arm64/boot/dts/qcom/pmr735b.dtsi
@@ -9,8 +9,6 @@
/ {
thermal-zones {
pmr735b_thermal: pmr735b-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
thermal-sensors = <&pmr735b_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sm8350.dtsi b/arch/arm64/boot/dts/qcom/sm8350.dtsi
index a5e7dbbd8c6c..cf9ed53a834c 100644
--- a/arch/arm64/boot/dts/qcom/sm8350.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8350.dtsi
@@ -3638,9 +3638,6 @@ compute-cb@8 {
thermal_zones: thermal-zones {
cpu0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 1>;
trips {
@@ -3682,9 +3679,6 @@ map1 {
};
cpu1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 2>;
trips {
@@ -3726,9 +3720,6 @@ map1 {
};
cpu2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 3>;
trips {
@@ -3770,9 +3761,6 @@ map1 {
};
cpu3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 4>;
trips {
@@ -3814,9 +3802,6 @@ map1 {
};
cpu4-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 7>;
trips {
@@ -3858,9 +3843,6 @@ map1 {
};
cpu5-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 8>;
trips {
@@ -3902,9 +3884,6 @@ map1 {
};
cpu6-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 9>;
trips {
@@ -3946,9 +3925,6 @@ map1 {
};
cpu7-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 10>;
trips {
@@ -3990,9 +3966,6 @@ map1 {
};
cpu4-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 11>;
trips {
@@ -4034,9 +4007,6 @@ map1 {
};
cpu5-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 12>;
trips {
@@ -4078,9 +4048,6 @@ map1 {
};
cpu6-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 13>;
trips {
@@ -4122,9 +4089,6 @@ map1 {
};
cpu7-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 14>;
trips {
@@ -4166,9 +4130,6 @@ map1 {
};
aoss0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 0>;
trips {
@@ -4181,9 +4142,6 @@ aoss0_alert0: trip-point0 {
};
cluster0-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 5>;
trips {
@@ -4201,9 +4159,6 @@ cluster0_crit: cluster0-crit {
};
cluster1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens0 6>;
trips {
@@ -4221,9 +4176,6 @@ cluster1_crit: cluster1-crit {
};
aoss1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 0>;
trips {
@@ -4236,9 +4188,6 @@ aoss1_alert0: trip-point0 {
};
gpu-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 1>;
cooling-maps {
@@ -4258,9 +4207,6 @@ gpu_top_alert0: trip-point0 {
};
gpu-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 2>;
cooling-maps {
@@ -4280,9 +4226,6 @@ gpu_bottom_alert0: trip-point0 {
};
nspss1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 3>;
trips {
@@ -4295,9 +4238,6 @@ nspss1_alert0: trip-point0 {
};
nspss2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 4>;
trips {
@@ -4310,9 +4250,6 @@ nspss2_alert0: trip-point0 {
};
nspss3-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 5>;
trips {
@@ -4325,9 +4262,6 @@ nspss3_alert0: trip-point0 {
};
video-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 6>;
trips {
@@ -4340,9 +4274,6 @@ video_alert0: trip-point0 {
};
mem-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 7>;
trips {
@@ -4355,9 +4286,6 @@ mem_alert0: trip-point0 {
};
modem1-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 8>;
trips {
@@ -4370,9 +4298,6 @@ modem1_alert0: trip-point0 {
};
modem2-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 9>;
trips {
@@ -4385,9 +4310,6 @@ modem2_alert0: trip-point0 {
};
modem3-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 10>;
trips {
@@ -4400,9 +4322,6 @@ modem3_alert0: trip-point0 {
};
modem4-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 11>;
trips {
@@ -4415,9 +4334,6 @@ modem4_alert0: trip-point0 {
};
camera-top-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 12>;
trips {
@@ -4430,9 +4346,6 @@ camera1_alert0: trip-point0 {
};
cam-bottom-thermal {
- polling-delay-passive = <250>;
- polling-delay = <1000>;
-
thermal-sensors = <&tsens1 13>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 29/31] arm64: dts: qcom: sm8450-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (27 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 28/31] arm64: dts: qcom: sm8350-*: " Konrad Dybcio
@ 2024-03-19 16:13 ` Konrad Dybcio
2024-03-19 16:14 ` [PATCH 30/31] arm64: dts: qcom: sm8550-*: " Konrad Dybcio
` (3 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:13 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/pm8450.dtsi | 3 --
arch/arm64/boot/dts/qcom/sm8450-hdk.dts | 16 ---------
arch/arm64/boot/dts/qcom/sm8450.dtsi | 64 ---------------------------------
3 files changed, 83 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/pm8450.dtsi b/arch/arm64/boot/dts/qcom/pm8450.dtsi
index ae5bce3cf46e..5be83d41f3a2 100644
--- a/arch/arm64/boot/dts/qcom/pm8450.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8450.dtsi
@@ -9,9 +9,6 @@
/ {
thermal-zones {
pm8450-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8450_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sm8450-hdk.dts b/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
index 0786cff07b89..4f17fceaca65 100644
--- a/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
+++ b/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
@@ -136,8 +136,6 @@ pmic_glink_sbu: endpoint {
thermal-zones {
camera-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
thermal-sensors = <&pmk8350_adc_tm 2>;
trips {
@@ -150,8 +148,6 @@ active-config0 {
};
rear-tof-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
thermal-sensors = <&pmk8350_adc_tm 5>;
trips {
@@ -164,8 +160,6 @@ active-config0 {
};
skin-msm-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
thermal-sensors = <&pmk8350_adc_tm 1>;
trips {
@@ -178,8 +172,6 @@ active-config0 {
};
therm1-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
thermal-sensors = <&pmk8350_adc_tm 3>;
trips {
@@ -192,8 +184,6 @@ active-config0 {
};
therm2-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
thermal-sensors = <&pmk8350_adc_tm 6>;
trips {
@@ -206,8 +196,6 @@ active-config0 {
};
usb-conn-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
thermal-sensors = <&pmk8350_adc_tm 7>;
trips {
@@ -220,8 +208,6 @@ active-config0 {
};
wide-rfc-thermal {
- polling-delay-passive = <250>;
- polling-delay = <0>;
thermal-sensors = <&pmk8350_adc_tm 4>;
trips {
@@ -234,8 +220,6 @@ active-config0 {
};
xo-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&pmk8350_adc_tm 0>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sm8450.dtsi b/arch/arm64/boot/dts/qcom/sm8450.dtsi
index b86be34a912b..29c3a74ceb5a 100644
--- a/arch/arm64/boot/dts/qcom/sm8450.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8450.dtsi
@@ -4595,8 +4595,6 @@ sound: sound {
thermal-zones {
aoss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 0>;
trips {
@@ -4615,8 +4613,6 @@ reset-mon-cfg {
};
cpuss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 1>;
trips {
@@ -4635,8 +4631,6 @@ reset-mon-cfg {
};
cpuss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 2>;
trips {
@@ -4655,8 +4649,6 @@ reset-mon-cfg {
};
cpuss3-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 3>;
trips {
@@ -4675,8 +4667,6 @@ reset-mon-cfg {
};
cpuss4-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 4>;
trips {
@@ -4695,8 +4685,6 @@ reset-mon-cfg {
};
cpu4-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 5>;
trips {
@@ -4721,8 +4709,6 @@ cpu4_top_crit: cpu-crit {
};
cpu4-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 6>;
trips {
@@ -4747,8 +4733,6 @@ cpu4_bottom_crit: cpu-crit {
};
cpu5-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 7>;
trips {
@@ -4773,8 +4757,6 @@ cpu5_top_crit: cpu-crit {
};
cpu5-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 8>;
trips {
@@ -4799,8 +4781,6 @@ cpu5_bottom_crit: cpu-crit {
};
cpu6-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 9>;
trips {
@@ -4825,8 +4805,6 @@ cpu6_top_crit: cpu-crit {
};
cpu6-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 10>;
trips {
@@ -4851,8 +4829,6 @@ cpu6_bottom_crit: cpu-crit {
};
cpu7-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 11>;
trips {
@@ -4877,8 +4853,6 @@ cpu7_top_crit: cpu-crit {
};
cpu7-middle-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 12>;
trips {
@@ -4903,8 +4877,6 @@ cpu7_middle_crit: cpu-crit {
};
cpu7-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 13>;
trips {
@@ -4929,8 +4901,6 @@ cpu7_bottom_crit: cpu-crit {
};
gpu-top-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 14>;
cooling-maps {
@@ -4968,8 +4938,6 @@ gpu_top_alert0: trip-point0 {
};
gpu-bottom-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 15>;
cooling-maps {
@@ -5007,8 +4975,6 @@ gpu_bottom_alert0: trip-point0 {
};
aoss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 0>;
trips {
@@ -5027,8 +4993,6 @@ reset-mon-cfg {
};
cpu0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 1>;
trips {
@@ -5053,8 +5017,6 @@ cpu0_crit: cpu-crit {
};
cpu1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 2>;
trips {
@@ -5079,8 +5041,6 @@ cpu1_crit: cpu-crit {
};
cpu2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 3>;
trips {
@@ -5105,8 +5065,6 @@ cpu2_crit: cpu-crit {
};
cpu3-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 4>;
trips {
@@ -5131,8 +5089,6 @@ cpu3_crit: cpu-crit {
};
cdsp0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 5>;
trips {
@@ -5163,8 +5119,6 @@ cdsp_0_config: junction-config {
};
cdsp1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 6>;
trips {
@@ -5195,8 +5149,6 @@ cdsp_1_config: junction-config {
};
cdsp2-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 7>;
trips {
@@ -5227,8 +5179,6 @@ cdsp_2_config: junction-config {
};
video-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 8>;
trips {
@@ -5247,8 +5197,6 @@ reset-mon-cfg {
};
mem-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 9>;
trips {
@@ -5273,8 +5221,6 @@ reset-mon-cfg {
};
modem0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 10>;
trips {
@@ -5305,8 +5251,6 @@ reset-mon-cfg {
};
modem1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 11>;
trips {
@@ -5337,8 +5281,6 @@ reset-mon-cfg {
};
modem2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 12>;
trips {
@@ -5369,8 +5311,6 @@ reset-mon-cfg {
};
modem3-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 13>;
trips {
@@ -5401,8 +5341,6 @@ reset-mon-cfg {
};
camera0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 14>;
trips {
@@ -5421,8 +5359,6 @@ reset-mon-cfg {
};
camera1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 15>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 30/31] arm64: dts: qcom: sm8550-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (28 preceding siblings ...)
2024-03-19 16:13 ` [PATCH 29/31] arm64: dts: qcom: sm8450-*: " Konrad Dybcio
@ 2024-03-19 16:14 ` Konrad Dybcio
2024-03-19 16:14 ` [PATCH 31/31] arm64: dts: qcom: sm8650-*: " Konrad Dybcio
` (2 subsequent siblings)
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:14 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/pm8010.dtsi | 6 ---
arch/arm64/boot/dts/qcom/pm8550.dtsi | 3 --
arch/arm64/boot/dts/qcom/pm8550b.dtsi | 3 --
arch/arm64/boot/dts/qcom/pm8550ve.dtsi | 3 --
arch/arm64/boot/dts/qcom/pm8550vs.dtsi | 12 -----
arch/arm64/boot/dts/qcom/pmr735d_a.dtsi | 3 --
arch/arm64/boot/dts/qcom/pmr735d_b.dtsi | 3 --
arch/arm64/boot/dts/qcom/sm8550.dtsi | 82 ---------------------------------
8 files changed, 115 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/pm8010.dtsi b/arch/arm64/boot/dts/qcom/pm8010.dtsi
index 0ea641e12209..583c3c49d603 100644
--- a/arch/arm64/boot/dts/qcom/pm8010.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8010.dtsi
@@ -9,9 +9,6 @@
/ {
thermal-zones {
pm8010-m-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8010_m_temp_alarm>;
trips {
@@ -30,9 +27,6 @@ trip1 {
};
pm8010-n-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8010_n_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm8550.dtsi b/arch/arm64/boot/dts/qcom/pm8550.dtsi
index 797a18c249a4..27719876a098 100644
--- a/arch/arm64/boot/dts/qcom/pm8550.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8550.dtsi
@@ -9,9 +9,6 @@
/ {
thermal-zones {
pm8550-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8550_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm8550b.dtsi b/arch/arm64/boot/dts/qcom/pm8550b.dtsi
index 72609f31c890..44c0d4cf3fc8 100644
--- a/arch/arm64/boot/dts/qcom/pm8550b.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8550b.dtsi
@@ -9,9 +9,6 @@
/ {
thermal-zones {
pm8550b-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8550b_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm8550ve.dtsi b/arch/arm64/boot/dts/qcom/pm8550ve.dtsi
index 4dc1f03ab2c7..7d13e8d9236b 100644
--- a/arch/arm64/boot/dts/qcom/pm8550ve.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8550ve.dtsi
@@ -9,9 +9,6 @@
/ {
thermal-zones {
pm8550ve-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8550ve_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pm8550vs.dtsi b/arch/arm64/boot/dts/qcom/pm8550vs.dtsi
index 97b1c18aa7d8..a7a85f14c650 100644
--- a/arch/arm64/boot/dts/qcom/pm8550vs.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8550vs.dtsi
@@ -9,9 +9,6 @@
/ {
thermal-zones {
pm8550vs-c-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8550vs_c_temp_alarm>;
trips {
@@ -30,9 +27,6 @@ trip1 {
};
pm8550vs-d-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8550vs_d_temp_alarm>;
trips {
@@ -51,9 +45,6 @@ trip1 {
};
pm8550vs-e-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8550vs_e_temp_alarm>;
trips {
@@ -72,9 +63,6 @@ trip1 {
};
pm8550vs-g-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pm8550vs_g_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pmr735d_a.dtsi b/arch/arm64/boot/dts/qcom/pmr735d_a.dtsi
index 37daaefe3431..e2ffe5d077a6 100644
--- a/arch/arm64/boot/dts/qcom/pmr735d_a.dtsi
+++ b/arch/arm64/boot/dts/qcom/pmr735d_a.dtsi
@@ -9,9 +9,6 @@
/ {
thermal-zones {
pmr735d-k-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pmr735d_k_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/pmr735d_b.dtsi b/arch/arm64/boot/dts/qcom/pmr735d_b.dtsi
index 3b470f6ac46f..e8ececc6eee3 100644
--- a/arch/arm64/boot/dts/qcom/pmr735d_b.dtsi
+++ b/arch/arm64/boot/dts/qcom/pmr735d_b.dtsi
@@ -9,9 +9,6 @@
/ {
thermal-zones {
pmr735d-l-thermal {
- polling-delay-passive = <100>;
- polling-delay = <0>;
-
thermal-sensors = <&pmr735d_l_temp_alarm>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/sm8550.dtsi b/arch/arm64/boot/dts/qcom/sm8550.dtsi
index 3904348075f6..b502471c7c9b 100644
--- a/arch/arm64/boot/dts/qcom/sm8550.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8550.dtsi
@@ -4537,8 +4537,6 @@ compute-cb@8 {
thermal-zones {
aoss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 0>;
trips {
@@ -4557,8 +4555,6 @@ reset-mon-config {
};
cpuss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 1>;
trips {
@@ -4577,8 +4573,6 @@ reset-mon-config {
};
cpuss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 2>;
trips {
@@ -4597,8 +4591,6 @@ reset-mon-config {
};
cpuss2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 3>;
trips {
@@ -4617,8 +4609,6 @@ reset-mon-config {
};
cpuss3-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 4>;
trips {
@@ -4637,8 +4627,6 @@ reset-mon-config {
};
cpu3-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 5>;
trips {
@@ -4663,8 +4651,6 @@ cpu3_top_crit: cpu-critical {
};
cpu3-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 6>;
trips {
@@ -4689,8 +4675,6 @@ cpu3_bottom_crit: cpu-critical {
};
cpu4-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 7>;
trips {
@@ -4715,8 +4699,6 @@ cpu4_top_crit: cpu-critical {
};
cpu4-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 8>;
trips {
@@ -4741,8 +4723,6 @@ cpu4_bottom_crit: cpu-critical {
};
cpu5-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 9>;
trips {
@@ -4767,8 +4747,6 @@ cpu5_top_crit: cpu-critical {
};
cpu5-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 10>;
trips {
@@ -4793,8 +4771,6 @@ cpu5_bottom_crit: cpu-critical {
};
cpu6-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 11>;
trips {
@@ -4819,8 +4795,6 @@ cpu6_top_crit: cpu-critical {
};
cpu6-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 12>;
trips {
@@ -4845,8 +4819,6 @@ cpu6_bottom_crit: cpu-critical {
};
cpu7-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 13>;
trips {
@@ -4871,8 +4843,6 @@ cpu7_top_crit: cpu-critical {
};
cpu7-middle-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 14>;
trips {
@@ -4897,8 +4867,6 @@ cpu7_middle_crit: cpu-critical {
};
cpu7-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 15>;
trips {
@@ -4923,8 +4891,6 @@ cpu7_bottom_crit: cpu-critical {
};
aoss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 0>;
trips {
@@ -4943,8 +4909,6 @@ reset-mon-config {
};
cpu0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 1>;
trips {
@@ -4969,8 +4933,6 @@ cpu0_crit: cpu-critical {
};
cpu1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 2>;
trips {
@@ -4995,8 +4957,6 @@ cpu1_crit: cpu-critical {
};
cpu2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 3>;
trips {
@@ -5021,8 +4981,6 @@ cpu2_crit: cpu-critical {
};
cdsp0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 4>;
trips {
@@ -5053,8 +5011,6 @@ cdsp0_junction_config: junction-config {
};
cdsp1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 5>;
trips {
@@ -5085,8 +5041,6 @@ cdsp1_junction_config: junction-config {
};
cdsp2-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 6>;
trips {
@@ -5117,8 +5071,6 @@ cdsp2_junction_config: junction-config {
};
cdsp3-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 7>;
trips {
@@ -5149,8 +5101,6 @@ cdsp3_junction_config: junction-config {
};
video-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 8>;
trips {
@@ -5169,8 +5119,6 @@ reset-mon-config {
};
mem-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 9>;
trips {
@@ -5195,8 +5143,6 @@ reset-mon-config {
};
modem0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 10>;
trips {
@@ -5227,8 +5173,6 @@ reset-mon-config {
};
modem1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 11>;
trips {
@@ -5259,8 +5203,6 @@ reset-mon-config {
};
modem2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 12>;
trips {
@@ -5291,8 +5233,6 @@ reset-mon-config {
};
modem3-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 13>;
trips {
@@ -5323,8 +5263,6 @@ reset-mon-config {
};
camera0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 14>;
trips {
@@ -5343,8 +5281,6 @@ reset-mon-config {
};
camera1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 15>;
trips {
@@ -5363,8 +5299,6 @@ reset-mon-config {
};
aoss2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 0>;
trips {
@@ -5383,8 +5317,6 @@ reset-mon-config {
};
gpuss-0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 1>;
cooling-maps {
@@ -5422,8 +5354,6 @@ gpu0_junction_config: junction-config {
};
gpuss-1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 2>;
cooling-maps {
@@ -5461,8 +5391,6 @@ gpu1_junction_config: junction-config {
};
gpuss-2-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 3>;
cooling-maps {
@@ -5500,8 +5428,6 @@ gpu2_junction_config: junction-config {
};
gpuss-3-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 4>;
cooling-maps {
@@ -5539,8 +5465,6 @@ gpu3_junction_config: junction-config {
};
gpuss-4-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 5>;
cooling-maps {
@@ -5578,8 +5502,6 @@ gpu4_junction_config: junction-config {
};
gpuss-5-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 6>;
cooling-maps {
@@ -5617,8 +5539,6 @@ gpu5_junction_config: junction-config {
};
gpuss-6-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 7>;
cooling-maps {
@@ -5656,8 +5576,6 @@ gpu6_junction_config: junction-config {
};
gpuss-7-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 8>;
cooling-maps {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* [PATCH 31/31] arm64: dts: qcom: sm8650-*: Remove thermal zone polling delays
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (29 preceding siblings ...)
2024-03-19 16:14 ` [PATCH 30/31] arm64: dts: qcom: sm8550-*: " Konrad Dybcio
@ 2024-03-19 16:14 ` Konrad Dybcio
2024-03-19 16:43 ` [PATCH 00/31] Clean up thermal zone polling-delay Johan Hovold
2024-03-21 17:25 ` Daniel Lezcano
32 siblings, 0 replies; 35+ messages in thread
From: Konrad Dybcio @ 2024-03-19 16:14 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
All of the thermal zone suppliers are interrupt-driven, remove the
bogus and unnecessary polling that only wastes CPU time.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/sm8650.dtsi | 88 ------------------------------------
1 file changed, 88 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sm8650.dtsi b/arch/arm64/boot/dts/qcom/sm8650.dtsi
index ba72d8f38420..f7707c155696 100644
--- a/arch/arm64/boot/dts/qcom/sm8650.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8650.dtsi
@@ -5091,8 +5091,6 @@ compute-cb@8 {
thermal-zones {
aoss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 0>;
trips {
@@ -5111,8 +5109,6 @@ aoss0-critical {
};
cpuss0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 1>;
trips {
@@ -5131,8 +5127,6 @@ cpuss0-critical {
};
cpuss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 2>;
trips {
@@ -5151,8 +5145,6 @@ cpuss1-critical {
};
cpuss2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 3>;
trips {
@@ -5171,8 +5163,6 @@ cpuss2-critical {
};
cpuss3-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 4>;
trips {
@@ -5191,8 +5181,6 @@ cpuss3-critical {
};
cpu2-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 5>;
trips {
@@ -5217,8 +5205,6 @@ cpu2-critical {
};
cpu2-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 6>;
trips {
@@ -5243,8 +5229,6 @@ cpu2-critical {
};
cpu3-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 7>;
trips {
@@ -5269,8 +5253,6 @@ cpu3-critical {
};
cpu3-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 8>;
trips {
@@ -5295,8 +5277,6 @@ cpu3-critical {
};
cpu4-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 9>;
trips {
@@ -5321,8 +5301,6 @@ cpu4-critical {
};
cpu4-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 10>;
trips {
@@ -5347,8 +5325,6 @@ cpu4-critical {
};
cpu5-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 11>;
trips {
@@ -5373,8 +5349,6 @@ cpu5-critical {
};
cpu5-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 12>;
trips {
@@ -5399,8 +5373,6 @@ cpu5-critical {
};
cpu6-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 13>;
trips {
@@ -5425,8 +5397,6 @@ cpu6-critical {
};
cpu6-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens0 14>;
trips {
@@ -5451,8 +5421,6 @@ cpu6-critical {
};
aoss1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 0>;
trips {
@@ -5471,8 +5439,6 @@ aoss1-critical {
};
cpu7-top-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 1>;
trips {
@@ -5497,8 +5463,6 @@ cpu7-critical {
};
cpu7-middle-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 2>;
trips {
@@ -5523,8 +5487,6 @@ cpu7-critical {
};
cpu7-bottom-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 3>;
trips {
@@ -5549,8 +5511,6 @@ cpu7-critical {
};
cpu0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 4>;
trips {
@@ -5575,8 +5535,6 @@ cpu0-critical {
};
cpu1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 5>;
trips {
@@ -5601,8 +5559,6 @@ cpu1-critical {
};
nsphvx0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 6>;
trips {
@@ -5621,8 +5577,6 @@ nsphvx1-critical {
};
nsphvx1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 7>;
trips {
@@ -5641,8 +5595,6 @@ nsphvx1-critical {
};
nsphmx0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 8>;
trips {
@@ -5661,8 +5613,6 @@ nsphmx0-critical {
};
nsphmx1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 9>;
trips {
@@ -5681,8 +5631,6 @@ nsphmx1-critical {
};
nsphmx2-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 10>;
trips {
@@ -5701,8 +5649,6 @@ nsphmx2-critical {
};
nsphmx3-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 11>;
trips {
@@ -5721,8 +5667,6 @@ nsphmx3-critical {
};
video-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 12>;
trips {
@@ -5741,8 +5685,6 @@ video-critical {
};
ddr-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 13>;
trips {
@@ -5761,8 +5703,6 @@ ddr-critical {
};
camera0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 14>;
trips {
@@ -5781,8 +5721,6 @@ camera0-critical {
};
camera1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens1 15>;
trips {
@@ -5801,8 +5739,6 @@ camera1-critical {
};
aoss2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 0>;
trips {
@@ -5821,8 +5757,6 @@ aoss2-critical {
};
gpuss0-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 1>;
trips {
@@ -5841,8 +5775,6 @@ gpuss0-critical {
};
gpuss1-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 2>;
trips {
@@ -5861,8 +5793,6 @@ gpuss1-critical {
};
gpuss2-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 3>;
trips {
@@ -5881,8 +5811,6 @@ gpuss2-critical {
};
gpuss3-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 4>;
trips {
@@ -5901,8 +5829,6 @@ gpuss3-critical {
};
gpuss4-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 5>;
trips {
@@ -5921,8 +5847,6 @@ gpuss4-critical {
};
gpuss5-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 6>;
trips {
@@ -5941,8 +5865,6 @@ gpuss5-critical {
};
gpuss6-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 7>;
trips {
@@ -5961,8 +5883,6 @@ gpuss6-critical {
};
gpuss7-thermal {
- polling-delay-passive = <10>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 8>;
trips {
@@ -5981,8 +5901,6 @@ gpuss7-critical {
};
modem0-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 9>;
trips {
@@ -6001,8 +5919,6 @@ modem0-critical {
};
modem1-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 10>;
trips {
@@ -6021,8 +5937,6 @@ modem1-critical {
};
modem2-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 11>;
trips {
@@ -6041,8 +5955,6 @@ modem2-critical {
};
modem3-thermal {
- polling-delay-passive = <0>;
- polling-delay = <0>;
thermal-sensors = <&tsens2 12>;
trips {
--
2.40.1
^ permalink raw reply related [flat|nested] 35+ messages in thread* Re: [PATCH 00/31] Clean up thermal zone polling-delay
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (30 preceding siblings ...)
2024-03-19 16:14 ` [PATCH 31/31] arm64: dts: qcom: sm8650-*: " Konrad Dybcio
@ 2024-03-19 16:43 ` Johan Hovold
2024-03-21 17:25 ` Daniel Lezcano
32 siblings, 0 replies; 35+ messages in thread
From: Johan Hovold @ 2024-03-19 16:43 UTC (permalink / raw)
To: Konrad Dybcio
Cc: Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
cros-qcom-dts-watchers, linux-arm-msm, devicetree, linux-kernel
On Tue, Mar 19, 2024 at 05:13:30PM +0100, Konrad Dybcio wrote:
> A trivial follow-up on the changes introduced in Commit 488164006a28
> ("thermal/of: Assume polling-delay(-passive) 0 when absent").
>
> Should probably wait until v6.9-rc1 so that the patch in question is
> in the base tree, otherwise TZs will fail to register.
>
> FWIW, Compile-tested only (except 8280).
Please provide some more details on how you verified these patches (or
that patch).
Last time you said you just checked that the interrupt counters were
incremented, which is not sufficient.
Johan
^ permalink raw reply [flat|nested] 35+ messages in thread* Re: [PATCH 00/31] Clean up thermal zone polling-delay
2024-03-19 16:13 [PATCH 00/31] Clean up thermal zone polling-delay Konrad Dybcio
` (31 preceding siblings ...)
2024-03-19 16:43 ` [PATCH 00/31] Clean up thermal zone polling-delay Johan Hovold
@ 2024-03-21 17:25 ` Daniel Lezcano
32 siblings, 0 replies; 35+ messages in thread
From: Daniel Lezcano @ 2024-03-21 17:25 UTC (permalink / raw)
To: Konrad Dybcio, Bjorn Andersson, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, cros-qcom-dts-watchers
Cc: linux-arm-msm, devicetree, linux-kernel
On 19/03/2024 17:13, Konrad Dybcio wrote:
> A trivial follow-up on the changes introduced in Commit 488164006a28
> ("thermal/of: Assume polling-delay(-passive) 0 when absent").
>
> Should probably wait until v6.9-rc1 so that the patch in question is
> in the base tree, otherwise TZs will fail to register.
>
> FWIW, Compile-tested only (except 8280).
>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---
Are you sure these changes are correct. They are not related to commit
488164006a28.
If the sensor has interrupt support, then it can specify:
polling-delay = <0>;
As a zero polling value can be omitted in the DT then it can be removed.
Then when a trip point is crossed, the interrupt fires but then it must
sample the temperature of the thermal zone to do the mitigation.
I doubt polling-delay-passive must be removed. The changes you
introduced just disable the mitigation and that will lead to board wild
reboots.
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
^ permalink raw reply [flat|nested] 35+ messages in thread