ipq806x: ecw5410: drop GPIO based MDIO1 node
authorRobert Marko <robert.marko@sartura.hr>
Sun, 31 Oct 2021 10:38:31 +0000 (11:38 +0100)
committerAdrian Schmutzler <freifunk@adrianschmutzler.de>
Sun, 31 Oct 2021 20:24:47 +0000 (21:24 +0100)
It looks like this is a leftover before there was a proper MDIO driver.
Since both PHY-s are connected to the HW MDIO bus there is no reason for
this to exist anymore, especially since it uses the same pins as the HW
controller and has the pinmux for the set to "MDIO" so this worked by
pure luck as GPIO MDIO would probe first and override the HW driver.

Move the GMAC3 to simply use the same MDIO bus phandle.

Signed-off-by: Robert Marko <robert.marko@sartura.hr>
target/linux/ipq806x/files/arch/arm/boot/dts/qcom-ipq8068-ecw5410.dts

index 43bed451cbfb08816b2956f946576370325200c6..57724c19d731d441cc2cd2a6968916cfdb91b92c 100644 (file)
@@ -34,7 +34,6 @@
 
        aliases {
                serial1 = &gsbi1_serial;
-               mdio-gpio0 = &mdio0;
                ethernet0 = &gmac3;
                ethernet1 = &gmac2;
 
        };
 };
 
-&soc {
-       mdio1: mdio {
-               compatible = "virtual,mdio-gpio";
-               #address-cells = <1>;
-               #size-cells = <0>;
-
-               status = "okay";
-
-               pinctrl-0 = <&mdio0_pins>;
-               pinctrl-names = "default";
+&mdio0 {
+       status = "okay";
 
-               gpios = <&qcom_pinmux 1 GPIO_ACTIVE_HIGH &qcom_pinmux 0 GPIO_ACTIVE_HIGH>;
+       pinctrl-0 = <&mdio0_pins>;
+       pinctrl-names = "default";
 
-               phy0: ethernet-phy@0 {
-                       reg = <0>;
-               };
+       phy0: ethernet-phy@0 {
+               reg = <0>;
+       };
 
-               phy1: ethernet-phy@1 {
-                       reg = <1>;
-               };
+       phy1: ethernet-phy@1 {
+               reg = <1>;
        };
 };
 
        status = "okay";
 
        qcom,id = <3>;
-       mdiobus = <&mdio1>;
+       mdiobus = <&mdio0>;
 
        phy-mode = "sgmii";
        phy-handle = <&phy0>;