Commit faee0fe2 authored by Marco Franchi's avatar Marco Franchi Committed by Shawn Guo

ARM: dts: imx53-tx53: Fix 'ethernet-phy@0' node with unit name and no reg property

The following build warning is seen with W=1:

Warning (unit_address_vs_reg): Node /soc/aips@60000000/ethernet@63fec000/ethernet-phy@0 has a unit name, but no reg property

Fix this warning by adding reg property from such node and, consequently,
includes a mdio node into the fec node, with the 'address-cells' and
'size-cells' to avoid these other following warnings:

Warning (reg_format): "reg" property in /soc/aips@60000000/ethernet@63fec000/ethernet-phy@0 has invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
Warning (avoid_default_addr_size): Relying on default #address-cells value for /soc/aips@60000000/ethernet@63fec000/ethernet-phy@0
Warning (avoid_default_addr_size): Relying on default #size-cells value for /soc/aips@60000000/ethernet@63fec000/ethernet-phy@0
Signed-off-by: default avatarMarco Franchi <marco.franchi@nxp.com>
Tested-by: default avatarLothar Waßmann <LW@karo-electronics.de>
Acked-by: default avatarLothar Waßmann <LW@karo-electronics.de>
Signed-off-by: default avatarShawn Guo <shawnguo@kernel.org>
parent 85f8ee78
...@@ -225,10 +225,16 @@ &fec { ...@@ -225,10 +225,16 @@ &fec {
mac-address = [000000000000]; /* placeholder; will be overwritten by bootloader */ mac-address = [000000000000]; /* placeholder; will be overwritten by bootloader */
status = "okay"; status = "okay";
phy0: ethernet-phy@0 { mdio {
interrupt-parent = <&gpio2>; #address-cells = <1>;
interrupts = <4 IRQ_TYPE_EDGE_FALLING>; #size-cells = <0>;
device_type = "ethernet-phy";
phy0: ethernet-phy@0 {
reg = <0>;
interrupt-parent = <&gpio2>;
interrupts = <4 IRQ_TYPE_EDGE_FALLING>;
device_type = "ethernet-phy";
};
}; };
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment