This shows the path we could take with this, deprecating the weird external bus thing. I don't know what to do about the irq lines with a pointless type flag that should be onecell:ed. I need proper schema checking to add LED support to the Marvell switch. Just how it is, it can't go on like this. Andrew: if you have lots of ideas and want to do lots of changes, feel free to just take over the patch set and do what you like, this is an RFC after all. Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> --- Linus Walleij (3): ARM: dts: marvell: Fix some common switch mistakes RFC: dt-bindings: marvell: Rewrite in schema RFC: net: dsa: mv88e6xxx: Register mdio-external .../bindings/net/dsa/marvell,mv88e6xxx.yaml | 249 +++++++++++++++++++++ .../devicetree/bindings/net/dsa/marvell.txt | 109 --------- MAINTAINERS | 2 +- arch/arm/boot/dts/marvell/armada-370-rd.dts | 2 - .../dts/marvell/armada-381-netgear-gs110emx.dts | 2 - .../dts/marvell/armada-385-clearfog-gtr-l8.dts | 2 +- .../dts/marvell/armada-385-clearfog-gtr-s4.dts | 2 +- arch/arm/boot/dts/marvell/armada-385-linksys.dtsi | 2 - arch/arm/boot/dts/marvell/armada-388-clearfog.dts | 2 - .../boot/dts/marvell/armada-xp-linksys-mamba.dts | 2 - drivers/net/dsa/mv88e6xxx/chip.c | 16 +- 11 files changed, 267 insertions(+), 123 deletions(-) --- base-commit: 69d714c69583c4387147d0b7f2f436d42baddadd change-id: 20231008-marvell-88e6152-wan-led-88c43b7fd2fd Best regards, -- Linus Walleij <linus.walleij@xxxxxxxxxx>