Re: [PATCH net-next v7 3/7] ARM: dts: marvell: Fix some common switch mistakes

From: Florian Fainelli
Date: Tue Oct 24 2023 - 12:35:58 EST


On 10/24/23 06:20, Linus Walleij wrote:
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.
- The ports node should be named ethernet-ports
- The ethernet-ports node should have port@0 etc children, no
plural "ports" in the children.
- Ports should be named ethernet-port@0 etc
- PHYs should be named ethernet-phy@0 etc

This serves as an example of fixes needed for introducing a
schema for the bindings, but the patch can simply be applied.

Reviewed-by: Andrew Lunn <andrew@xxxxxxx>
Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>

Reviewed-by: Florian Fainelli <florian.fainelli@xxxxxxxxxxxx>
--
Florian