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-16-e0aee1dbcd78@linaro.org>
Date: Tue, 19 Mar 2024 17:13:46 +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 16/31] arm64: dts: qcom: sc7180-*: 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/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@...00000 {
 
 	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


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ