Re: [PATCH 4/5] arm64: dts: qcom: nord: Add GPU cooling

[Shawn Guo]

On Fri, Oct 09, 2026 at 02:36:38AM +0530, Akhil P Oommen wrote:
> From: Aastha Pandey <aastha.pandey@xxxxxxxxxxxxxxxx>
> 
> Unlike the CPU, the GPU does not throttle its speed automatically when
> it reaches high temperatures.
> 
> Set up GPU cooling on Nord by throttling the Adreno 753 GPU when it
> reaches 105°C. Label the GPU thermal zones and their passive trip
> points, map them to the GPU as a cooling device, and add a passive
> polling delay so the thermal framework can monitor the zones while
> throttling.
> 
> Signed-off-by: Aastha Pandey <aastha.pandey@xxxxxxxxxxxxxxxx>
> Signed-off-by: Akhil P Oommen <akhilpo@xxxxxxxxxxxxxxxx>
> ---
>  arch/arm64/boot/dts/qcom/nord-embedded.dtsi | 177 ++++++++++++++++++++++++++++
>  arch/arm64/boot/dts/qcom/nord.dtsi          |  80 ++++++-------
>  2 files changed, 217 insertions(+), 40 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/nord-embedded.dtsi b/arch/arm64/boot/dts/qcom/nord-embedded.dtsi
> index 4379092b7bf8..23370007c974 100644
> --- a/arch/arm64/boot/dts/qcom/nord-embedded.dtsi
> +++ b/arch/arm64/boot/dts/qcom/nord-embedded.dtsi
> @@ -17,6 +17,7 @@
>  #include <dt-bindings/interconnect/qcom,nord-rpmh.h>
>  #include <dt-bindings/power/qcom,rpmhpd.h>
>  #include <dt-bindings/soc/qcom,rpmh-rsc.h>
> +#include <dt-bindings/thermal/thermal.h>
>  
>  #include "nord.dtsi"
>  
> @@ -2042,3 +2043,179 @@ opp-403000000 {
>  		};
>  	};
>  };
> +
> +&gpu_0_0_0_thermal {
> +	polling-delay-passive = <10>;
> +
> +	cooling-maps {
> +		map0 {
> +			trip = <&gpu_0_0_0_alert0>;
> +			cooling-device = <&gpu_0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
> +		};
> +	};
> +};

I'm not sure why this is Embedded variant specific. Can we not just
update nord.dtsi as needed? Would that break GearVM variant in any way?

Shawn