arm64: dts: ti: k3-am64: Enable ECAP nodes at the board level
authorAndrew Davis <afd@ti.com>
Mon, 17 Oct 2022 19:25:27 +0000 (14:25 -0500)
committerNishanth Menon <nm@ti.com>
Fri, 28 Oct 2022 13:14:48 +0000 (08:14 -0500)
ECAP nodes defined in the top-level AM64x SoC dtsi files are incomplete
and will not be functional unless they are extended with pinmux
information. (These and the EPWM nodes could be used to trigger internal
actions but they are not used like that currently)

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

Disable the ECAP nodes 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-6-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 672575f44529b11322d2656e3830de75cb5eb47c..ef1833f65bdc612d9d9e8736c9c50a1f36b1f5cb 100644 (file)
                power-domains = <&k3_pds 51 TI_SCI_PD_EXCLUSIVE>;
                clocks = <&k3_clks 51 0>;
                clock-names = "fck";
+               status = "disabled";
        };
 
        ecap1: pwm@23110000 {
                power-domains = <&k3_pds 52 TI_SCI_PD_EXCLUSIVE>;
                clocks = <&k3_clks 52 0>;
                clock-names = "fck";
+               status = "disabled";
        };
 
        ecap2: pwm@23120000 {
                power-domains = <&k3_pds 53 TI_SCI_PD_EXCLUSIVE>;
                clocks = <&k3_clks 53 0>;
                clock-names = "fck";
+               status = "disabled";
        };
 
        main_rti0: watchdog@e000000 {
index cef3afa10c39b81103dd1996e2151b6c5874a062..43d50ecfb211f83db83eb8fef7edf998d6a5d184 100644 (file)
 };
 
 &ecap0 {
+       status = "okay";
        /* PWM is available on Pin 1 of header J12 */
        pinctrl-names = "default";
        pinctrl-0 = <&main_ecap0_pins_default>;
 };
 
-&ecap1 {
-       status = "disabled";
-};
-
-&ecap2 {
-       status = "disabled";
-};
-
 &icssg0_mdio {
        status = "disabled";
 };
index 51f4ae165c13d5e795a94a094487858f27917d2c..8b9987ccdc1bc1a67bf5bb5c7c4326fafa4e0a61 100644 (file)
 };
 
 &ecap0 {
+       status = "okay";
        /* PWM is available on Pin 1 of header J3 */
        pinctrl-names = "default";
        pinctrl-0 = <&main_ecap0_pins_default>;
 };
 
-&ecap1 {
-       status = "disabled";
-};
-
-&ecap2 {
-       status = "disabled";
-};
-
 &icssg0_mdio {
        status = "disabled";
 };