Received-SPF: pass (google.com: domain of linux-kernel+bounces-108726-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Wed, 20 Mar 2024 15:33:56 +0530
From: Varadarajan Narayanan <quic_varada@quicinc.com>
To: Konrad Dybcio <konrad.dybcio@linaro.org>
CC: 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>, <linux-arm-msm@vger.kernel.org>,
        <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 03/31] arm64: dts: qcom: ipq9574-*: Remove thermal zone
 polling delays
Message-ID: <Zfq0jF93iJVgd1+R@hu-varada-blr.qualcomm.com>
References: <20240319-topic-msm-polling-cleanup-v1-0-e0aee1dbcd78@linaro.org>
 <20240319-topic-msm-polling-cleanup-v1-3-e0aee1dbcd78@linaro.org>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20240319-topic-msm-polling-cleanup-v1-3-e0aee1dbcd78@linaro.org>

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