Re: [PATCH v6 9/9] arm64: dts: qcom: hamoa: Enable CDSP cooling

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] The commit message claims to set up CDSP cooling for 'both instances', but the patch actually configures 4 NSP thermal zones mapping to a single CDSP instance.
--

--- Patch [9]: [PATCH v6 9/9] arm64: dts: qcom: hamoa: Enable CDSP cooling ---
commit 11f46654cec2615152b5d3c181397f2d61caf2f7
Author: Dipa Ramesh Mantre <dipa.mantre@oss.qualcomm.com>

arm64: dts: qcom: hamoa: Enable CDSP cooling

This commit enables CDSP cooling for the hamoa platform. It configures
a polling delay for passive cooling and maps thermal zones to the CDSP
cooling device to throttle it when high temperatures are reached.

> Set up CDSP cooling for both instances by throttling the cdsp, when
> it reaches 95°C.

[Severity: Medium]
This isn't a bug, but there appears to be a mismatch between the commit
message and the actual code changes. The description mentions setting up
cooling for "both instances", which implies two instances. 

However, the patch actually configures four NSP thermal zones (nsp0 through
nsp3) that all map to a single remoteproc_cdsp node in
arch/arm64/boot/dts/qcom/hamoa.dtsi.

Could the commit message be updated to accurately reflect the correct
number of instances and thermal zones being configured?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260727-b4-qmi-tmd-v6-0-973cd3a226af@oss.qualcomm.com?part=9