Re: [PATCH v9 6/6] arm64: dts: qcom: Add device tree for Nord RRD board

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [High] The UFS reset GPIO index 181 requested in `nord-rrd.dts` is out of bounds with respect to the `gpio-ranges` defined in the SoC base DT.
--

--- Patch [6]: [PATCH v9 6/6] arm64: dts: qcom: Add device tree for Nord RRD board ---
commit 7d4cf30013817c741d9f964a48bd5626ffbd7ee5
Author: Shawn Guo <shengchao.guo@oss.qualcomm.com>

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

This commit adds the initial device tree for the Qualcomm Dragonwing IQ10
RRD board. It enables various hardware components including the debug UART,
UFS storage, PMICs, I2C, and SPI.

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

[ ... ]

> +&ufs_mem_hc {
> +	reset-gpios = <&tlmm 181 GPIO_ACTIVE_LOW>;

[Severity: High]
Is this GPIO index correct for the UFS reset? The requested index is 181,
but it appears the base SoC device tree (nord.dtsi) only maps 181 GPIOs
(indices 0 through 180) via its gpio-ranges property: 
gpio-ranges = <&tlmm 0 0 181>;

Will this cause the UFS driver to fail claiming its reset GPIO during probe,
preventing the primary storage from initializing during boot?

> +
> +	vcc-supply = <&vreg_l3i_2p5>;
> +	vcc-max-microamp = <1300000>;
> +	vccq-supply = <&vreg_l2i_1p2>;
> +	vccq-max-microamp = <1200000>;
> +
> +	status = "okay";
> +};

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