mediatek: fix DTS defining mt7530 switch phys but not referencing them
authorDaniel Golle <daniel@makrotopia.org>
Fri, 12 Apr 2024 12:24:38 +0000 (13:24 +0100)
committerDaniel Golle <daniel@makrotopia.org>
Sat, 13 Apr 2024 17:49:30 +0000 (18:49 +0100)
commiteb13076e77c412cfe4dd6de48138507ea6f3c8b3
tree89d63b114aa60a7028b6e425a351f7d05c3c0e34
parentc08f8a4a4913d80ed3fb261dadc711e94b8428dd
mediatek: fix DTS defining mt7530 switch phys but not referencing them

The upstream solution to define the MDIO bus in DT is a bit
more strict than our previous downstream solution doing the same thing
and now requires switch PHYs to be referenced in DT as well.

Arınç Ünal told us in #15141:
"With [the now upstream patch written by him which we backported], the
switch MDIO bus won't be assigned to ds->user_mii_bus when the switch
MDIO bus is defined on the device tree anymore. This was not the case
with the downstream patch.

When ds->user_mii_bus is populated, DSA will 1:1 map the port with
PHY. Meaning port with address 1 will be mapped to PHY with address 1.
Because that ds->user_mii_bus is not populated when the switch MDIO
bus is defined on the device tree, on every port node, the PHY address
must be supplied by the phy-handle property."

Add those phy-handles to affected devices' DT.

Fixes: 4354b34f6f ("generic: 6.6: sync mt7530 DSA driver with upstream")
Fixes: 401a6ccfaf ("generic: 6.1: sync mt7530 DSA driver with upstream")
Signed-off-by: Daniel Golle <daniel@makrotopia.org>
target/linux/mediatek/dts/mt7986a-acer-predator-w6.dts
target/linux/mediatek/dts/mt7986a-asus-tuf-ax4200.dts
target/linux/mediatek/dts/mt7986a-asus-tuf-ax6000.dts