MIPS: mscc: ocelot: mark the phy-mode for internal PHY ports
authorVladimir Oltean <vladimir.oltean@nxp.com>
Thu, 19 Aug 2021 17:04:16 +0000 (20:04 +0300)
committerThomas Bogendoerfer <tsbogend@alpha.franken.de>
Sat, 21 Aug 2021 08:38:48 +0000 (10:38 +0200)
The ocelot driver was converted to phylink, and that expects a valid
phy_interface_t. Without a phy-mode, of_get_phy_mode returns
PHY_INTERFACE_MODE_NA, which is not ideal because phylink rejects that.

The ocelot driver was patched to treat PHY_INTERFACE_MODE_NA as
PHY_INTERFACE_MODE_INTERNAL to work with the broken DT blobs, but we
should fix the device trees and specify the phy-mode too.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
arch/mips/boot/dts/mscc/ocelot_pcb120.dts
arch/mips/boot/dts/mscc/ocelot_pcb123.dts

index d2dc6b3..bd24069 100644 (file)
 &port0 {
        status = "okay";
        phy-handle = <&phy0>;
+       phy-mode = "internal";
 };
 
 &port1 {
        status = "okay";
        phy-handle = <&phy1>;
+       phy-mode = "internal";
 };
 
 &port2 {
        status = "okay";
        phy-handle = <&phy2>;
+       phy-mode = "internal";
 };
 
 &port3 {
        status = "okay";
        phy-handle = <&phy3>;
+       phy-mode = "internal";
 };
 
 &port4 {
index 7d7e638..0185045 100644 (file)
 &port0 {
        status = "okay";
        phy-handle = <&phy0>;
+       phy-mode = "internal";
 };
 
 &port1 {
        status = "okay";
        phy-handle = <&phy1>;
+       phy-mode = "internal";
 };
 
 &port2 {
        status = "okay";
        phy-handle = <&phy2>;
+       phy-mode = "internal";
 };
 
 &port3 {
        status = "okay";
        phy-handle = <&phy3>;
+       phy-mode = "internal";
 };