Re: [PATCH v3 3/3] arm64: dts: freescale: add Nodebox v3 CPU Module

[Frank Li]

On Tue, Sep 08, 2026 at 04:12:45AM +0200, Vincent Jardin via B4 Relay wrote:
> From: Vincent Jardin <vjardin@xxxxxxx>
>
> The Nodebox v3 is a Free Mobile CPU module built around the NXP
> LX2160A (16x Cortex-A72, DPAA2).
>
> The same .dtsi is used by the U-Boot port, so both Linux and
> u-boot should not drift.

Needn't this paragraph.

>
> Signed-off-by: Vincent Jardin <vjardin@xxxxxxx>
> ---
>  arch/arm64/boot/dts/freescale/Makefile             |   2 +
>  .../arm64/boot/dts/freescale/fsl-lx2160a-nbxv3.dts |  34 +++
>  .../boot/dts/freescale/fsl-lx2160a-nbxv3.dtsi      | 335 +++++++++++++++++++++
>  3 files changed, 371 insertions(+)
>
...
> +
> +&gpio1 {
> +	status = "okay";
> +	gpio-line-names =
> +		"", "", "", "", "", "",
> +		"cfg-rcw-src2", "cfg-rcw-src3",
> +		"reset-req-b",
> +		"altfn:sdhc2-clk", "altfn:sdhc2-ds",
> +		"altfn:sdhc2-dat0", "altfn:sdhc2-dat1",
> +		"altfn:sdhc2-dat2", "altfn:sdhc2-dat3",
> +		"altfn:sdhc2-dat4", "altfn:sdhc2-dat5",
> +		"altfn:sdhc2-dat6", "altfn:sdhc2-dat7",
> +		"altfn:sdhc2-cmd",
> +		"cfg-svr1",
> +		"altfn:xspi1-cs0", "altfn:xspi1-sck",
> +		"altfn:xspi1-dqs",
> +		"altfn:xspi1-data0", "altfn:xspi1-data1",
> +		"", "",
> +		"flash-reset#", "pltrst#", "", "ethsw-reset#";
> +
> +	pltrst-hog {
> +		gpio-hog;
> +		gpios = <29 GPIO_ACTIVE_LOW>;
> +		output-low;
> +		line-name = "pltrst#";
> +	};
> +
> +	/* Boot NOR reset, released: electrical high */
> +	flash-reset-hog {
> +		gpio-hog;
> +		gpios = <28 GPIO_ACTIVE_LOW>;
> +		output-low;
> +		line-name = "flash-reset#";
> +	};
> +
> +	/* Ethernet switch reset, released: electrical high */
> +	ethsw-reset-hog {
> +		gpio-hog;
> +		gpios = <31 GPIO_ACTIVE_LOW>;
> +		output-low;
> +		line-name = "ethsw-reset#";
> +	};

Does related devices have "reset" property to descript reset?

> +};
> +
> +&gpio2 {
> +	status = "okay";
> +	gpio-line-names =
> +		"lan8023-mdint",
> +		"i2c2-alert#", "i2c4-alert#", "i2c6-alert#",
> +		"u800-tca9539-int#",
> +		"spi3-alert#",
> +		"wake0", "wake1",
> +		"carrier-hot#",
> +		"u1300-rtl8211fd-int#",
> +		"ethsw-int#",
> +		"", "",
> +		"u1700-zl30733-clk-gpio0", "u1700-zl30733-clk-gpio1",
> +		"u1700-zl30733-clk-gpio2", "u1700-zl30733-clk-gpio3",
> +		"", "", "", "", "", "", "", "",
> +		"", "", "", "", "", "", "";
> +};
> +
> +&gpio3 {
> +	status = "okay";
> +	gpio-line-names =
> +		"altfn:ec1-txd3", "altfn:ec1-txd2",
> +		"altfn:ec1-txd1", "altfn:ec1-txd0",
> +		"altfn:ec1-tx-en", "altfn:ec1-gtx-clk",
> +		"altfn:ec1-rxd3", "altfn:ec1-rxd2",
> +		"altfn:ec1-rxd1", "altfn:ec1-rxd0",
> +		"altfn:ec1-rx-clk", "altfn:ec1-rx-dv",
> +		"", "",
> +		"ptp-refclk-in",
> +		"", "", "", "", "", "", "", "",
> +		"u1700-zl30733-pps-out",
> +		"",
> +		"ethsw-mdio-en",
> +		"usb01-oc#",	/* USB1_PWRFAULT and USB2_PWRFAULT share one net */
> +		"",
> +		"usb01-oc#",
> +		"altfn:spi3-sck",
> +		"", "";
> +
> +	/* Low enables the carrier-side EMDIO2 level translator */
> +	ethsw-mdio-en-hog {
> +		gpio-hog;
> +		gpios = <25 GPIO_ACTIVE_LOW>;
> +		output-high;
> +		line-name = "ethsw-mdio-en";

The same here, most EN module as regualtor.

Frank