From fa3646c0033eb93d6397b55c1a4e921dff1049f8 Mon Sep 17 00:00:00 2001 From: Robert Marko Date: Sun, 31 Oct 2021 11:38:31 +0100 Subject: [PATCH] ipq806x: ecw5410: drop GPIO based MDIO1 node 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 --- .../arm/boot/dts/qcom-ipq8068-ecw5410.dts | 29 +++++++------------ 1 file changed, 10 insertions(+), 19 deletions(-) diff --git a/target/linux/ipq806x/files/arch/arm/boot/dts/qcom-ipq8068-ecw5410.dts b/target/linux/ipq806x/files/arch/arm/boot/dts/qcom-ipq8068-ecw5410.dts index 43bed451cb..57724c19d7 100644 --- a/target/linux/ipq806x/files/arch/arm/boot/dts/qcom-ipq8068-ecw5410.dts +++ b/target/linux/ipq806x/files/arch/arm/boot/dts/qcom-ipq8068-ecw5410.dts @@ -34,7 +34,6 @@ aliases { serial1 = &gsbi1_serial; - mdio-gpio0 = &mdio0; ethernet0 = &gmac3; ethernet1 = &gmac2; @@ -293,26 +292,18 @@ }; }; -&soc { - mdio1: mdio { - compatible = "virtual,mdio-gpio"; - #address-cells = <1>; - #size-cells = <0>; +&mdio0 { + status = "okay"; - status = "okay"; + pinctrl-0 = <&mdio0_pins>; + pinctrl-names = "default"; - pinctrl-0 = <&mdio0_pins>; - pinctrl-names = "default"; + phy0: ethernet-phy@0 { + reg = <0>; + }; - gpios = <&qcom_pinmux 1 GPIO_ACTIVE_HIGH &qcom_pinmux 0 GPIO_ACTIVE_HIGH>; - - phy0: ethernet-phy@0 { - reg = <0>; - }; - - phy1: ethernet-phy@1 { - reg = <1>; - }; + phy1: ethernet-phy@1 { + reg = <1>; }; }; @@ -330,7 +321,7 @@ status = "okay"; qcom,id = <3>; - mdiobus = <&mdio1>; + mdiobus = <&mdio0>; phy-mode = "sgmii"; phy-handle = <&phy0>;