diff mbox series

[v2,3/3] ARM64: dts: freescale: ZII: Add missing phy-mode

Message ID 20230408152801.2336041-4-andrew@lunn.ch (mailing list archive)
State Not Applicable
Headers show
Series Add missing DSA properties for marvell switches | expand

Checks

Context Check Description
netdev/tree_selection success Not a local patch

Commit Message

Andrew Lunn April 8, 2023, 3:28 p.m. UTC
The DSA framework has got more picky about always having a phy-mode
for the CPU port. The imx8mq Ethernet is being configured to RMII. Set
the switch phy-mode based on this.

Additionally, the cpu label has never actually been used in the
binding, so remove it.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
---
v2: Use rev-rmii for the side 'playing PHY'
---
 arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Vladimir Oltean April 11, 2023, 11:40 a.m. UTC | #1
On Sat, Apr 08, 2023 at 05:28:01PM +0200, Andrew Lunn wrote:
> The DSA framework has got more picky about always having a phy-mode
> for the CPU port. The imx8mq Ethernet is being configured to RMII. Set
> the switch phy-mode based on this.
> 
> Additionally, the cpu label has never actually been used in the
> binding, so remove it.
> 
> Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> ---
> v2: Use rev-rmii for the side 'playing PHY'
> ---

Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi
index 3a52679ecd68..cb777b47baf9 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi
@@ -177,7 +177,7 @@  port@1 {
 
 				port@2 {
 					reg = <2>;
-					label = "cpu";
+					phy-mode = "rev-rmii";
 					ethernet = <&fec1>;
 
 					fixed-link {