Re: [PATCH 2/2] arm64: dts: qcom: sc8280xp: Add more thermal zones

From: Konrad Dybcio

Date: Fri May 15 2026 - 07:17:20 EST


On 5/7/26 4:34 PM, Xilin Wu wrote:
> The SC8280XP contains a lot of TSENS sensors which are not yet described.
> Add the thermal zones for them.
>
> Signed-off-by: Xilin Wu <sophon@xxxxxxxxx>
> ---

Very nice to see, just a couple nits:

tsens0 11 -> NSP1-0
tsens1 11 -> NSP1-2
tsens2 10 -> NSP1-1

cluster0/1-thermal are a little inaccurate, let's rename them to
cpuss0/1 as they are not really bound to either cluster

and please change mem-N -> ddrss-N

Konrad