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

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: <Zfq0jF93iJVgd1+R@hu-varada-blr.qualcomm.com>
Date: Wed, 20 Mar 2024 15:33:56 +0530
From: Varadarajan Narayanan <quic_varada@...cinc.com>
To: Konrad Dybcio <konrad.dybcio@...aro.org>
CC: 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>, <linux-arm-msm@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 03/31] arm64: dts: qcom: ipq9574-*: Remove thermal zone
 polling delays

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

Reviewed-by: Varadarajan Narayanan <quic_varada@...cinc.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@...8000 {
>  
>  	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
> 
>