Re: [PATCH v2 2/2] ARM: dts: zynq: Add ALINX AX7Z020B development board

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] ARM: dts: zynq: unaddressed mdio child in gem0 triggers dtc W=1
  warning
--

commit 3898a396db65d191993e3c2e496f86fb25c55e4f
Author: Pufan Jin <cyan2254650260@gmail.com>

ARM: dts: zynq: Add ALINX AX7Z020B development board

This commit adds devicetree support for the ALINX AC7Z020 system on module
and the ALINX AX7Z020B development board.

Link: https://www.alinx.com/

> diff --git a/arch/arm/boot/dts/xilinx/zynq-alinx-ax7z020b.dts b/arch/arm/boot/dts/xilinx/zynq-alinx-ax7z020b.dts
> new file mode 100644
> index 0000000000000..472d9fcaafa69
> --- /dev/null
> +++ b/arch/arm/boot/dts/xilinx/zynq-alinx-ax7z020b.dts

[ ... ]

> +&gem0 {
> +	status = "okay";
> +	phy-mode = "rgmii-id";
> +	phy-handle = <&ethernet_phy>;
> +
> +	mdio {
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +
> +		ethernet_phy: ethernet-phy@1 {
> +			reg = <1>;

[Severity: Low]
Does wrapping ethernet-phy@1 in an unaddressed mdio node trigger a dtc
avoid_unnecessary_addr_size warning when building with W=1?

gem0 in zynq-7000.dtsi defines #address-cells and #size-cells, so dtc warns
when its only direct child has no reg or ranges property.  Placing
ethernet-phy@1 directly under &gem0, as other Zynq-7000 boards do, or
deleting #address-cells and #size-cells on &gem0 avoids the warning.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261111162917.97532-1-cyan2254650260@gmail.com?part=2