From: Konrad Dybcio <konrad.dybcio@linaro.org>
To: Bjorn Andersson <andersson@kernel.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Conor Dooley <conor+dt@kernel.org>,
cros-qcom-dts-watchers@chromium.org
Cc: linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
Konrad Dybcio <konrad.dybcio@linaro.org>
Subject: [PATCH 16/31] arm64: dts: qcom: sc7180-*: Remove thermal zone polling delays
Date: Tue, 19 Mar 2024 17:13:46 +0100 [thread overview]
Message-ID: <20240319-topic-msm-polling-cleanup-v1-16-e0aee1dbcd78@linaro.org> (raw)
In-Reply-To: <20240319-topic-msm-polling-cleanup-v1-0-e0aee1dbcd78@linaro.org>
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
next prev parent reply other threads:[~2024-03-19 16:13 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
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 ` [PATCH 03/31] arm64: dts: qcom: ipq9574-*: " Konrad Dybcio
2024-03-20 10:03 ` Varadarajan Narayanan
2024-03-19 16:13 ` [PATCH 04/31] arm64: dts: qcom: msm8916-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 05/31] arm64: dts: qcom: msm8939-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 06/31] arm64: dts: qcom: msm8953-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 07/31] arm64: dts: qcom: msm8976-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 08/31] arm64: dts: qcom: msm8996-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 09/31] arm64: dts: qcom: msm8998-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 10/31] arm64: dts: qcom: pm7550ba: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 11/31] arm64: dts: qcom: pms405: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 12/31] arm64: dts: qcom: pmx75: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 13/31] arm64: dts: qcom: qcm2290-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 14/31] arm64: dts: qcom: qcs404-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 15/31] arm64: dts: qcom: sa8775p-*: " Konrad Dybcio
2024-03-19 16:13 ` Konrad Dybcio [this message]
2024-03-19 16:13 ` [PATCH 17/31] arm64: dts: qcom: sc7280-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 18/31] arm64: dts: qcom: sc8180x-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 19/31] arm64: dts: qcom: sc8280xp-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 20/31] arm64: dts: qcom: sdm660-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 21/31] arm64: dts: qcom: sdm845-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 22/31] arm64: dts: qcom: sm6115-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 23/31] arm64: dts: qcom: sm6125-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 24/31] arm64: dts: qcom: sm6350-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 25/31] arm64: dts: qcom: sm6375-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 26/31] arm64: dts: qcom: sm8150-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 27/31] arm64: dts: qcom: sm8250-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 28/31] arm64: dts: qcom: sm8350-*: " Konrad Dybcio
2024-03-19 16:13 ` [PATCH 29/31] arm64: dts: qcom: sm8450-*: " Konrad Dybcio
2024-03-19 16:14 ` [PATCH 30/31] arm64: dts: qcom: sm8550-*: " Konrad Dybcio
2024-03-19 16:14 ` [PATCH 31/31] arm64: dts: qcom: sm8650-*: " 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
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240319-topic-msm-polling-cleanup-v1-16-e0aee1dbcd78@linaro.org \
--to=konrad.dybcio@linaro.org \
--cc=andersson@kernel.org \
--cc=conor+dt@kernel.org \
--cc=cros-qcom-dts-watchers@chromium.org \
--cc=devicetree@vger.kernel.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®