| Message ID | 20240319-topic-msm-polling-cleanup-v1-3-e0aee1dbcd78@linaro.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Clean up thermal zone polling-delay | expand |
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 > >
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 {
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(-)