arm64: dts: ti: k3-am64: Enable MDIO nodes at the board level
authorAndrew Davis <afd@ti.com>
Mon, 17 Oct 2022 19:25:30 +0000 (14:25 -0500)
committerNishanth Menon <nm@ti.com>
Fri, 28 Oct 2022 13:14:48 +0000 (08:14 -0500)
MDIO nodes defined in the top-level AM64x SoC dtsi files are incomplete
and will not be functional unless they are extended with a pinmux.

As the attached PHY is only known about at the board integration level,
these nodes should only be enabled when provided with this information.

Disable the MDIO nodes (in both CPSW and ICSSG) in the dtsi files and
only enable the ones that are actually pinned out on a given board.

Signed-off-by: Andrew Davis <afd@ti.com>
Signed-off-by: Nishanth Menon <nm@ti.com>
Reviewed-by: Bryan Brattlof <bb@ti.com>
Acked-by: Vignesh Raghavendra <vigneshr@ti.com>
Link: https://lore.kernel.org/r/20221017192532.23825-9-afd@ti.com
arch/arm64/boot/dts/ti/k3-am64-main.dtsi
arch/arm64/boot/dts/ti/k3-am642-evm.dts
arch/arm64/boot/dts/ti/k3-am642-sk.dts

index b193f88..7ea07f7 100644 (file)
                        clocks = <&k3_clks 13 0>;
                        clock-names = "fck";
                        bus_freq = <1000000>;
+                       status = "disabled";
                };
 
                cpts@3d000 {
                        #address-cells = <1>;
                        #size-cells = <0>;
                        bus_freq = <1000000>;
+                       status = "disabled";
                };
        };
 
                        clocks = <&k3_clks 82 0>;
                        clock-names = "fck";
                        bus_freq = <1000000>;
+                       status = "disabled";
                };
        };
 
index 2319ba4..9a3bf85 100644 (file)
 };
 
 &cpsw3g_mdio {
+       status = "okay";
        pinctrl-names = "default";
        pinctrl-0 = <&mdio1_pins_default>;
 
        pinctrl-0 = <&main_ecap0_pins_default>;
 };
 
-&icssg0_mdio {
-       status = "disabled";
-};
-
-&icssg1_mdio {
-       status = "disabled";
-};
-
 &main_mcan0 {
        pinctrl-names = "default";
        pinctrl-0 = <&main_mcan0_pins_default>;
index c1d6346..451e148 100644 (file)
 };
 
 &cpsw3g_mdio {
+       status = "okay";
        pinctrl-names = "default";
        pinctrl-0 = <&mdio1_pins_default>;
 
        pinctrl-0 = <&main_ecap0_pins_default>;
 };
 
-&icssg0_mdio {
-       status = "disabled";
-};
-
-&icssg1_mdio {
-       status = "disabled";
-};
-
 &main_mcan0 {
        status = "disabled";
 };