Re: [PATCH 03/31] arm64: dts: qcom: ipq9574-*: Remove thermal zone polling delays

[Varadarajan Narayanan]

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@xxxxxxxxxx>
> ---
>  arch/arm64/boot/dts/qcom/ipq9574.dtsi | 26 --------------------------
>  1 file changed, 26 deletions(-)

Reviewed-by: Varadarajan Narayanan <quic_varada@xxxxxxxxxxx>

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
> 
>