lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20240319-topic-msm-polling-cleanup-v1-15-e0aee1dbcd78@linaro.org>
Date: Tue, 19 Mar 2024 17:13:45 +0100
From: Konrad Dybcio <konrad.dybcio@...aro.org>
To: Bjorn Andersson <andersson@...nel.org>, Rob Herring <robh@...nel.org>, 
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>, 
 Conor Dooley <conor+dt@...nel.org>, cros-qcom-dts-watchers@...omium.org
Cc: linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org, 
 linux-kernel@...r.kernel.org, Konrad Dybcio <konrad.dybcio@...aro.org>
Subject: [PATCH 15/31] arm64: dts: qcom: sa8775p-*: Remove thermal zone
 polling delays

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@...aro.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@...40000 {
 
 	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


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ