Re: [PATCH v9 2/3] ARM64: dts: marvell: Fix some common switch mistakes

Gregory CLEMENT
2023-12-08T15:12:50Z

Linus Walleij <linus.walleij@linaro.org> writes:

> Fix some errors in the Marvell MV88E6xxx switch descriptions:
> - The top node had no address size or cells.
> - switch0@0 is not OK, should be ethernet-switch@0.
> - ports should be ethernet-ports
> - port@0 should be ethernet-port@0
> - PHYs should be named ethernet-phy@
>
> Reviewed-by: Andrew Lunn <andrew@lunn.ch>
> Reviewed-by: Florian Fainelli <florian.fainelli@broadcom.com>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

Applied on mvebu/dt64

Thanks,

Gregory

> ---
> .../dts/marvell/armada-3720-espressobin-ultra.dts | 14 ++--
> .../boot/dts/marvell/armada-3720-espressobin.dtsi | 20 +++--
> .../boot/dts/marvell/armada-3720-gl-mv1000.dts | 20 +++--
> .../boot/dts/marvell/armada-3720-turris-mox.dts | 85 ++++++++++++----------
> .../boot/dts/marvell/armada-7040-mochabin.dts | 24 +++---
> .../dts/marvell/armada-8040-clearfog-gt-8k.dts | 22 +++---
> arch/arm64/boot/dts/marvell/cn9130-crb.dtsi | 42 +++++------
> 7 files changed, 115 insertions(+), 112 deletions(-)

--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering