[PATCH v4 0/2] Add support for ICSSM Ethernet on AM57x, AM437x, and AM335x

Hi,

This series adds support for ICSSM Ethernet on Texas Instruments AM57x,
AM437x and AM335x platforms.

The AM57x and AM437x IDKs support two PRU-ICSS instances, each consisting
of two PRU cores, with each PRU-ICSS instance capable of handling two
Ethernet ports. For the AM57x platforms, the PRU-ICSS2 node has been added
to the am57xx-idk-common.dtsi, while for the AM437x platform, the PRU-ICSS1
node has been added to the am437x-idk-evm.dts.

The AM335x ICE features a single PRU-ICSS instance. A new device tree overlay
source file, am335x-icev2-prueth-overlay.dtso, has been introduced to define the
PRU-ICSS node for the AM335x platform.

This is v4 of the patch series [v1]. It has no changes from [v3].
This series is based on the latest next-20260105 linux-next.

Changes from v3 to v4 :

*) No code changes were made, only the version was updated.
*) Rebased the series on latest linux-next.

Changes from v2 to v3 :

*) Addressed Andrew Davis's comment by placing PRUETH nodes in a new overlay file
am335x-icev2-prueth-overlay.dtso.
*) Rebased the series on latest linux-next.

Changes from v1 to v2 :

*) Addressed Andrew Lunn's comment on patch 1 of the series.
*) Addressed MD Danish Anwar comment on patch 1 of the series.
*) Rebased the series on latest linux-next.

[v1] https://lore.kernel.org/all/20251013125401.1435486-1-parvathi@couthit.com/
[v2] https://lore.kernel.org/all/20251103124820.1679167-1-parvathi@couthit.com/
[v3] https://lore.kernel.org/all/20251217130715.1327138-1-parvathi@couthit.com/

Thanks and Regards,
Parvathi.

Roger Quadros (2):
  arm: dts: ti: Adds device tree nodes for PRU Cores, IEP and eCAP
    modules of PRU-ICSS2 Instance.
  arm: dts: ti: Adds support for AM335x and AM437x

 arch/arm/boot/dts/ti/omap/Makefile            |   5 +
 .../ti/omap/am335x-icev2-prueth-overlay.dtso  | 190 ++++++++++++++++++
 arch/arm/boot/dts/ti/omap/am33xx-l4.dtsi      |  11 +
 arch/arm/boot/dts/ti/omap/am4372.dtsi         |  11 +
 arch/arm/boot/dts/ti/omap/am437x-idk-evm.dts  | 137 ++++++++++++-
 arch/arm/boot/dts/ti/omap/am57-pruss.dtsi     |  11 +
 arch/arm/boot/dts/ti/omap/am571x-idk.dts      |   8 +-
 arch/arm/boot/dts/ti/omap/am572x-idk.dts      |  10 +-
 arch/arm/boot/dts/ti/omap/am574x-idk.dts      |  10 +-
 .../boot/dts/ti/omap/am57xx-idk-common.dtsi   |  61 ++++++
 10 files changed, 444 insertions(+), 10 deletions(-)
 create mode 100644 arch/arm/boot/dts/ti/omap/am335x-icev2-prueth-overlay.dtso

-- 
2.43.0

From: Roger Quadros <redacted>

The TI Sitara AM57xx series of devices consists of 2 PRU-ICSS instances
(PRU-ICSS1 and PRU-ICSS2). This patch adds the device tree nodes for the
PRU-ICSS2 instance to support DUAL-MAC mode of operation.

Each PRU-ICSS instance consists of two PRU cores along with various
peripherals such as the Interrupt Controller (PRU_INTC), the Industrial
Ethernet Peripheral(IEP), the Real Time Media Independent Interface
controller (MII_RT), and the Enhanced Capture (eCAP) event module.

am57-pruss.dtsi - Adds IEP and eCAP peripheral as child nodes of
the PRUSS subsystem node.

am57xx-idk-common.dtsi - Adds PRU-ICSS2 instance node along with
PRU eth port information and corresponding port configuration. It includes
interrupt mapping for packet reception, HW timestamp collection, and
PRU Ethernet ports in MII mode.

am571x-idk.dts, am572x-idk.dts and am574x-idk.dts - GPIO configuration
along with delay configuration for individual PRU Ethernet port.

Signed-off-by: Roger Quadros <redacted>
Signed-off-by: Andrew F. Davis <redacted>
Signed-off-by: Murali Karicheri <redacted>
Signed-off-by: Basharath Hussain Khaja <redacted>
Signed-off-by: Parvathi Pudi <parvathi@couthit.com>
---
 arch/arm/boot/dts/ti/omap/am57-pruss.dtsi     | 11 ++++
 arch/arm/boot/dts/ti/omap/am571x-idk.dts      |  8 ++-
 arch/arm/boot/dts/ti/omap/am572x-idk.dts      | 10 +--
 arch/arm/boot/dts/ti/omap/am574x-idk.dts      | 10 +--
 .../boot/dts/ti/omap/am57xx-idk-common.dtsi   | 61 +++++++++++++++++++
 5 files changed, 91 insertions(+), 9 deletions(-)

diff --git a/arch/arm/boot/dts/ti/omap/am57-pruss.dtsi b/arch/arm/boot/dts/ti/omap/am57-pruss.dtsi
index 46c5383f0eee..f73316625608 100644
--- a/arch/arm/boot/dts/ti/omap/am57-pruss.dtsi
+++ b/arch/arm/boot/dts/ti/omap/am57-pruss.dtsi

@@ -170,6 +170,17 @@ pruss2_iepclk_mux: iepclk-mux@30 {
 				};
 			};
 
+			pruss2_iep: iep@2e000 {
+				compatible = "ti,am5728-icss-iep";
+				reg = <0x2e000 0x31c>;
+				clocks = <&pruss2_iepclk_mux>;
+			};
+
+			pruss2_ecap: ecap@30000 {
+				compatible = "ti,pruss-ecap";
+				reg = <0x30000 0x60>;
+			};
+
 			pruss2_mii_rt: mii-rt@32000 {
 				compatible = "ti,pruss-mii", "syscon";
 				reg = <0x32000 0x58>;

diff --git a/arch/arm/boot/dts/ti/omap/am571x-idk.dts b/arch/arm/boot/dts/ti/omap/am571x-idk.dts
index 322cf79d22e9..02653b440585 100644
--- a/arch/arm/boot/dts/ti/omap/am571x-idk.dts
+++ b/arch/arm/boot/dts/ti/omap/am571x-idk.dts

@@ -214,5 +214,11 @@ &pruss1_mdio {
 };
 
 &pruss2_mdio {
-	status = "disabled";
+	reset-gpios = <&gpio5 9 GPIO_ACTIVE_LOW>;
+	reset-delay-us = <2>;   /* PHY datasheet states 1uS min */
+};
+
+&pruss2_eth {
+	ti,pruss-gp-mux-sel = <4>,      /* MII2, needed for PRUSS1_MII0 */
+			      <4>;      /* MII2, needed for PRUSS1_MII1 */
 };

diff --git a/arch/arm/boot/dts/ti/omap/am572x-idk.dts b/arch/arm/boot/dts/ti/omap/am572x-idk.dts
index 94a738cb0a4d..54a8ccb9ca14 100644
--- a/arch/arm/boot/dts/ti/omap/am572x-idk.dts
+++ b/arch/arm/boot/dts/ti/omap/am572x-idk.dts

@@ -28,10 +28,12 @@ &mmc2 {
 	pinctrl-2 = <&mmc2_pins_ddr_rev20>;
 };
 
-&pruss1_mdio {
-	status = "disabled";
+&pruss2_eth0_phy {
+	reset-gpios = <&gpio5 8 GPIO_ACTIVE_LOW>;
+	reset-assert-us = <2>;   /* PHY datasheet states 1uS min */
 };
 
-&pruss2_mdio {
-	status = "disabled";
+&pruss2_eth1_phy {
+	reset-gpios = <&gpio5 9 GPIO_ACTIVE_LOW>;
+	reset-assert-us = <2>;   /* PHY datasheet states 1uS min */
 };

diff --git a/arch/arm/boot/dts/ti/omap/am574x-idk.dts b/arch/arm/boot/dts/ti/omap/am574x-idk.dts
index 47b9174d2353..47b6c6cb210c 100644
--- a/arch/arm/boot/dts/ti/omap/am574x-idk.dts
+++ b/arch/arm/boot/dts/ti/omap/am574x-idk.dts

@@ -40,10 +40,12 @@ &emif1 {
 	status = "okay";
 };
 
-&pruss1_mdio {
-	status = "disabled";
+&pruss2_eth0_phy {
+	reset-gpios = <&gpio5 8 GPIO_ACTIVE_LOW>;
+	reset-assert-us = <2>;   /* PHY datasheet states 1uS min */
 };
 
-&pruss2_mdio {
-	status = "disabled";
+&pruss2_eth1_phy {
+	reset-gpios = <&gpio5 9 GPIO_ACTIVE_LOW>;
+	reset-assert-us = <2>;   /* PHY datasheet states 1uS min */
 };

diff --git a/arch/arm/boot/dts/ti/omap/am57xx-idk-common.dtsi b/arch/arm/boot/dts/ti/omap/am57xx-idk-common.dtsi
index 43e3623f079c..5eccff3bb4b6 100644
--- a/arch/arm/boot/dts/ti/omap/am57xx-idk-common.dtsi
+++ b/arch/arm/boot/dts/ti/omap/am57xx-idk-common.dtsi

@@ -155,6 +155,52 @@ src_clk_x1: src_clk_x1 {
 		compatible = "fixed-clock";
 		clock-frequency = <20000000>;
 	};
+
+	/* Dual-MAC Ethernet application node on PRU-ICSS2 */
+	pruss2_eth: pruss2-eth {
+		compatible = "ti,am57-prueth";
+		ti,prus = <&pru2_0>, <&pru2_1>;
+		sram = <&ocmcram1>;
+		ti,mii-rt = <&pruss2_mii_rt>;
+		ti,iep = <&pruss2_iep>;
+		ti,ecap = <&pruss2_ecap>;
+		interrupts = <20 2 2>, <21 3 3>;
+		interrupt-names = "rx_hp", "rx_lp";
+		interrupt-parent = <&pruss2_intc>;
+
+		ethernet-ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			pruss2_emac0: ethernet-port@0 {
+				reg = <0>;
+				phy-handle = <&pruss2_eth0_phy>;
+				phy-mode = "mii";
+				interrupts = <20 2 2>, <26 6 6>, <23 6 6>;
+				interrupt-names = "rx", "emac_ptp_tx",
+						  "hsr_ptp_tx";
+				/* Filled in by bootloader */
+				local-mac-address = [00 00 00 00 00 00];
+			};
+
+			pruss2_emac1: ethernet-port@1 {
+				reg = <1>;
+				phy-handle = <&pruss2_eth1_phy>;
+				phy-mode = "mii";
+				interrupts = <21 3 3>, <27 9 7>, <24 9 7>;
+				interrupt-names = "rx", "emac_ptp_tx",
+						  "hsr_ptp_tx";
+				/* Filled in by bootloader */
+				local-mac-address = [00 00 00 00 00 00];
+			};
+		};
+	};
+
+};
+
+&pruss2_iep {
+	interrupt-parent = <&pruss2_intc>;
+	interrupts = <7 7 8>;
+	interrupt-names = "iep_cap_cmp";
 };
 
 &dra7_pmx_core {

@@ -606,3 +652,18 @@ dpi_out: endpoint {
 		};
 	};
 };
+
+&pruss2_mdio {
+	status = "okay";
+	pruss2_eth0_phy: ethernet-phy@0 {
+		reg = <0>;
+		interrupt-parent = <&gpio3>;
+		interrupts = <30 IRQ_TYPE_LEVEL_LOW>;
+	};
+
+	pruss2_eth1_phy: ethernet-phy@1 {
+		reg = <1>;
+		interrupt-parent = <&gpio3>;
+		interrupts = <31 IRQ_TYPE_LEVEL_LOW>;
+	};
+};

-- 
2.43.0

From: Roger Quadros <redacted>

PRU-ICSS instance consists of two PRU cores along with various
peripherals such as the Interrupt Controller (PRU_INTC), the Industrial
Ethernet Peripheral(IEP), the Real Time Media Independent Interface
controller (MII_RT), and the Enhanced Capture (eCAP) event module.

The TI Sitara AM335x ICE-V2 consists of single PRU-ICSS instance,
This patch adds the new device tree overlay file in-order to enable
PRU-ICSS instance, along with makefile changes.

The TI Sitara AM437x series of devices consists of 2 PRU-ICSS instances
(PRU-ICSS0 and PRU-ICSS1). This patch adds the device tree nodes for the
PRU-ICSS1 instance to support DUAL-MAC mode of operation. Support for
Ethernet over PRU is available only for ICSS1 instance.

am33xx-l4.dtsi, am4372.dtsi - Adds IEP and eCAP peripheral as child nodes
of the PRUSS subsystem node.

am335x-icev2-prueth.dtso, am437x-idk-evm.dts - Adds PRU-ICSS
instance node along with PRU eth port information and corresponding
port configuration. It includes interrupt mapping for packet reception,
HW timestamp collection, and PRU Ethernet ports in MII mode,

GPIO configuration, boot strapping along with delay configuration for
individual PRU Ethernet port and other required nodes.

Signed-off-by: Roger Quadros <redacted>
Signed-off-by: Andrew F. Davis <redacted>
Signed-off-by: Murali Karicheri <redacted>
Signed-off-by: Basharath Hussain Khaja <redacted>
Signed-off-by: Parvathi Pudi <parvathi@couthit.com>
---
 arch/arm/boot/dts/ti/omap/Makefile            |   5 +
 .../ti/omap/am335x-icev2-prueth-overlay.dtso  | 190 ++++++++++++++++++
 arch/arm/boot/dts/ti/omap/am33xx-l4.dtsi      |  11 +
 arch/arm/boot/dts/ti/omap/am4372.dtsi         |  11 +
 arch/arm/boot/dts/ti/omap/am437x-idk-evm.dts  | 137 ++++++++++++-
 5 files changed, 353 insertions(+), 1 deletion(-)
 create mode 100644 arch/arm/boot/dts/ti/omap/am335x-icev2-prueth-overlay.dtso

diff --git a/arch/arm/boot/dts/ti/omap/Makefile b/arch/arm/boot/dts/ti/omap/Makefile
index 14e500846875..c68948035eca 100644
--- a/arch/arm/boot/dts/ti/omap/Makefile
+++ b/arch/arm/boot/dts/ti/omap/Makefile

@@ -82,6 +82,10 @@ dtb-$(CONFIG_ARCH_OMAP4) += \
 	omap4-var-stk-om44.dtb \
 	omap4-xyboard-mz609.dtb \
 	omap4-xyboard-mz617.dtb
+
+am335x-icev2-prueth-dtbs := am335x-icev2.dtb \
+	am335x-icev2-prueth-overlay.dtbo
+
 dtb-$(CONFIG_SOC_AM33XX) += \
 	am335x-baltos-ir2110.dtb \
 	am335x-baltos-ir3220.dtb \

@@ -100,6 +104,7 @@ dtb-$(CONFIG_SOC_AM33XX) += \
 	am335x-evmsk.dtb \
 	am335x-guardian.dtb \
 	am335x-icev2.dtb \
+	am335x-icev2-prueth.dtb \
 	am335x-lxm.dtb \
 	am335x-mba335x.dtb \
 	am335x-moxa-uc-2101.dtb \

diff --git a/arch/arm/boot/dts/ti/omap/am335x-icev2-prueth-overlay.dtso b/arch/arm/boot/dts/ti/omap/am335x-icev2-prueth-overlay.dtso
new file mode 100644
index 000000000000..abde5119875f
--- /dev/null
+++ b/arch/arm/boot/dts/ti/omap/am335x-icev2-prueth-overlay.dtso

@@ -0,0 +1,190 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * DT overlay for IDK AM335x
+ *
+ * Copyright (C) 2018 Texas Instruments Incorporated - http://www.ti.com/
+ */
+
+/*
+ * AM335x ICE V2 board
+ * http://www.ti.com/tool/tmdsice3359
+ */
+
+/dts-v1/;
+/plugin/;
+
+#include <dt-bindings/bus/ti-sysc.h>
+#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/pinctrl/am33xx.h>
+#include <dt-bindings/clock/am3.h>
+
+&{/} {
+        /* Dual-MAC Ethernet application node on PRU-ICSS */
+        pruss_eth: pruss-eth {
+                compatible = "ti,am3359-prueth";
+                ti,prus = <&pru0>, <&pru1>;
+                sram = <&ocmcram>;
+                ti,mii-rt = <&pruss_mii_rt>;
+                ti,iep = <&pruss_iep>;
+                ti,ecap = <&pruss_ecap>;
+                interrupts = <20 2 2>, <21 3 3>;
+                interrupt-names = "rx_hp", "rx_lp";
+                interrupt-parent = <&pruss_intc>;
+
+                pinctrl-0 = <&pruss_eth_default>;
+                pinctrl-names = "default";
+
+                ethernet-ports {
+                        #address-cells = <1>;
+                        #size-cells = <0>;
+                        pruss_emac0: ethernet-port@0 {
+                                reg = <0>;
+                                phy-handle = <&pruss_eth0_phy>;
+                                phy-mode = "mii";
+                                interrupts = <20 2 2>, <26 6 6>, <23 6 6>;
+                                interrupt-names = "rx", "emac_ptp_tx",
+                                                  "hsr_ptp_tx";
+                                /* Filled in by bootloader */
+                                local-mac-address = [00 00 00 00 00 00];
+                        };
+
+                        pruss_emac1: ethernet-port@1 {
+                                reg = <1>;
+                                phy-handle = <&pruss_eth1_phy>;
+                                phy-mode = "mii";
+                                interrupts = <21 3 3>, <27 9 7>, <24 9 7>;
+                                interrupt-names = "rx", "emac_ptp_tx",
+                                                  "hsr_ptp_tx";
+                                /* Filled in by bootloader */
+                                local-mac-address = [00 00 00 00 00 00];
+                        };
+                };
+        };
+};
+
+&am33xx_pinmux {
+	/* MDIO node for PRU-ICSS */
+        pruss_mdio_default: pruss_mdio_default {
+                pinctrl-single,pins = <
+                        /* gpmc_clk.pr1_mdio_mdclk */
+                        AM33XX_PADCONF(0x88c, PIN_OUTPUT, MUX_MODE5)
+                        /* gpmc_csn3.pr1_mdio_data */
+                        AM33XX_PADCONF(0x888, PIN_INPUT, MUX_MODE5)
+                        /* gpmc_ben0_cle.gpio2_5 */
+                        AM33XX_PADCONF(0x89c, PIN_INPUT_PULLUP, MUX_MODE7)
+                        /* disable CPSW MDIO */
+                        /* mdio_data.gpio0_0 */
+                        AM33XX_PADCONF(0x948, PIN_INPUT_PULLUP, MUX_MODE7)
+                        /* mdio_clk.gpio0_1 */
+                        AM33XX_PADCONF(0x94c, PIN_INPUT_PULLUP, MUX_MODE7)
+                >;
+        };
+
+	/* Pinmux configuration for PRU-ICSS */
+        pruss_eth_default: pruss_eth_default {
+                pinctrl-single,pins = <
+                        /* dss_data0.pr1_mii_mt0_clk */
+                        AM33XX_PADCONF(0x8a0, PIN_INPUT,   MUX_MODE2)
+                        /* dss_data5.pr1_mii0_txd0 */
+                        AM33XX_PADCONF(0x8b4, PIN_OUTPUT,  MUX_MODE2)
+                        /* dss_data4.pr1_mii0_txd1 */
+                        AM33XX_PADCONF(0x8b0, PIN_OUTPUT,  MUX_MODE2)
+                        /* dss_data3.pr1_mii0_txd2 */
+                        AM33XX_PADCONF(0x8ac, PIN_OUTPUT,  MUX_MODE2)
+                        /* dss_data2.pr1_mii0_txd3 */
+                        AM33XX_PADCONF(0x8a8, PIN_OUTPUT,  MUX_MODE2)
+                        /* dss_data11.pr1_mii0_rxd0 */
+                        AM33XX_PADCONF(0x8cc, PIN_INPUT,   MUX_MODE5)
+                        /* dss_data10.pr1_mii0_rxd1 */
+                        AM33XX_PADCONF(0x8c8, PIN_INPUT,   MUX_MODE5)
+                        /* dss_data9.pr1_mii0_rxd2 */
+                        AM33XX_PADCONF(0x8c4, PIN_INPUT,   MUX_MODE5)
+                        /* dss_data8.pr1_mii0_rxd3 */
+                        AM33XX_PADCONF(0x8c0, PIN_INPUT,   MUX_MODE5)
+                        /* dss_data1.pr1_mii0_txen */
+                        AM33XX_PADCONF(0x8a4, PIN_OUTPUT,  MUX_MODE2)
+                        /* dss_data14.pr1_mii_mr0_clk */
+                        AM33XX_PADCONF(0x8d8, PIN_INPUT,   MUX_MODE5)
+                        /* dss_data15.pr1_mii0_rxdv */
+                        AM33XX_PADCONF(0x8dc, PIN_INPUT,   MUX_MODE5)
+                        /* dss_data13.pr1_mii0_rxer */
+                        AM33XX_PADCONF(0x8d4, PIN_INPUT,   MUX_MODE5)
+                        /* dss_data12.pr1_mii0_rxlink */
+                        AM33XX_PADCONF(0x8d0, PIN_INPUT,   MUX_MODE5)
+                        /* dss_pclk.pr1_mii0_crs */
+                        AM33XX_PADCONF(0x8e8, PIN_INPUT,   MUX_MODE2)
+
+                        /* gpmc_a0.pr1_mii_mt1_clk */
+                        AM33XX_PADCONF(0x840, PIN_INPUT,   MUX_MODE5)
+                        /* gpmc_a4.pr1_mii1_txd0 */
+                        AM33XX_PADCONF(0x850, PIN_OUTPUT,  MUX_MODE5)
+                        /* gpmc_a3.pr1_mii1_txd1 */
+                        AM33XX_PADCONF(0x84c, PIN_OUTPUT,  MUX_MODE5)
+                        /* gpmc_a2.pr1_mii1_txd2 */
+                        AM33XX_PADCONF(0x848, PIN_OUTPUT,  MUX_MODE5)
+                        /* gpmc_a1.pr1_mii1_txd3 */
+                        AM33XX_PADCONF(0x844, PIN_OUTPUT,  MUX_MODE5)
+                        /* gpmc_a8.pr1_mii1_rxd0 */
+                        AM33XX_PADCONF(0x860, PIN_INPUT,   MUX_MODE5)
+                        /* gpmc_a7.pr1_mii1_rxd1 */
+                        AM33XX_PADCONF(0x85c, PIN_INPUT,   MUX_MODE5)
+                        /* gpmc_a6.pr1_mii1_rxd2 */
+                        AM33XX_PADCONF(0x858, PIN_INPUT,   MUX_MODE5)
+                        /* gpmc_a5.pr1_mii1_rxd3 */
+                        AM33XX_PADCONF(0x854, PIN_INPUT,   MUX_MODE5)
+                        /* gpmc_wpn.pr1_mii1_txen */
+                        AM33XX_PADCONF(0x874, PIN_OUTPUT,  MUX_MODE5)
+                        /* gpmc_a9.pr1_mii_mr1_clk */
+                        AM33XX_PADCONF(0x864, PIN_INPUT,   MUX_MODE5)
+                        /* gpmc_a10.pr1_mii1_rxdv */
+                        AM33XX_PADCONF(0x868, PIN_INPUT,   MUX_MODE5)
+                        /* gpmc_a11.pr1_mii1_rxer */
+                        AM33XX_PADCONF(0x86c, PIN_INPUT,   MUX_MODE5)
+                        /* gpmc_ben1.pr1_mii1_rxlink */
+                        AM33XX_PADCONF(0x878, PIN_INPUT,   MUX_MODE5)
+                        /* lcd_ac_bias_en.pr1_mii1_crs */
+                        AM33XX_PADCONF(0x8ec, PIN_INPUT,   MUX_MODE2)
+                        /* gpmc_wait0.pr1_mii1_col */
+                        AM33XX_PADCONF(0x870, PIN_INPUT,   MUX_MODE5)
+                >;
+        };
+};
+
+&gpio3 {
+        mux-mii-hog {
+                /* ETH1 mux: Low for MII-PRU, high for RMII-CPSW */
+                output-low;
+        };
+};
+
+/*
+ * Disable CPSW switch node and
+ * MDIO configuration to prevent
+ * conflict with PRU-ICSS
+ */
+&mac_sw {
+        status = "disable";
+};
+
+&davinci_mdio_sw {
+        status = "disable";
+};
+
+/* PRU-ICSS MDIO configuration */
+&pruss_mdio {
+        pinctrl-0 = <&pruss_mdio_default>;
+        pinctrl-names = "default";
+        reset-gpios = <&gpio2 5 GPIO_ACTIVE_LOW>;
+        reset-delay-us = <2>; /* PHY datasheet states 1uS min */
+        status = "okay";
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        pruss_eth0_phy: ethernet-phy@1 {
+                 reg = <1>;
+        };
+
+        pruss_eth1_phy: ethernet-phy@3 {
+                 reg = <3>;
+        };
+};

diff --git a/arch/arm/boot/dts/ti/omap/am33xx-l4.dtsi b/arch/arm/boot/dts/ti/omap/am33xx-l4.dtsi
index 89d16fcc773e..a63ef307d918 100644
--- a/arch/arm/boot/dts/ti/omap/am33xx-l4.dtsi
+++ b/arch/arm/boot/dts/ti/omap/am33xx-l4.dtsi

@@ -896,6 +896,17 @@ pruss_mii_rt: mii-rt@32000 {
 					reg = <0x32000 0x58>;
 				};
 
+				pruss_iep: iep@2e000 {
+					compatible = "ti,am3356-icss-iep";
+					reg = <0x2e000 0x31c>;
+					clocks = <&pruss_iepclk_mux>;
+				};
+
+				pruss_ecap: ecap@30000 {
+					compatible = "ti,pruss-ecap";
+					reg = <0x30000 0x60>;
+				};
+
 				pruss_intc: interrupt-controller@20000 {
 					compatible = "ti,pruss-intc";
 					reg = <0x20000 0x2000>;

diff --git a/arch/arm/boot/dts/ti/omap/am4372.dtsi b/arch/arm/boot/dts/ti/omap/am4372.dtsi
index 504fa6b57d39..494f251c8e6a 100644
--- a/arch/arm/boot/dts/ti/omap/am4372.dtsi
+++ b/arch/arm/boot/dts/ti/omap/am4372.dtsi

@@ -476,6 +476,17 @@ pruss1_mii_rt: mii-rt@32000 {
 					reg = <0x32000 0x58>;
 				};
 
+				pruss1_iep: iep@2e000 {
+					compatible = "ti,am4376-icss-iep";
+					reg = <0x2e000 0x31c>;
+					clocks = <&pruss1_iepclk_mux>;
+				};
+
+				pruss1_ecap: ecap@30000 {
+					compatible = "ti,pruss-ecap";
+					reg = <0x30000 0x60>;
+				};
+
 				pruss1_intc: interrupt-controller@20000 {
 					compatible = "ti,pruss-intc";
 					reg = <0x20000 0x2000>;

diff --git a/arch/arm/boot/dts/ti/omap/am437x-idk-evm.dts b/arch/arm/boot/dts/ti/omap/am437x-idk-evm.dts
index 826f687c368a..3d755d875a70 100644
--- a/arch/arm/boot/dts/ti/omap/am437x-idk-evm.dts
+++ b/arch/arm/boot/dts/ti/omap/am437x-idk-evm.dts

@@ -168,6 +168,48 @@ led-out7 {
 			default-state = "off";
 		};
 	};
+
+	/* Dual-MAC Ethernet application node on PRU-ICSS1 */
+	pruss1_eth: pruss1-eth {
+		compatible = "ti,am4376-prueth";
+		ti,prus = <&pru1_0>, <&pru1_1>;
+		sram = <&ocmcram>;
+		ti,mii-rt = <&pruss1_mii_rt>;
+		ti,iep = <&pruss1_iep>;
+		ti,ecap = <&pruss1_ecap>;
+		interrupts = <20 2 2>, <21 3 3>;
+		interrupt-names = "rx_hp", "rx_lp";
+		interrupt-parent = <&pruss1_intc>;
+
+		pinctrl-0 = <&pruss1_eth_default>;
+		pinctrl-names = "default";
+
+		ethernet-ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			pruss1_emac0: ethernet-port@0 {
+				reg = <0>;
+				phy-handle = <&pruss1_eth0_phy>;
+				phy-mode = "mii";
+				interrupts = <20 2 2>, <26 6 6>, <23 6 6>;
+				interrupt-names = "rx", "emac_ptp_tx",
+						  "hsr_ptp_tx";
+				/* Filled in by bootloader */
+				local-mac-address = [00 00 00 00 00 00];
+			};
+
+			pruss1_emac1: ethernet-port@1 {
+				reg = <1>;
+				phy-handle = <&pruss1_eth1_phy>;
+				phy-mode = "mii";
+				interrupts = <21 3 3>, <27 9 5>, <24 9 5>;
+				interrupt-names = "rx", "emac_ptp_tx",
+						  "hsr_ptp_tx";
+				/* Filled in by bootloader */
+				local-mac-address = [00 00 00 00 00 00];
+			};
+		};
+	};
 };
 
 &am43xx_pinmux {

@@ -303,6 +345,86 @@ AM4372_IOPAD(0x94c, PIN_INPUT_PULLDOWN | MUX_MODE7)
 		>;
 	};
 
+	pruss1_mdio_default: pruss1_mdio_default {
+		pinctrl-single,pins = <
+			/* gpmc_clk.pr1_mdio_mdclk */
+			AM4372_IOPAD(0x88c, (PIN_OUTPUT | MUX_MODE5))
+			/* xdma_event_intr0.pr1_mdio_data */
+			AM4372_IOPAD(0xa70, (PIN_INPUT  | MUX_MODE8))
+			/* cam1_data6.gpio4_20 */
+			AM4372_IOPAD(0xa00, (PIN_INPUT_PULLUP | MUX_MODE7))
+		>;
+	};
+
+	pruss1_eth_default: pruss1_eth_default {
+		pinctrl-single,pins = <
+			/* dss_data0.pr1_mii_mt0_clk */
+			AM4372_IOPAD(0x8a0, (PIN_INPUT  | MUX_MODE2))
+			/* dss_data5.pr1_mii0_txd0 */
+			AM4372_IOPAD(0x8b4, (PIN_OUTPUT | MUX_MODE2))
+			/* dss_data4.pr1_mii0_txd1 */
+			AM4372_IOPAD(0x8b0, (PIN_OUTPUT | MUX_MODE2))
+			/* dss_data3.pr1_mii0_txd2 */
+			AM4372_IOPAD(0x8ac, (PIN_OUTPUT | MUX_MODE2))
+			/* dss_data2.pr1_mii0_txd3 */
+			AM4372_IOPAD(0x8a8, (PIN_OUTPUT | MUX_MODE2))
+			/* dss_data11.pr1_mii0_rxd0 */
+			AM4372_IOPAD(0x8cc, (PIN_INPUT  | MUX_MODE5))
+			/* dss_data10.pr1_mii0_rxd1 */
+			AM4372_IOPAD(0x8c8, (PIN_INPUT  | MUX_MODE5))
+			/* dss_data9.pr1_mii0_rxd2 */
+			AM4372_IOPAD(0x8c4, (PIN_INPUT  | MUX_MODE5))
+			/* dss_data8.pr1_mii0_rxd3 */
+			AM4372_IOPAD(0x8c0, (PIN_INPUT  | MUX_MODE5))
+			/* dss_data1.pr1_mii0_txen */
+			AM4372_IOPAD(0x8a4, (PIN_OUTPUT | MUX_MODE2))
+			/* dss_data14.pr1_mii_mr0_clk */
+			AM4372_IOPAD(0x8d8, (PIN_INPUT  | MUX_MODE5))
+			/* dss_data15.pr1_mii0_rxdv */
+			AM4372_IOPAD(0x8dc, (PIN_INPUT  | MUX_MODE5))
+			/* dss_data13.pr1_mii0_rxer */
+			AM4372_IOPAD(0x8d4, (PIN_INPUT  | MUX_MODE5))
+			/* dss_data12.pr1_mii0_rxlink */
+			AM4372_IOPAD(0x8d0, (PIN_INPUT  | MUX_MODE5))
+			/* gpio5_10.pr1_mii0_crs */
+			AM4372_IOPAD(0xa40, (PIN_INPUT  | MUX_MODE5))
+			/* gpio5_8.pr1_mii0_col */
+			AM4372_IOPAD(0xa38, (PIN_INPUT  | MUX_MODE5))
+			/* gpmc_a6.pr1_mii_mt1_clk */
+			AM4372_IOPAD(0x858, (PIN_INPUT  | MUX_MODE5))
+			/* gpmc_a5.pr1_mii1_txd0 */
+			AM4372_IOPAD(0x854, (PIN_OUTPUT | MUX_MODE5))
+			/* gpmc_a4.pr1_mii1_txd1 */
+			AM4372_IOPAD(0x850, (PIN_OUTPUT | MUX_MODE5))
+			/* gpmc_a3.pr1_mii1_txd2 */
+			AM4372_IOPAD(0x84c, (PIN_OUTPUT | MUX_MODE5))
+			/* gpmc_a2.pr1_mii1_txd3 */
+			AM4372_IOPAD(0x848, (PIN_OUTPUT | MUX_MODE5))
+			/* gpmc_a11.pr1_mii1_rxd0 */
+			AM4372_IOPAD(0x86c, (PIN_INPUT  | MUX_MODE5))
+			/* gpmc_a10.pr1_mii1_rxd1 */
+			AM4372_IOPAD(0x868, (PIN_INPUT  | MUX_MODE5))
+			/* gpmc_a9.pr1_mii1_rxd2 */
+			AM4372_IOPAD(0x864, (PIN_INPUT  | MUX_MODE5))
+			/* gpmc_a8.pr1_mii1_rxd3 */
+			AM4372_IOPAD(0x860, (PIN_INPUT  | MUX_MODE5))
+			/* gpmc_a0.pr1_mii1_txen */
+			AM4372_IOPAD(0x840, (PIN_OUTPUT | MUX_MODE5))
+			/* gpmc_a7.pr1_mii_mr1_clk */
+			AM4372_IOPAD(0x85c, (PIN_INPUT  | MUX_MODE5))
+			/* gpmc_a1.pr1_mii1_rxdv */
+			AM4372_IOPAD(0x844, (PIN_INPUT  | MUX_MODE5))
+			/* gpmc_wpn.pr1_mii1_rxer */
+			AM4372_IOPAD(0x874, (PIN_INPUT  | MUX_MODE5))
+			/* gpio5_13.pr1_mii1_rxlink */
+			AM4372_IOPAD(0xa4c, (PIN_INPUT  | MUX_MODE5))
+			/* gpio5_11.pr1_mii1_crs */
+			AM4372_IOPAD(0xa44, (PIN_INPUT  | MUX_MODE5))
+			/* gpmc_be1n.pr1_mii1_col */
+			AM4372_IOPAD(0x878, (PIN_INPUT  | MUX_MODE5))
+		>;
+	};
+
 	qspi_pins_default: qspi-default-pins {
 		pinctrl-single,pins = <
 			AM4372_IOPAD(0x87c, PIN_OUTPUT_PULLUP | MUX_MODE3)	/* gpmc_csn0.qspi_csn */

@@ -539,5 +661,18 @@ opp-100-600000000 {
 };
 
 &pruss1_mdio {
-	status = "disabled";
+	pinctrl-0 = <&pruss1_mdio_default>;
+	pinctrl-names = "default";
+	status = "okay";
+
+	reset-gpios = <&gpio4 20 GPIO_ACTIVE_LOW>;
+	reset-delay-us = <2>;	/* PHY datasheet states 1uS min */
+
+	pruss1_eth0_phy: ethernet-phy@0 {
+		reg = <0>;
+	};
+
+	pruss1_eth1_phy: ethernet-phy@1 {
+		reg = <1>;
+	};
 };

-- 
2.43.0

On 1/5/26 10:21 AM, Parvathi Pudi wrote:

Have you regenerated this list lately? The pinmux tool usually puts the
comment after the PADCONF entry. It also now also shows the pin number
in the comment which is nice:

	AM33XX_IOPAD(0x88c, PIN_OUTPUT, MUX_MODE5) /* (V12) gpmc_clk.pr1_mdio_mdclk */
	AM33XX_IOPAD(0x888, PIN_INPUT, MUX_MODE5) /* (T13) gpmc_csn3.pr1_mdio_data */

I'd recommend regenerating these nodes to match the latest pinmux tool output.

+                        /* gpmc_ben0_cle.gpio2_5 */
+                        AM33XX_PADCONF(0x89c, PIN_INPUT_PULLUP, MUX_MODE7)
+                        /* disable CPSW MDIO */

Is this needed? If you disable the CPSW MDIO node the pinmux should be unset,
so not sure why you are muxing these to GPIO pins.

Also, this patch is a bit busy, might be easier to review if you split it
into one for AM335x and one for AM437x changes.

Andrew

Hi,

On 1/5/26 10:21 AM, Parvathi Pudi wrote:

quoted

From: Roger Quadros <redacted>

PRU-ICSS instance consists of two PRU cores along with various
peripherals such as the Interrupt Controller (PRU_INTC), the Industrial
Ethernet Peripheral(IEP), the Real Time Media Independent Interface
controller (MII_RT), and the Enhanced Capture (eCAP) event module.

The TI Sitara AM335x ICE-V2 consists of single PRU-ICSS instance,
This patch adds the new device tree overlay file in-order to enable
PRU-ICSS instance, along with makefile changes.

The TI Sitara AM437x series of devices consists of 2 PRU-ICSS instances
(PRU-ICSS0 and PRU-ICSS1). This patch adds the device tree nodes for the
PRU-ICSS1 instance to support DUAL-MAC mode of operation. Support for
Ethernet over PRU is available only for ICSS1 instance.

am33xx-l4.dtsi, am4372.dtsi - Adds IEP and eCAP peripheral as child nodes
of the PRUSS subsystem node.

am335x-icev2-prueth.dtso, am437x-idk-evm.dts - Adds PRU-ICSS
instance node along with PRU eth port information and corresponding
port configuration. It includes interrupt mapping for packet reception,
HW timestamp collection, and PRU Ethernet ports in MII mode,

GPIO configuration, boot strapping along with delay configuration for
individual PRU Ethernet port and other required nodes.

Signed-off-by: Roger Quadros <redacted>
Signed-off-by: Andrew F. Davis <redacted>
Signed-off-by: Murali Karicheri <redacted>
Signed-off-by: Basharath Hussain Khaja <redacted>
Signed-off-by: Parvathi Pudi <parvathi@couthit.com>
---
  arch/arm/boot/dts/ti/omap/Makefile            |   5 +
  .../ti/omap/am335x-icev2-prueth-overlay.dtso  | 190 ++++++++++++++++++
  arch/arm/boot/dts/ti/omap/am33xx-l4.dtsi      |  11 +
  arch/arm/boot/dts/ti/omap/am4372.dtsi         |  11 +
  arch/arm/boot/dts/ti/omap/am437x-idk-evm.dts  | 137 ++++++++++++-
  5 files changed, 353 insertions(+), 1 deletion(-)
  create mode 100644 arch/arm/boot/dts/ti/omap/am335x-icev2-prueth-overlay.dtso

diff --git a/arch/arm/boot/dts/ti/omap/Makefile

b/arch/arm/boot/dts/ti/omap/Makefile
index 14e500846875..c68948035eca 100644

--- a/arch/arm/boot/dts/ti/omap/Makefile
+++ b/arch/arm/boot/dts/ti/omap/Makefile

@@ -82,6 +82,10 @@ dtb-$(CONFIG_ARCH_OMAP4) += \
  	omap4-var-stk-om44.dtb \
  	omap4-xyboard-mz609.dtb \
  	omap4-xyboard-mz617.dtb
+
+am335x-icev2-prueth-dtbs := am335x-icev2.dtb \
+	am335x-icev2-prueth-overlay.dtbo
+
  dtb-$(CONFIG_SOC_AM33XX) += \
  	am335x-baltos-ir2110.dtb \
  	am335x-baltos-ir3220.dtb \

@@ -100,6 +104,7 @@ dtb-$(CONFIG_SOC_AM33XX) += \
  	am335x-evmsk.dtb \
  	am335x-guardian.dtb \
  	am335x-icev2.dtb \
+	am335x-icev2-prueth.dtb \
  	am335x-lxm.dtb \
  	am335x-mba335x.dtb \
  	am335x-moxa-uc-2101.dtb \

diff --git a/arch/arm/boot/dts/ti/omap/am335x-icev2-prueth-overlay.dtso

b/arch/arm/boot/dts/ti/omap/am335x-icev2-prueth-overlay.dtso
new file mode 100644
index 000000000000..abde5119875f

--- /dev/null
+++ b/arch/arm/boot/dts/ti/omap/am335x-icev2-prueth-overlay.dtso

@@ -0,0 +1,190 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * DT overlay for IDK AM335x
+ *
+ * Copyright (C) 2018 Texas Instruments Incorporated - http://www.ti.com/
+ */
+
+/*
+ * AM335x ICE V2 board
+ * http://www.ti.com/tool/tmdsice3359
+ */
+
+/dts-v1/;
+/plugin/;
+
+#include <dt-bindings/bus/ti-sysc.h>
+#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/pinctrl/am33xx.h>
+#include <dt-bindings/clock/am3.h>
+
+&{/} {
+        /* Dual-MAC Ethernet application node on PRU-ICSS */
+        pruss_eth: pruss-eth {
+                compatible = "ti,am3359-prueth";
+                ti,prus = <&pru0>, <&pru1>;
+                sram = <&ocmcram>;
+                ti,mii-rt = <&pruss_mii_rt>;
+                ti,iep = <&pruss_iep>;
+                ti,ecap = <&pruss_ecap>;
+                interrupts = <20 2 2>, <21 3 3>;
+                interrupt-names = "rx_hp", "rx_lp";
+                interrupt-parent = <&pruss_intc>;
+
+                pinctrl-0 = <&pruss_eth_default>;
+                pinctrl-names = "default";
+
+                ethernet-ports {
+                        #address-cells = <1>;
+                        #size-cells = <0>;
+                        pruss_emac0: ethernet-port@0 {
+                                reg = <0>;
+                                phy-handle = <&pruss_eth0_phy>;
+                                phy-mode = "mii";
+                                interrupts = <20 2 2>, <26 6 6>, <23 6 6>;
+                                interrupt-names = "rx", "emac_ptp_tx",
+                                                  "hsr_ptp_tx";
+                                /* Filled in by bootloader */
+                                local-mac-address = [00 00 00 00 00 00];
+                        };
+
+                        pruss_emac1: ethernet-port@1 {
+                                reg = <1>;
+                                phy-handle = <&pruss_eth1_phy>;
+                                phy-mode = "mii";
+                                interrupts = <21 3 3>, <27 9 7>, <24 9 7>;
+                                interrupt-names = "rx", "emac_ptp_tx",
+                                                  "hsr_ptp_tx";
+                                /* Filled in by bootloader */
+                                local-mac-address = [00 00 00 00 00 00];
+                        };
+                };
+        };
+};
+
+&am33xx_pinmux {
+	/* MDIO node for PRU-ICSS */
+        pruss_mdio_default: pruss_mdio_default {
+                pinctrl-single,pins = <
+                        /* gpmc_clk.pr1_mdio_mdclk */
+                        AM33XX_PADCONF(0x88c, PIN_OUTPUT, MUX_MODE5)
+                        /* gpmc_csn3.pr1_mdio_data */
+                        AM33XX_PADCONF(0x888, PIN_INPUT, MUX_MODE5)

Have you regenerated this list lately? The pinmux tool usually puts the
comment after the PADCONF entry. It also now also shows the pin number
in the comment which is nice:

AM33XX_IOPAD(0x88c, PIN_OUTPUT, MUX_MODE5) /* (V12) gpmc_clk.pr1_mdio_mdclk */
AM33XX_IOPAD(0x888, PIN_INPUT, MUX_MODE5) /* (T13) gpmc_csn3.pr1_mdio_data */

I'd recommend regenerating these nodes to match the latest pinmux tool output.

Sure, we will check and regenerate these nodes using the pinmux tool and
update accordingly.

quoted

+                        /* gpmc_ben0_cle.gpio2_5 */
+                        AM33XX_PADCONF(0x89c, PIN_INPUT_PULLUP, MUX_MODE7)
+                        /* disable CPSW MDIO */

Is this needed? If you disable the CPSW MDIO node the pinmux should be unset,
so not sure why you are muxing these to GPIO pins.

We will review and get back with more details on this.

Also, this patch is a bit busy, might be easier to review if you split it
into one for AM335x and one for AM437x changes.

We will split the changes into two patches, one for AM335x
and one for AM437x, and address this in the next version.


Thanks and Regards,
Parvathi.

Hi,

Hi,

quoted

On 1/5/26 10:21 AM, Parvathi Pudi wrote:

quoted

From: Roger Quadros <redacted>

PRU-ICSS instance consists of two PRU cores along with various
peripherals such as the Interrupt Controller (PRU_INTC), the Industrial
Ethernet Peripheral(IEP), the Real Time Media Independent Interface
controller (MII_RT), and the Enhanced Capture (eCAP) event module.

The TI Sitara AM335x ICE-V2 consists of single PRU-ICSS instance,
This patch adds the new device tree overlay file in-order to enable
PRU-ICSS instance, along with makefile changes.

The TI Sitara AM437x series of devices consists of 2 PRU-ICSS instances
(PRU-ICSS0 and PRU-ICSS1). This patch adds the device tree nodes for the
PRU-ICSS1 instance to support DUAL-MAC mode of operation. Support for
Ethernet over PRU is available only for ICSS1 instance.

am33xx-l4.dtsi, am4372.dtsi - Adds IEP and eCAP peripheral as child nodes
of the PRUSS subsystem node.

am335x-icev2-prueth.dtso, am437x-idk-evm.dts - Adds PRU-ICSS
instance node along with PRU eth port information and corresponding
port configuration. It includes interrupt mapping for packet reception,
HW timestamp collection, and PRU Ethernet ports in MII mode,

GPIO configuration, boot strapping along with delay configuration for
individual PRU Ethernet port and other required nodes.

Signed-off-by: Roger Quadros <redacted>
Signed-off-by: Andrew F. Davis <redacted>
Signed-off-by: Murali Karicheri <redacted>
Signed-off-by: Basharath Hussain Khaja <redacted>
Signed-off-by: Parvathi Pudi <parvathi@couthit.com>
---
  arch/arm/boot/dts/ti/omap/Makefile            |   5 +
  .../ti/omap/am335x-icev2-prueth-overlay.dtso  | 190 ++++++++++++++++++
  arch/arm/boot/dts/ti/omap/am33xx-l4.dtsi      |  11 +
  arch/arm/boot/dts/ti/omap/am4372.dtsi         |  11 +
  arch/arm/boot/dts/ti/omap/am437x-idk-evm.dts  | 137 ++++++++++++-
  5 files changed, 353 insertions(+), 1 deletion(-)
  create mode 100644 arch/arm/boot/dts/ti/omap/am335x-icev2-prueth-overlay.dtso

diff --git a/arch/arm/boot/dts/ti/omap/Makefile

b/arch/arm/boot/dts/ti/omap/Makefile
index 14e500846875..c68948035eca 100644

--- a/arch/arm/boot/dts/ti/omap/Makefile
+++ b/arch/arm/boot/dts/ti/omap/Makefile

@@ -82,6 +82,10 @@ dtb-$(CONFIG_ARCH_OMAP4) += \
  	omap4-var-stk-om44.dtb \
  	omap4-xyboard-mz609.dtb \
  	omap4-xyboard-mz617.dtb
+
+am335x-icev2-prueth-dtbs := am335x-icev2.dtb \
+	am335x-icev2-prueth-overlay.dtbo
+
  dtb-$(CONFIG_SOC_AM33XX) += \
  	am335x-baltos-ir2110.dtb \
  	am335x-baltos-ir3220.dtb \

@@ -100,6 +104,7 @@ dtb-$(CONFIG_SOC_AM33XX) += \
  	am335x-evmsk.dtb \
  	am335x-guardian.dtb \
  	am335x-icev2.dtb \
+	am335x-icev2-prueth.dtb \
  	am335x-lxm.dtb \
  	am335x-mba335x.dtb \
  	am335x-moxa-uc-2101.dtb \

diff --git a/arch/arm/boot/dts/ti/omap/am335x-icev2-prueth-overlay.dtso

b/arch/arm/boot/dts/ti/omap/am335x-icev2-prueth-overlay.dtso
new file mode 100644
index 000000000000..abde5119875f

--- /dev/null
+++ b/arch/arm/boot/dts/ti/omap/am335x-icev2-prueth-overlay.dtso

@@ -0,0 +1,190 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * DT overlay for IDK AM335x
+ *
+ * Copyright (C) 2018 Texas Instruments Incorporated - http://www.ti.com/
+ */
+
+/*
+ * AM335x ICE V2 board
+ * http://www.ti.com/tool/tmdsice3359
+ */
+
+/dts-v1/;
+/plugin/;
+
+#include <dt-bindings/bus/ti-sysc.h>
+#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/pinctrl/am33xx.h>
+#include <dt-bindings/clock/am3.h>
+
+&{/} {
+        /* Dual-MAC Ethernet application node on PRU-ICSS */
+        pruss_eth: pruss-eth {
+                compatible = "ti,am3359-prueth";
+                ti,prus = <&pru0>, <&pru1>;
+                sram = <&ocmcram>;
+                ti,mii-rt = <&pruss_mii_rt>;
+                ti,iep = <&pruss_iep>;
+                ti,ecap = <&pruss_ecap>;
+                interrupts = <20 2 2>, <21 3 3>;
+                interrupt-names = "rx_hp", "rx_lp";
+                interrupt-parent = <&pruss_intc>;
+
+                pinctrl-0 = <&pruss_eth_default>;
+                pinctrl-names = "default";
+
+                ethernet-ports {
+                        #address-cells = <1>;
+                        #size-cells = <0>;
+                        pruss_emac0: ethernet-port@0 {
+                                reg = <0>;
+                                phy-handle = <&pruss_eth0_phy>;
+                                phy-mode = "mii";
+                                interrupts = <20 2 2>, <26 6 6>, <23 6 6>;
+                                interrupt-names = "rx", "emac_ptp_tx",
+                                                  "hsr_ptp_tx";
+                                /* Filled in by bootloader */
+                                local-mac-address = [00 00 00 00 00 00];
+                        };
+
+                        pruss_emac1: ethernet-port@1 {
+                                reg = <1>;
+                                phy-handle = <&pruss_eth1_phy>;
+                                phy-mode = "mii";
+                                interrupts = <21 3 3>, <27 9 7>, <24 9 7>;
+                                interrupt-names = "rx", "emac_ptp_tx",
+                                                  "hsr_ptp_tx";
+                                /* Filled in by bootloader */
+                                local-mac-address = [00 00 00 00 00 00];
+                        };
+                };
+        };
+};
+
+&am33xx_pinmux {
+	/* MDIO node for PRU-ICSS */
+        pruss_mdio_default: pruss_mdio_default {
+                pinctrl-single,pins = <
+                        /* gpmc_clk.pr1_mdio_mdclk */
+                        AM33XX_PADCONF(0x88c, PIN_OUTPUT, MUX_MODE5)
+                        /* gpmc_csn3.pr1_mdio_data */
+                        AM33XX_PADCONF(0x888, PIN_INPUT, MUX_MODE5)

Have you regenerated this list lately? The pinmux tool usually puts the
comment after the PADCONF entry. It also now also shows the pin number
in the comment which is nice:

AM33XX_IOPAD(0x88c, PIN_OUTPUT, MUX_MODE5) /* (V12) gpmc_clk.pr1_mdio_mdclk */
AM33XX_IOPAD(0x888, PIN_INPUT, MUX_MODE5) /* (T13) gpmc_csn3.pr1_mdio_data */

I'd recommend regenerating these nodes to match the latest pinmux tool output.

Sure, we will check and regenerate these nodes using the pinmux tool and
update accordingly.

quoted

quoted

+                        /* gpmc_ben0_cle.gpio2_5 */
+                        AM33XX_PADCONF(0x89c, PIN_INPUT_PULLUP, MUX_MODE7)
+                        /* disable CPSW MDIO */

Is this needed? If you disable the CPSW MDIO node the pinmux should be unset,
so not sure why you are muxing these to GPIO pins.

We will review and get back with more details on this.

On the AM335x board, the CPSW MDIO and PRUSS MDIO signals are routed to the same physical
pins (as shown in the schematic, see page 10 “MII_MUX” in tmdxice3359_sch_3h0013_v2_1a.pdf
from https://www.ti.com/lit/zip/TIDR336 ). Because of this shared routing, the pinmux
configuration applied by U-Boot for CPSW MDIO remains active even if the CPSW MDIO node is
later disabled in Linux, and Linux does not automatically revert the pins to their reset state.

To prevent signal contention and ensure correct PRUSS MDIO operation, we explicitly re-mux
the CPSW MDIO pins to GPIO in the PRUSS pinctrl. Without this explicit reconfiguration, we
observed PHY connection issues during startup.


Thanks and Regards,
Parvathi.

On the AM335x board, the CPSW MDIO and PRUSS MDIO signals are routed to the same physical
pins (as shown in the schematic, see page 10 “MII_MUX” in tmdxice3359_sch_3h0013_v2_1a.pdf
from https://www.ti.com/lit/zip/TIDR336 ). Because of this shared routing, the pinmux
configuration applied by U-Boot for CPSW MDIO remains active even if the CPSW MDIO node is
later disabled in Linux, and Linux does not automatically revert the pins to their reset state.

It is generally a bad idea to rely on the bootloader. I would make the
CPSW MDIO configure the pins how it needs it. The PRUSS MDIO should
also configure the pins how it needs them. However, it is not as
simple as that...

Looking at the schematic, what you have is ugly. You literally wire
the outputs together, without a hardware mux. For MDC you assume one
is Hi-Z, while the other drives the line. For MDIO it does not matter,
both are inputs. so Hi-Z.

I actually think you might need to represent this in Linux, with
something i would call a pinmux-mux. You give it two sets of pinmux
configurations. The active device claims the mux and gets it to set
the two sets of pinmux as needed. Also, just setting the pinmux to
GPIO is not sufficient, you also need to ensure the GPIO is configured
for input, so the lines go Hi-Z. Often pinmux and GPIO controllers are
interconnected, so the pinmux subsystem might be able to do that for
you.

I don't know if a pinmux-mux already exists in Linux. You probably
want to ask on the pinmux mailing list, or they might have a different
idea how to cleanly do this.

	Andrew

Hi,

quoted

On the AM335x board, the CPSW MDIO and PRUSS MDIO signals are routed to the same
physical
pins (as shown in the schematic, see page 10 “MII_MUX” in
tmdxice3359_sch_3h0013_v2_1a.pdf
from https://www.ti.com/lit/zip/TIDR336 ). Because of this shared routing, the
pinmux
configuration applied by U-Boot for CPSW MDIO remains active even if the CPSW
MDIO node is
later disabled in Linux, and Linux does not automatically revert the pins to
their reset state.

It is generally a bad idea to rely on the bootloader. I would make the
CPSW MDIO configure the pins how it needs it. The PRUSS MDIO should
also configure the pins how it needs them. However, it is not as
simple as that...

On AM335x, the MDIO interface is selected by a hardware jumper before the
bootloader runs. The selection is latched in hardware, so only one MDIO
controller (CPSW or PRUSS) is active at a time.

Since the hardware choice is fixed prior to Linux boot, we decided to ensure
in U-Boot that the pins are configured according to the selected hardware latch.
Linux still configures the active MDIO controller as required via its DT and
pinctrl settings. The U‑Boot change ensures a clean initial state that matches
the hardware latched configuration, and we have this prepared as a patch that
will be submitted to mainline U‑Boot shortly.

Looking at the schematic, what you have is ugly. You literally wire
the outputs together, without a hardware mux. For MDC you assume one
is Hi-Z, while the other drives the line. For MDIO it does not matter,
both are inputs. so Hi-Z.

Although the schematic shows the signals tied together without an explicit mux,
the hardware jumper selects RMII (CPSW) or MII (PRUSS) before boot. Only the
selected block is enabled, and the other remains inactive, so the lines are never
driven concurrently.

I actually think you might need to represent this in Linux, with
something i would call a pinmux-mux. You give it two sets of pinmux
configurations. The active device claims the mux and gets it to set
the two sets of pinmux as needed. Also, just setting the pinmux to
GPIO is not sufficient, you also need to ensure the GPIO is configured
for input, so the lines go Hi-Z. Often pinmux and GPIO controllers are
interconnected, so the pinmux subsystem might be able to do that for
you.

I don't know if a pinmux-mux already exists in Linux. You probably
want to ask on the pinmux mailing list, or they might have a different
idea how to cleanly do this.

Andrew

In this case, the selection is static and determined by hardware via a jumper,
not switchable at runtime. Linux reflects this by choosing the Device Tree
configuration corresponding to the latched jumper, it enables the selected
MDIO controller and sets up its pinmux, while the other MDIO controller
remains disabled.

We will update the patches by removing the CPSW pin reset configuration in
ICSS context and post the next version shortly.

Thanks and Regards,
Parvathi

Hi,

Am 05.01.2026 um 17:21 schrieb Parvathi Pudi [off-list ref]:

From: Roger Quadros <redacted>

PRU-ICSS instance consists of two PRU cores along with various
peripherals such as the Interrupt Controller (PRU_INTC), the Industrial
Ethernet Peripheral(IEP), the Real Time Media Independent Interface
controller (MII_RT), and the Enhanced Capture (eCAP) event module.

I am just wondering about the subject of this patch.

It reads as if general AM335x and AM437x support is introduced for the first time.
IMHO it should tell which subsystem/improvement is added.

BR and thanks,
Nikolaus

Hi, 

Hi,

quoted

Am 05.01.2026 um 17:21 schrieb Parvathi Pudi [off-list ref]:

From: Roger Quadros <redacted>

PRU-ICSS instance consists of two PRU cores along with various
peripherals such as the Interrupt Controller (PRU_INTC), the Industrial
Ethernet Peripheral(IEP), the Real Time Media Independent Interface
controller (MII_RT), and the Enhanced Capture (eCAP) event module.

I am just wondering about the subject of this patch.

It reads as if general AM335x and AM437x support is introduced for the first
time.
IMHO it should tell which subsystem/improvement is added.

Sure, we will address this in the next version.

Thanks and Regards,
Parvathi