Skip to content

Commit 44ad820

Browse files
Rafał Miłeckiffainelli
authored andcommitted
ARM: dts: BCM53573: Fix Ethernet info for Luxul devices
Both Luxul's XAP devices (XAP-810 and XAP-1440) are access points that use a non-default design. They don't include switch but have a single Ethernet port and BCM54210E PHY connected to the Ethernet controller's MDIO bus. Support for those devices regressed due to two changes: 1. Describing MDIO bus with switch After commit 9fb90ae ("ARM: dts: BCM53573: Describe on-SoC BCM53125 rev 4 switch") Linux stopped probing for MDIO devices. 2. Dropping hardcoded BCM54210E delays In commit fea7fda ("net: phy: broadcom: Fix RGMII delays configuration for BCM54210E") support for other PHY modes was added but that requires a proper "phy-mode" value in DT. Both above changes are correct (they don't need to be reverted or anything) but they need this fix for DT data to be correct and for Linux to work properly. Fixes: 9fb90ae ("ARM: dts: BCM53573: Describe on-SoC BCM53125 rev 4 switch") Signed-off-by: Rafał Miłecki <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Florian Fainelli <[email protected]>
1 parent 91994e5 commit 44ad820

File tree

2 files changed

+26
-0
lines changed

2 files changed

+26
-0
lines changed

arch/arm/boot/dts/broadcom/bcm47189-luxul-xap-1440.dts

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,3 +45,16 @@
4545
};
4646
};
4747
};
48+
49+
&gmac0 {
50+
phy-mode = "rgmii";
51+
phy-handle = <&bcm54210e>;
52+
53+
mdio {
54+
/delete-node/ switch@1e;
55+
56+
bcm54210e: ethernet-phy@0 {
57+
reg = <0>;
58+
};
59+
};
60+
};

arch/arm/boot/dts/broadcom/bcm47189-luxul-xap-810.dts

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,3 +81,16 @@
8181
};
8282
};
8383
};
84+
85+
&gmac0 {
86+
phy-mode = "rgmii";
87+
phy-handle = <&bcm54210e>;
88+
89+
mdio {
90+
/delete-node/ switch@1e;
91+
92+
bcm54210e: ethernet-phy@0 {
93+
reg = <0>;
94+
};
95+
};
96+
};

0 commit comments

Comments
 (0)