Re: [PATCH v6 5/6] arm64: dts: qcom: Add device tree for Nord Ride board

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] Unused SCMI channel 15 enabled in device tree
--

--- Patch [5]: [PATCH v6 5/6] arm64: dts: qcom: Add device tree for Nord Ride board ---
commit e4e282e1a5c3a7d85da20a185bf57e0515d81e29
Author: Deepti Jaggi <deepti.jaggi@oss.qualcomm.com>

arm64: dts: qcom: Add device tree for Nord Ride board

Add initial device tree for the Qualcomm SA8797P Ride reference board,
which is built on Nord GearVM variant. Configures UART, UFS storage, 
and thermal zones.

> diff --git a/arch/arm64/boot/dts/qcom/nord-ride.dts b/arch/arm64/boot/dts/qcom/nord-ride.dts
> new file mode 100644
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/nord-ride.dts

[ ... ]

> +&scmi3 {
> +	status = "okay";
> +};
> +
> +&scmi11 {
> +	status = "okay";
> +};
> +
> +&scmi15 {
> +	status = "okay";
> +};

[Severity: Low]
Is scmi15 needed here? It doesn't look like there are any devices in this 
board or included dtsi files that consume its protocols. 

Enabling it will cause the kernel to map shared memory and issue SMC calls 
to discover protocols (like power domains, DVFS, resets) that have no 
consumers, which wastes a small amount of memory and boot time.

> +
> +&scmi23 {
> +	status = "okay";
> +};

[ ... ]

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260716142605.2398433-1-shengchao.guo@oss.qualcomm.com?part=5