mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH v3 0/4] Add USB2.0 support for RZ/G3E
@ 2026-09-09  8:54 Tommaso Merciai
  2026-09-09  8:54 ` [PATCH v3 1/4] arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes Tommaso Merciai
                   ` (4 more replies)
  0 siblings, 5 replies; 14+ messages in thread
From: Tommaso Merciai @ 2026-09-09  8:54 UTC (permalink / raw)
  To: tomm.merciai, prabhakar.mahadev-lad.rj
  Cc: linux-renesas-soc, biju.das.jz, Tommaso Merciai,
	Geert Uytterhoeven, Magnus Damm, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, devicetree, linux-kernel

Dear All,

This patch series adds USB2.0 support for the Renesas R9A09G047 SoC and
enables it on the RZ/G3E SMARK II board.

Kind regards,
Tommaso

v2->v3:
 - No changes just rebased on top of next-20260908.

v1->v2:
 - Included PATCH v7 7/9 and PATCH v7 6/9 from
   v7: https://lore.kernel.org/all/20260812141940.246223-1-biju.das.jz@bp.renesas.com/
 - Drop placeholder patch
 - PATCH 4/4: Disabled RZ/G3L common nodes from r9a08g046l48-smarc.dts.
   Update commit body, and improved usb20_pins, usb21_pins comments.

Biju Das (2):
  arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes
  arm64: dts: renesas: r9a08g046: Add USB2.0 function device nodes

Tommaso Merciai (2):
  arm64: dts: renesas: r9a09g047: Add USB2.0 support
  arm64: dts: renesas: r9a09g047e57-smarc: Enable USB2.0 support

 arch/arm64/boot/dts/renesas/r9a08g046.dtsi    | 142 ++++++++++++++++++
 .../boot/dts/renesas/r9a08g046l48-smarc.dts   |  20 +++
 arch/arm64/boot/dts/renesas/r9a09g047.dtsi    | 124 +++++++++++++++
 .../boot/dts/renesas/r9a09g047e57-smarc.dts   |  45 ++++++
 .../boot/dts/renesas/renesas-smarc2.dtsi      |  25 +++
 5 files changed, 356 insertions(+)

-- 
2.54.0


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [PATCH v3 1/4] arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes
  2026-09-09  8:54 [PATCH v3 0/4] Add USB2.0 support for RZ/G3E Tommaso Merciai
@ 2026-09-09  8:54 ` Tommaso Merciai
  2026-09-14 17:28   ` Geert Uytterhoeven
  2026-09-09  8:54 ` [PATCH v3 2/4] arm64: dts: renesas: r9a08g046: Add USB2.0 function " Tommaso Merciai
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 14+ messages in thread
From: Tommaso Merciai @ 2026-09-09  8:54 UTC (permalink / raw)
  To: tomm.merciai, prabhakar.mahadev-lad.rj
  Cc: linux-renesas-soc, biju.das.jz, Geert Uytterhoeven, Magnus Damm,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, devicetree,
	linux-kernel, Tommaso Merciai

From: Biju Das <biju.das.jz@bp.renesas.com>

Add USB2.0 device nodes to the RZ/G3L (r9a08g046) SoC DTSI, covering
the USB PHY controller, OHCI/EHCI host controllers, and USB2 PHYs for
both ports.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>
---
v2->v3:
 - No changes just rebased on top of next-20260908.

v1->v2:
 - Cherry picked from "Add RZ/G3L USB2.0 host support" v7.
   v7: https://lore.kernel.org/all/20260812141940.246223-7-biju.das.jz@bp.renesas.com/

 arch/arm64/boot/dts/renesas/r9a08g046.dtsi | 104 +++++++++++++++++++++
 1 file changed, 104 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r9a08g046.dtsi b/arch/arm64/boot/dts/renesas/r9a08g046.dtsi
index 608ba1f9e2f8..10123281c53c 100644
--- a/arch/arm64/boot/dts/renesas/r9a08g046.dtsi
+++ b/arch/arm64/boot/dts/renesas/r9a08g046.dtsi
@@ -1127,6 +1127,110 @@ queue3 {
 			};
 		};
 
+		phyrst: usbphy-ctrl@11e00000 {
+			compatible = "renesas,r9a08g046-usbphy-ctrl";
+			reg = <0 0x11e00000 0 0x10000>;
+			clocks = <&cpg CPG_MOD R9A08G046_USB_PCLK>;
+			resets = <&cpg R9A08G046_USB_PRESETN>;
+			power-domains = <&cpg>;
+			#reset-cells = <1>;
+			status = "disabled";
+
+			regulators {
+				usb0_vbus_otg: vbus0 {
+					regulator-name = "usb0_vbus";
+				};
+
+				usb1_vbus_otg: vbus1 {
+					regulator-name = "usb1_vbus";
+				};
+			};
+		};
+
+		ohci0: usb@11e10000 {
+			compatible = "generic-ohci";
+			reg = <0 0x11e10000 0 0x100>;
+			interrupts = <GIC_SPI 113 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD R9A08G046_USB_PCLK>,
+				 <&cpg CPG_MOD R9A08G046_USB_U2H0_HCLK>;
+			resets = <&phyrst 0>,
+				 <&cpg R9A08G046_USB_U2H0_HRESETN>;
+			phys = <&usb2_phy0 1>;
+			phy-names = "usb";
+			power-domains = <&cpg>;
+			status = "disabled";
+		};
+
+		ohci1: usb@11e90000 {
+			compatible = "generic-ohci";
+			reg = <0 0x11e90000 0 0x100>;
+			interrupts = <GIC_SPI 118 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD R9A08G046_USB_PCLK>,
+				 <&cpg CPG_MOD R9A08G046_USB_U2H1_HCLK>;
+			resets = <&phyrst 1>,
+				 <&cpg R9A08G046_USB_U2H1_HRESETN>;
+			phys = <&usb2_phy1 1>;
+			phy-names = "usb";
+			power-domains = <&cpg>;
+			status = "disabled";
+		};
+
+		ehci0: usb@11e10100 {
+			compatible = "generic-ehci";
+			reg = <0 0x11e10100 0 0x100>;
+			interrupts = <GIC_SPI 114 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD R9A08G046_USB_PCLK>,
+				 <&cpg CPG_MOD R9A08G046_USB_U2H0_HCLK>;
+			resets = <&phyrst 0>,
+				 <&cpg R9A08G046_USB_U2H0_HRESETN>;
+			phys = <&usb2_phy0 2>;
+			phy-names = "usb";
+			companion = <&ohci0>;
+			power-domains = <&cpg>;
+			status = "disabled";
+		};
+
+		ehci1: usb@11e90100 {
+			compatible = "generic-ehci";
+			reg = <0 0x11e90100 0 0x100>;
+			interrupts = <GIC_SPI 119 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD R9A08G046_USB_PCLK>,
+				 <&cpg CPG_MOD R9A08G046_USB_U2H1_HCLK>;
+			resets = <&phyrst 1>,
+				 <&cpg R9A08G046_USB_U2H1_HRESETN>;
+			phys = <&usb2_phy1 2>;
+			phy-names = "usb";
+			companion = <&ohci1>;
+			power-domains = <&cpg>;
+			status = "disabled";
+		};
+
+		usb2_phy0: usb-phy@11e10200 {
+			compatible = "renesas,usb2-phy-r9a08g046";
+			reg = <0 0x11e10200 0 0x700>;
+			interrupts = <GIC_SPI 116 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD R9A08G046_USB_PCLK>,
+				 <&cpg CPG_MOD R9A08G046_USB_U2H0_HCLK>;
+			resets = <&phyrst 0>,
+				 <&cpg R9A08G046_USB_U2H0_HRESETN>;
+			#phy-cells = <1>;
+			power-domains = <&cpg>;
+			status = "disabled";
+		};
+
+		usb2_phy1: usb-phy@11e90200 {
+			compatible = "renesas,usb2-phy-r9a08g046";
+			reg = <0 0x11e90200 0 0x700>;
+			interrupts = <GIC_SPI 121 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD R9A08G046_USB_PCLK>,
+				 <&cpg CPG_MOD R9A08G046_USB_U2H1_HCLK>;
+			resets = <&phyrst 1>,
+				 <&cpg R9A08G046_USB_U2H1_HRESETN>;
+			#phy-cells = <1>;
+			power-domains = <&cpg>;
+			status = "disabled";
+		};
+
 		pcie: pcie@11e40000 {
 			reg = <0 0x11e40000 0 0x10000>;
 			ranges = <0x02000000 0 0x30000000 0 0x30000000 0 0x08000000>;
-- 
2.54.0


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [PATCH v3 2/4] arm64: dts: renesas: r9a08g046: Add USB2.0 function device nodes
  2026-09-09  8:54 [PATCH v3 0/4] Add USB2.0 support for RZ/G3E Tommaso Merciai
  2026-09-09  8:54 ` [PATCH v3 1/4] arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes Tommaso Merciai
@ 2026-09-09  8:54 ` Tommaso Merciai
  2026-09-09  8:55 ` [PATCH v3 3/4] arm64: dts: renesas: r9a09g047: Add USB2.0 support Tommaso Merciai
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 14+ messages in thread
From: Tommaso Merciai @ 2026-09-09  8:54 UTC (permalink / raw)
  To: tomm.merciai, prabhakar.mahadev-lad.rj
  Cc: linux-renesas-soc, biju.das.jz, Geert Uytterhoeven, Magnus Damm,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, devicetree,
	linux-kernel, Tommaso Merciai

From: Biju Das <biju.das.jz@bp.renesas.com>

Add USB2.0 function device nodes to the RZ/G3L (r9a08g046) SoC DTSI.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>
---
v2->v3:
 - No changes just rebased on top of next-20260908.

v1->v2:
 - Cherry picked from "Add RZ/G3L USB2.0 host support" v7.
   v7: https://lore.kernel.org/all/20260812141940.246223-8-biju.das.jz@bp.renesas.com/

 arch/arm64/boot/dts/renesas/r9a08g046.dtsi | 38 ++++++++++++++++++++++
 1 file changed, 38 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r9a08g046.dtsi b/arch/arm64/boot/dts/renesas/r9a08g046.dtsi
index 10123281c53c..a2773e813dcd 100644
--- a/arch/arm64/boot/dts/renesas/r9a08g046.dtsi
+++ b/arch/arm64/boot/dts/renesas/r9a08g046.dtsi
@@ -1231,6 +1231,44 @@ usb2_phy1: usb-phy@11e90200 {
 			status = "disabled";
 		};
 
+		hsusb0: usb@11e30000 {
+			compatible = "renesas,usbhs-r9a08g046",
+				     "renesas,rzg2l-usbhs";
+			reg = <0 0x11e30000 0 0x10000>;
+			interrupts = <GIC_SPI 123 IRQ_TYPE_EDGE_RISING>,
+				     <GIC_SPI 124 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 125 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD R9A08G046_USB_PCLK>,
+				 <&cpg CPG_MOD R9A08G046_USB_U2P0_EXR_CPUCLK>;
+			resets = <&phyrst 0>,
+				 <&cpg R9A08G046_USB_U2P0_EXL_SYSRST>;
+			renesas,buswait = <7>;
+			phys = <&usb2_phy0 3>;
+			phy-names = "usb";
+			power-domains = <&cpg>;
+			status = "disabled";
+		};
+
+		hsusb1: usb@11eb0000 {
+			compatible = "renesas,usbhs-r9a08g046",
+				     "renesas,rzg2l-usbhs";
+			reg = <0 0x11eb0000 0 0x10000>;
+			interrupts = <GIC_SPI 127 IRQ_TYPE_EDGE_RISING>,
+				     <GIC_SPI 128 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 129 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 126 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD R9A08G046_USB_PCLK>,
+				 <&cpg CPG_MOD R9A08G046_USB_U2P1_EXR_CPUCLK>;
+			resets = <&phyrst 1>,
+				 <&cpg R9A08G046_USB_U2P1_EXL_SYSRST>;
+			renesas,buswait = <7>;
+			phys = <&usb2_phy1 3>;
+			phy-names = "usb";
+			power-domains = <&cpg>;
+			status = "disabled";
+		};
+
 		pcie: pcie@11e40000 {
 			reg = <0 0x11e40000 0 0x10000>;
 			ranges = <0x02000000 0 0x30000000 0 0x30000000 0 0x08000000>;
-- 
2.54.0


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [PATCH v3 3/4] arm64: dts: renesas: r9a09g047: Add USB2.0 support
  2026-09-09  8:54 [PATCH v3 0/4] Add USB2.0 support for RZ/G3E Tommaso Merciai
  2026-09-09  8:54 ` [PATCH v3 1/4] arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes Tommaso Merciai
  2026-09-09  8:54 ` [PATCH v3 2/4] arm64: dts: renesas: r9a08g046: Add USB2.0 function " Tommaso Merciai
@ 2026-09-09  8:55 ` Tommaso Merciai
  2026-09-14 17:34   ` Geert Uytterhoeven
  2026-09-09  8:55 ` [PATCH v3 4/4] arm64: dts: renesas: r9a09g047e57-smarc: Enable " Tommaso Merciai
  2026-09-21  9:57 ` [PATCH v3 0/4] Add USB2.0 support for RZ/G3E Tommaso Merciai
  4 siblings, 1 reply; 14+ messages in thread
From: Tommaso Merciai @ 2026-09-09  8:55 UTC (permalink / raw)
  To: tomm.merciai, prabhakar.mahadev-lad.rj
  Cc: linux-renesas-soc, biju.das.jz, Tommaso Merciai,
	Geert Uytterhoeven, Magnus Damm, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, devicetree, linux-kernel

The Renesas RZ/G3E ("R9A09G047") SoC supports 1x channel with OTG/DRD
and 1x channel with host interface.

Add the EHCI, OHCI, USB2.0 PHY and reset control nodes for USB2.0 channels
in R9A09G047 SoC DTSI.

Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>
---
v2->v3:
 - No changes just rebased on top of next-20260908.

v1->v2:
 - No changes.

 arch/arm64/boot/dts/renesas/r9a09g047.dtsi | 124 +++++++++++++++++++++
 1 file changed, 124 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r9a09g047.dtsi b/arch/arm64/boot/dts/renesas/r9a09g047.dtsi
index 72120e2abf8b..56ceedfb8ae8 100644
--- a/arch/arm64/boot/dts/renesas/r9a09g047.dtsi
+++ b/arch/arm64/boot/dts/renesas/r9a09g047.dtsi
@@ -1773,6 +1773,130 @@ usb3_phy: usb-phy@15870000 {
 			status = "disabled";
 		};
 
+		ohci0: usb@15800000 {
+			compatible = "generic-ohci";
+			reg = <0 0x15800000 0 0x100>;
+			interrupts = <GIC_SPI 742 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD 0xb3>, <&cpg CPG_MOD 0xb6>;
+			resets = <&usb20phyrst>, <&cpg 0xac>;
+			phys = <&usb2_phy0 1>;
+			phy-names = "usb";
+			power-domains = <&cpg>;
+			status = "disabled";
+		};
+
+		ohci1: usb@15810000 {
+			compatible = "generic-ohci";
+			reg = <0 0x15810000 0 0x100>;
+			interrupts = <GIC_SPI 747 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD 0xb4>, <&cpg CPG_MOD 0xb7>;
+			resets = <&usb21phyrst>, <&cpg 0xad>;
+			phys = <&usb2_phy1 1>;
+			phy-names = "usb";
+			power-domains = <&cpg>;
+			status = "disabled";
+		};
+
+		ehci0: usb@15800100 {
+			compatible = "generic-ehci";
+			reg = <0 0x15800100 0 0x100>;
+			interrupts = <GIC_SPI 743 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD 0xb3>, <&cpg CPG_MOD 0xb6>;
+			resets = <&usb20phyrst>, <&cpg 0xac>;
+			phys = <&usb2_phy0 2>;
+			phy-names = "usb";
+			companion = <&ohci0>;
+			power-domains = <&cpg>;
+			status = "disabled";
+		};
+
+		ehci1: usb@15810100 {
+			compatible = "generic-ehci";
+			reg = <0 0x15810100 0 0x100>;
+			interrupts = <GIC_SPI 748 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD 0xb4>, <&cpg CPG_MOD 0xb7>;
+			resets = <&usb21phyrst>, <&cpg 0xad>;
+			phys = <&usb2_phy1 2>;
+			phy-names = "usb";
+			companion = <&ohci1>;
+			power-domains = <&cpg>;
+			status = "disabled";
+		};
+
+		usb2_phy0: usb-phy@15800200 {
+			compatible = "renesas,usb2-phy-r9a09g047",
+				     "renesas,usb2-phy-r9a09g057";
+			reg = <0 0x15800200 0 0x700>;
+			interrupts = <GIC_SPI 745 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD 0xb3>,
+				 <&cpg CPG_CORE R9A09G047_USB2_0_CLK_CORE0>;
+			clock-names = "fck", "usb_x1";
+			resets = <&usb20phyrst>, <&cpg 0xac>;
+			#phy-cells = <1>;
+			power-domains = <&cpg>;
+			mux-states = <&usb20phyrst 1>;
+			status = "disabled";
+
+			usb2_phy0_vbus_otg: vbus-regulator {
+				regulator-name = "USB2PHY0-VBUS-OTG";
+			};
+		};
+
+		usb2_phy1: usb-phy@15810200 {
+			compatible = "renesas,usb2-phy-r9a09g047",
+				     "renesas,usb2-phy-r9a09g057";
+			reg = <0 0x15810200 0 0x700>;
+			interrupts = <GIC_SPI 750 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD 0xb4>,
+				 <&cpg CPG_CORE R9A09G047_USB2_0_CLK_CORE1>;
+			clock-names = "fck", "usb_x1";
+			resets = <&usb21phyrst>, <&cpg 0xad>;
+			#phy-cells = <1>;
+			power-domains = <&cpg>;
+			mux-states = <&usb21phyrst 0>;
+			status = "disabled";
+		};
+
+		hsusb0: usb@15820000 {
+			compatible = "renesas,usbhs-r9a09g047",
+				     "renesas,rzg2l-usbhs";
+			reg = <0 0x15820000 0 0x10000>;
+			interrupts = <GIC_SPI 751 IRQ_TYPE_EDGE_RISING>,
+				     <GIC_SPI 752 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 753 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 754 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&cpg CPG_MOD 0xb3>, <&cpg CPG_MOD 0xb5>;
+			resets = <&usb20phyrst>, <&cpg 0xae>;
+			phys = <&usb2_phy0 3>;
+			phy-names = "usb";
+			power-domains = <&cpg>;
+			status = "disabled";
+		};
+
+		usb20phyrst: reset-controller@15830000 {
+			compatible = "renesas,r9a09g047-usb2phy-reset",
+				     "renesas,r9a09g057-usb2phy-reset";
+			reg = <0 0x15830000 0 0x10000>;
+			clocks = <&cpg CPG_MOD 0xb6>;
+			resets = <&cpg 0xaf>;
+			power-domains = <&cpg>;
+			#reset-cells = <0>;
+			#mux-state-cells = <1>;
+			status = "disabled";
+		};
+
+		usb21phyrst: reset-controller@15840000 {
+			compatible = "renesas,r9a09g047-usb2phy-reset",
+				     "renesas,r9a09g057-usb2phy-reset";
+			reg = <0 0x15840000 0 0x10000>;
+			clocks = <&cpg CPG_MOD 0xb7>;
+			resets = <&cpg 0xaf>;
+			power-domains = <&cpg>;
+			#reset-cells = <0>;
+			#mux-state-cells = <1>;
+			status = "disabled";
+		};
+
 		sdhi0: mmc@15c00000 {
 			compatible = "renesas,sdhi-r9a09g047", "renesas,sdhi-r9a09g057";
 			reg = <0x0 0x15c00000 0 0x10000>;
-- 
2.54.0


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [PATCH v3 4/4] arm64: dts: renesas: r9a09g047e57-smarc: Enable USB2.0 support
  2026-09-09  8:54 [PATCH v3 0/4] Add USB2.0 support for RZ/G3E Tommaso Merciai
                   ` (2 preceding siblings ...)
  2026-09-09  8:55 ` [PATCH v3 3/4] arm64: dts: renesas: r9a09g047: Add USB2.0 support Tommaso Merciai
@ 2026-09-09  8:55 ` Tommaso Merciai
  2026-09-14 17:42   ` Geert Uytterhoeven
  2026-09-21  9:57 ` [PATCH v3 0/4] Add USB2.0 support for RZ/G3E Tommaso Merciai
  4 siblings, 1 reply; 14+ messages in thread
From: Tommaso Merciai @ 2026-09-09  8:55 UTC (permalink / raw)
  To: tomm.merciai, prabhakar.mahadev-lad.rj
  Cc: linux-renesas-soc, biju.das.jz, Tommaso Merciai,
	Geert Uytterhoeven, Magnus Damm, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, devicetree, linux-kernel

Enable USB2.0 support on the RZ/G3E EVK board. The USB1B_1A_HOST and
USB5_4_HOST connectors support host operation only, while USB0_OTG
supports both host and peripheral operation.

renesas-smarc2.dtsi is shared by all boards based on the RZ SMARC
Carrier II Board. USB2.0 is not yet supported on the RZ/G3L SMARC
EVK, so disable the common USB2.0 nodes in its board DTS until proper
support is added.

Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>
---
v2->v3:
 - No changes just rebased on top of next-20260908.

v1->v2:
 - Disabled RZ/G3L common nodes from r9a08g046l48-smarc.dts in this way
   we can drop placeholder patch.
 - Update commit body.
 - Improved usb20_pins, usb21_pins comments.

 .../boot/dts/renesas/r9a08g046l48-smarc.dts   | 20 +++++++++
 .../boot/dts/renesas/r9a09g047e57-smarc.dts   | 45 +++++++++++++++++++
 .../boot/dts/renesas/renesas-smarc2.dtsi      | 25 +++++++++++
 3 files changed, 90 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r9a08g046l48-smarc.dts b/arch/arm64/boot/dts/renesas/r9a08g046l48-smarc.dts
index 5289efd1a430..ccd6e6394532 100644
--- a/arch/arm64/boot/dts/renesas/r9a08g046l48-smarc.dts
+++ b/arch/arm64/boot/dts/renesas/r9a08g046l48-smarc.dts
@@ -71,6 +71,18 @@ codec_dai: codec {
 #endif
 };
 
+&ehci0 {
+	status = "disabled";
+};
+
+&ehci1 {
+	status = "disabled";
+};
+
+&hsusb0 {
+	status = "disabled";
+};
+
 &i2c2 {
 	pinctrl-0 = <&i2c2_pins>;
 	pinctrl-names = "default";
@@ -122,6 +134,14 @@ &keys {
 #endif
 };
 
+&ohci0 {
+	status = "disabled";
+};
+
+&ohci1 {
+	status = "disabled";
+};
+
 &pinctrl {
 	audio_clk_pins: audio-clock {
 		pinmux = <RZG3L_PORT_PINMUX(H, 4, 6)>, /* AUDIO_CLK_B */
diff --git a/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts b/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
index 469a6282b2a3..36482357c2fc 100644
--- a/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
+++ b/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
@@ -298,6 +298,28 @@ sd1-data {
 		};
 	};
 
+	usb20_pins: usb20 {
+		ovc {
+			pinmux = <RZG3E_PORT_PINMUX(0, 0, 12)>; /* USB20_OVRCURN */
+			bias-pull-up;
+		};
+
+		vbus {
+			pinmux = <RZG3E_PORT_PINMUX(0, 1, 12)>; /* USB20_VBUSEN */
+		};
+	};
+
+	usb21_pins: usb21 {
+		ovc {
+			pinmux = <RZG3E_PORT_PINMUX(G, 6, 12)>; /* USB21_OVRCURN */
+			bias-pull-up;
+		};
+
+		vbus {
+			pinmux = <RZG3E_PORT_PINMUX(K, 3, 12)>; /* USB21_VBUSEN */
+		};
+	};
+
 	usb3_pins: usb3 {
 		pinmux = <RZG3E_PORT_PINMUX(4, 1, 12)>, /* USB30_VBUSEN */
 			 <RZG3E_PORT_PINMUX(4, 0, 12)>; /* USB30_OVRCURN */
@@ -394,6 +416,29 @@ &ssi4 {
 	shared-pin;
 };
 
+&usb20phyrst {
+	status = "okay";
+};
+
+&usb21phyrst {
+	status = "okay";
+};
+
+&usb2_phy0 {
+	pinctrl-0 = <&usb20_pins>;
+	pinctrl-names = "default";
+
+	vbus-supply = <&usb2_phy0_vbus_otg>;
+	status = "okay";
+};
+
+&usb2_phy1 {
+	pinctrl-0 = <&usb21_pins>;
+	pinctrl-names = "default";
+
+	status = "okay";
+};
+
 &usb3_phy {
 	status = "okay";
 };
diff --git a/arch/arm64/boot/dts/renesas/renesas-smarc2.dtsi b/arch/arm64/boot/dts/renesas/renesas-smarc2.dtsi
index 696a933af808..e07fd9396adb 100644
--- a/arch/arm64/boot/dts/renesas/renesas-smarc2.dtsi
+++ b/arch/arm64/boot/dts/renesas/renesas-smarc2.dtsi
@@ -91,11 +91,36 @@ &canfd {
 	status = "okay";
 };
 
+&ehci0 {
+	dr_mode = "otg";
+	status = "okay";
+};
+
+&ehci1 {
+	dr_mode = "host";
+	status = "okay";
+};
+
+&hsusb0 {
+	dr_mode = "otg";
+	status = "okay";
+};
+
 &i2c0 {
 	status = "okay";
 	clock-frequency = <400000>;
 };
 
+&ohci0 {
+	dr_mode = "otg";
+	status = "okay";
+};
+
+&ohci1 {
+	dr_mode = "host";
+	status = "okay";
+};
+
 &pcie {
 	status = "okay";
 };
-- 
2.54.0


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH v3 1/4] arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes
  2026-09-09  8:54 ` [PATCH v3 1/4] arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes Tommaso Merciai
@ 2026-09-14 17:28   ` Geert Uytterhoeven
  2026-09-15  9:40     ` Biju Das
  0 siblings, 1 reply; 14+ messages in thread
From: Geert Uytterhoeven @ 2026-09-14 17:28 UTC (permalink / raw)
  To: Tommaso Merciai, biju.das.jz
  Cc: tomm.merciai, prabhakar.mahadev-lad.rj, linux-renesas-soc,
	Magnus Damm, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	devicetree, linux-kernel

Hi Tommaso, Biju,

On Wed, 9 Sept 2026 at 10:55, Tommaso Merciai
<tommaso.merciai.xr@bp.renesas.com> wrote:
> From: Biju Das <biju.das.jz@bp.renesas.com>
>
> Add USB2.0 device nodes to the RZ/G3L (r9a08g046) SoC DTSI, covering
> the USB PHY controller, OHCI/EHCI host controllers, and USB2 PHYs for
> both ports.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>

Thanks for your patch!

> --- a/arch/arm64/boot/dts/renesas/r9a08g046.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r9a08g046.dtsi

> +               usb2_phy0: usb-phy@11e10200 {
> +                       compatible = "renesas,usb2-phy-r9a08g046";
> +                       reg = <0 0x11e10200 0 0x700>;
> +                       interrupts = <GIC_SPI 116 IRQ_TYPE_LEVEL_HIGH>;
> +                       clocks = <&cpg CPG_MOD R9A08G046_USB_PCLK>,
> +                                <&cpg CPG_MOD R9A08G046_USB_U2H0_HCLK>;

No clock-names, while other SoCs using the same bindings do have it?

> +                       resets = <&phyrst 0>,
> +                                <&cpg R9A08G046_USB_U2H0_HRESETN>;
> +                       #phy-cells = <1>;
> +                       power-domains = <&cpg>;
> +                       status = "disabled";
> +               };
> +
> +               usb2_phy1: usb-phy@11e90200 {
> +                       compatible = "renesas,usb2-phy-r9a08g046";
> +                       reg = <0 0x11e90200 0 0x700>;
> +                       interrupts = <GIC_SPI 121 IRQ_TYPE_LEVEL_HIGH>;
> +                       clocks = <&cpg CPG_MOD R9A08G046_USB_PCLK>,
> +                                <&cpg CPG_MOD R9A08G046_USB_U2H1_HCLK>;

Likewise.

> +                       resets = <&phyrst 1>,
> +                                <&cpg R9A08G046_USB_U2H1_HRESETN>;
> +                       #phy-cells = <1>;
> +                       power-domains = <&cpg>;
> +                       status = "disabled";
> +               };
> +
>                 pcie: pcie@11e40000 {
>                         reg = <0 0x11e40000 0 0x10000>;
>                         ranges = <0x02000000 0 0x30000000 0 0x30000000 0 0x08000000>;

Looks like U2H[01]_INT and U2H[01]_WAKEON_INT are missing,
but they are also not listed in the DT bindings?

The rest LGTM.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH v3 3/4] arm64: dts: renesas: r9a09g047: Add USB2.0 support
  2026-09-09  8:55 ` [PATCH v3 3/4] arm64: dts: renesas: r9a09g047: Add USB2.0 support Tommaso Merciai
@ 2026-09-14 17:34   ` Geert Uytterhoeven
  0 siblings, 0 replies; 14+ messages in thread
From: Geert Uytterhoeven @ 2026-09-14 17:34 UTC (permalink / raw)
  To: Tommaso Merciai
  Cc: tomm.merciai, prabhakar.mahadev-lad.rj, linux-renesas-soc,
	biju.das.jz, Magnus Damm, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, devicetree, linux-kernel

On Wed, 9 Sept 2026 at 10:55, Tommaso Merciai
<tommaso.merciai.xr@bp.renesas.com> wrote:
> The Renesas RZ/G3E ("R9A09G047") SoC supports 1x channel with OTG/DRD
> and 1x channel with host interface.
>
> Add the EHCI, OHCI, USB2.0 PHY and reset control nodes for USB2.0 channels
> in R9A09G047 SoC DTSI.
>
> Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH v3 4/4] arm64: dts: renesas: r9a09g047e57-smarc: Enable USB2.0 support
  2026-09-09  8:55 ` [PATCH v3 4/4] arm64: dts: renesas: r9a09g047e57-smarc: Enable " Tommaso Merciai
@ 2026-09-14 17:42   ` Geert Uytterhoeven
  2026-09-21 10:06     ` Tommaso Merciai
  0 siblings, 1 reply; 14+ messages in thread
From: Geert Uytterhoeven @ 2026-09-14 17:42 UTC (permalink / raw)
  To: Tommaso Merciai
  Cc: tomm.merciai, prabhakar.mahadev-lad.rj, linux-renesas-soc,
	biju.das.jz, Geert Uytterhoeven, Magnus Damm, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, devicetree, linux-kernel

Hi Tommaso,

On Wed, 9 Sept 2026 at 10:55, Tommaso Merciai
<tommaso.merciai.xr@bp.renesas.com> wrote:
> Enable USB2.0 support on the RZ/G3E EVK board. The USB1B_1A_HOST and
> USB5_4_HOST connectors support host operation only, while USB0_OTG
> supports both host and peripheral operation.
>
> renesas-smarc2.dtsi is shared by all boards based on the RZ SMARC
> Carrier II Board. USB2.0 is not yet supported on the RZ/G3L SMARC
> EVK, so disable the common USB2.0 nodes in its board DTS until proper
> support is added.
>
> Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>

Thanks for your patch!

> --- a/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
> +++ b/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
> @@ -298,6 +298,28 @@ sd1-data {
>                 };
>         };
>
> +       usb20_pins: usb20 {
> +               ovc {
> +                       pinmux = <RZG3E_PORT_PINMUX(0, 0, 12)>; /* USB20_OVRCURN */
> +                       bias-pull-up;
> +               };
> +
> +               vbus {
> +                       pinmux = <RZG3E_PORT_PINMUX(0, 1, 12)>; /* USB20_VBUSEN */
> +               };
> +       };
> +
> +       usb21_pins: usb21 {
> +               ovc {
> +                       pinmux = <RZG3E_PORT_PINMUX(G, 6, 12)>; /* USB21_OVRCURN */
> +                       bias-pull-up;
> +               };
> +
> +               vbus {
> +                       pinmux = <RZG3E_PORT_PINMUX(K, 3, 12)>; /* USB21_VBUSEN */
> +               };

While these two signals are wired to the SLG46108, which generates
USB1_EN_OC# from that, the latter does not seem to be wired on the
carrier board (SMARC_MODULE_CN P67)?

> +       };
> +
>         usb3_pins: usb3 {
>                 pinmux = <RZG3E_PORT_PINMUX(4, 1, 12)>, /* USB30_VBUSEN */
>                          <RZG3E_PORT_PINMUX(4, 0, 12)>; /* USB30_OVRCURN */

The rest LGTM.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 14+ messages in thread

* RE: [PATCH v3 1/4] arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes
  2026-09-14 17:28   ` Geert Uytterhoeven
@ 2026-09-15  9:40     ` Biju Das
  0 siblings, 0 replies; 14+ messages in thread
From: Biju Das @ 2026-09-15  9:40 UTC (permalink / raw)
  To: geert, Tommaso Merciai
  Cc: Tommaso Merciai, Prabhakar Mahadev Lad, linux-renesas-soc,
	magnus.damm, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	devicetree, linux-kernel

Hi Geert,

Thanks for the feedback.

> -----Original Message-----
> From: Geert Uytterhoeven <geert@linux-m68k.org>
> Sent: 14 September 2026 18:29
> Subject: Re: [PATCH v3 1/4] arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes
> 
> Hi Tommaso, Biju,
> 
> On Wed, 9 Sept 2026 at 10:55, Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com> wrote:
> > From: Biju Das <biju.das.jz@bp.renesas.com>
> >
> > Add USB2.0 device nodes to the RZ/G3L (r9a08g046) SoC DTSI, covering
> > the USB PHY controller, OHCI/EHCI host controllers, and USB2 PHYs for
> > both ports.
> >
> > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> > Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>
> 
> Thanks for your patch!
> 
> > --- a/arch/arm64/boot/dts/renesas/r9a08g046.dtsi
> > +++ b/arch/arm64/boot/dts/renesas/r9a08g046.dtsi
> 
> > +               usb2_phy0: usb-phy@11e10200 {
> > +                       compatible = "renesas,usb2-phy-r9a08g046";
> > +                       reg = <0 0x11e10200 0 0x700>;
> > +                       interrupts = <GIC_SPI 116 IRQ_TYPE_LEVEL_HIGH>;
> > +                       clocks = <&cpg CPG_MOD R9A08G046_USB_PCLK>,
> > +                                <&cpg CPG_MOD
> > + R9A08G046_USB_U2H0_HCLK>;
> 
> No clock-names, while other SoCs using the same bindings do have it?

Ok, I see clock names fck and usb_x1 for non RZ/G2L SoCs.

Shall we create a patch with pclk and hclk for all RZ/G2L SoCs (binding + SoC DTSI)??

> 
> > +                       resets = <&phyrst 0>,
> > +                                <&cpg R9A08G046_USB_U2H0_HRESETN>;
> > +                       #phy-cells = <1>;
> > +                       power-domains = <&cpg>;
> > +                       status = "disabled";
> > +               };
> > +
> > +               usb2_phy1: usb-phy@11e90200 {
> > +                       compatible = "renesas,usb2-phy-r9a08g046";
> > +                       reg = <0 0x11e90200 0 0x700>;
> > +                       interrupts = <GIC_SPI 121 IRQ_TYPE_LEVEL_HIGH>;
> > +                       clocks = <&cpg CPG_MOD R9A08G046_USB_PCLK>,
> > +                                <&cpg CPG_MOD
> > + R9A08G046_USB_U2H1_HCLK>;
> 
> Likewise.
> 
> > +                       resets = <&phyrst 1>,
> > +                                <&cpg R9A08G046_USB_U2H1_HRESETN>;
> > +                       #phy-cells = <1>;
> > +                       power-domains = <&cpg>;
> > +                       status = "disabled";
> > +               };
> > +
> >                 pcie: pcie@11e40000 {
> >                         reg = <0 0x11e40000 0 0x10000>;
> >                         ranges = <0x02000000 0 0x30000000 0 0x30000000
> > 0 0x08000000>;
> 
> Looks like U2H[01]_INT and U2H[01]_WAKEON_INT are missing, but they are also not listed in the DT
> bindings?

These interrupts are not part of UCOM module, that is the reason it is not listed in
DT bindings.

Cheers,
Biju

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH v3 0/4] Add USB2.0 support for RZ/G3E
  2026-09-09  8:54 [PATCH v3 0/4] Add USB2.0 support for RZ/G3E Tommaso Merciai
                   ` (3 preceding siblings ...)
  2026-09-09  8:55 ` [PATCH v3 4/4] arm64: dts: renesas: r9a09g047e57-smarc: Enable " Tommaso Merciai
@ 2026-09-21  9:57 ` Tommaso Merciai
  2026-09-21 12:31   ` Geert Uytterhoeven
  4 siblings, 1 reply; 14+ messages in thread
From: Tommaso Merciai @ 2026-09-21  9:57 UTC (permalink / raw)
  To: tomm.merciai, prabhakar.mahadev-lad.rj
  Cc: linux-renesas-soc, biju.das.jz, Geert Uytterhoeven, Magnus Damm,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, devicetree,
	linux-kernel

Hi Geert,

On Wed, Sep 09, 2026 at 10:54:57AM +0200, Tommaso Merciai wrote:
> Dear All,
> 
> This patch series adds USB2.0 support for the Renesas R9A09G047 SoC and
> enables it on the RZ/G3E SMARK II board.
> 
> Kind regards,
> Tommaso
> 
> v2->v3:
>  - No changes just rebased on top of next-20260908.
> 
> v1->v2:
>  - Included PATCH v7 7/9 and PATCH v7 6/9 from
>    v7: https://lore.kernel.org/all/20260812141940.246223-1-biju.das.jz@bp.renesas.com/
>  - Drop placeholder patch
>  - PATCH 4/4: Disabled RZ/G3L common nodes from r9a08g046l48-smarc.dts.
>    Update commit body, and improved usb20_pins, usb21_pins comments.
> 
> Biju Das (2):
>   arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes
>   arm64: dts: renesas: r9a08g046: Add USB2.0 function device nodes
> 
> Tommaso Merciai (2):
>   arm64: dts: renesas: r9a09g047: Add USB2.0 support
>   arm64: dts: renesas: r9a09g047e57-smarc: Enable USB2.0 support

RZ/G3L USB2.0 binding patch is not queued.
Shall I send a new version without RZ/G3L patches ie, using placeholders [1] ?


[1] https://lore.kernel.org/all/aec1fce5558c6c88725fe710a2f9e4b1eb9e8d64.1787062965.git.tommaso.merciai.xr@bp.renesas.com/

Kind regards,
Tommaso

> 
>  arch/arm64/boot/dts/renesas/r9a08g046.dtsi    | 142 ++++++++++++++++++
>  .../boot/dts/renesas/r9a08g046l48-smarc.dts   |  20 +++
>  arch/arm64/boot/dts/renesas/r9a09g047.dtsi    | 124 +++++++++++++++
>  .../boot/dts/renesas/r9a09g047e57-smarc.dts   |  45 ++++++
>  .../boot/dts/renesas/renesas-smarc2.dtsi      |  25 +++
>  5 files changed, 356 insertions(+)
> 
> -- 
> 2.54.0
> 

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH v3 4/4] arm64: dts: renesas: r9a09g047e57-smarc: Enable USB2.0 support
  2026-09-14 17:42   ` Geert Uytterhoeven
@ 2026-09-21 10:06     ` Tommaso Merciai
  0 siblings, 0 replies; 14+ messages in thread
From: Tommaso Merciai @ 2026-09-21 10:06 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: tomm.merciai, prabhakar.mahadev-lad.rj, linux-renesas-soc,
	biju.das.jz, Geert Uytterhoeven, Magnus Damm, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, devicetree, linux-kernel

Hi Geert,
Thanks for your review.

On Mon, Sep 14, 2026 at 07:42:34PM +0200, Geert Uytterhoeven wrote:
> Hi Tommaso,
> 
> On Wed, 9 Sept 2026 at 10:55, Tommaso Merciai
> <tommaso.merciai.xr@bp.renesas.com> wrote:
> > Enable USB2.0 support on the RZ/G3E EVK board. The USB1B_1A_HOST and
> > USB5_4_HOST connectors support host operation only, while USB0_OTG
> > supports both host and peripheral operation.
> >
> > renesas-smarc2.dtsi is shared by all boards based on the RZ SMARC
> > Carrier II Board. USB2.0 is not yet supported on the RZ/G3L SMARC
> > EVK, so disable the common USB2.0 nodes in its board DTS until proper
> > support is added.
> >
> > Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>
> 
> Thanks for your patch!
> 
> > --- a/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
> > +++ b/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
> > @@ -298,6 +298,28 @@ sd1-data {
> >                 };
> >         };
> >
> > +       usb20_pins: usb20 {
> > +               ovc {
> > +                       pinmux = <RZG3E_PORT_PINMUX(0, 0, 12)>; /* USB20_OVRCURN */
> > +                       bias-pull-up;
> > +               };
> > +
> > +               vbus {
> > +                       pinmux = <RZG3E_PORT_PINMUX(0, 1, 12)>; /* USB20_VBUSEN */
> > +               };
> > +       };
> > +
> > +       usb21_pins: usb21 {
> > +               ovc {
> > +                       pinmux = <RZG3E_PORT_PINMUX(G, 6, 12)>; /* USB21_OVRCURN */
> > +                       bias-pull-up;
> > +               };
> > +
> > +               vbus {
> > +                       pinmux = <RZG3E_PORT_PINMUX(K, 3, 12)>; /* USB21_VBUSEN */
> > +               };
> 
> While these two signals are wired to the SLG46108, which generates
> USB1_EN_OC# from that, the latter does not seem to be wired on the
> carrier board (SMARC_MODULE_CN P67)?

You are correct, USB1_EN_OC# -> P67 is not wired on the carrier board.
I will drop usb21_pins in v4.

Kind regards,
Tommaso


> 
> > +       };
> > +
> >         usb3_pins: usb3 {
> >                 pinmux = <RZG3E_PORT_PINMUX(4, 1, 12)>, /* USB30_VBUSEN */
> >                          <RZG3E_PORT_PINMUX(4, 0, 12)>; /* USB30_OVRCURN */
> 
> The rest LGTM.
> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> -- 
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH v3 0/4] Add USB2.0 support for RZ/G3E
  2026-09-21  9:57 ` [PATCH v3 0/4] Add USB2.0 support for RZ/G3E Tommaso Merciai
@ 2026-09-21 12:31   ` Geert Uytterhoeven
  2026-09-21 15:45     ` Biju Das
  2026-09-21 16:24     ` Tommaso Merciai
  0 siblings, 2 replies; 14+ messages in thread
From: Geert Uytterhoeven @ 2026-09-21 12:31 UTC (permalink / raw)
  To: Tommaso Merciai
  Cc: tomm.merciai, prabhakar.mahadev-lad.rj, linux-renesas-soc,
	biju.das.jz, Magnus Damm, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, devicetree, linux-kernel, Bartosz Golaszewski

Hi Tommaso,

On Mon, 21 Sept 2026 at 11:57, Tommaso Merciai
<tommaso.merciai.xr@bp.renesas.com> wrote:
> On Wed, Sep 09, 2026 at 10:54:57AM +0200, Tommaso Merciai wrote:
> > This patch series adds USB2.0 support for the Renesas R9A09G047 SoC and
> > enables it on the RZ/G3E SMARK II board.
> >
> > Kind regards,
> > Tommaso
> >
> > v2->v3:
> >  - No changes just rebased on top of next-20260908.
> >
> > v1->v2:
> >  - Included PATCH v7 7/9 and PATCH v7 6/9 from
> >    v7: https://lore.kernel.org/all/20260812141940.246223-1-biju.das.jz@bp.renesas.com/
> >  - Drop placeholder patch
> >  - PATCH 4/4: Disabled RZ/G3L common nodes from r9a08g046l48-smarc.dts.
> >    Update commit body, and improved usb20_pins, usb21_pins comments.
> >
> > Biju Das (2):
> >   arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes
> >   arm64: dts: renesas: r9a08g046: Add USB2.0 function device nodes
> >
> > Tommaso Merciai (2):
> >   arm64: dts: renesas: r9a09g047: Add USB2.0 support
> >   arm64: dts: renesas: r9a09g047e57-smarc: Enable USB2.0 support
>
> RZ/G3L USB2.0 binding patch is not queued.
> Shall I send a new version without RZ/G3L patches ie, using placeholders [1] ?

That might be a good idea, to unblock mainlining of RZ/G3E USB.

However, it looks like the only missing piece is DT bindings for
"renesas,r9a08g046-usbphy-ctrl"? How come we ended up with
a driver in linux-next without bindings?

> [1] https://lore.kernel.org/all/aec1fce5558c6c88725fe710a2f9e4b1eb9e8d64.1787062965.git.tommaso.merciai.xr@bp.renesas.com/

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 14+ messages in thread

* RE: [PATCH v3 0/4] Add USB2.0 support for RZ/G3E
  2026-09-21 12:31   ` Geert Uytterhoeven
@ 2026-09-21 15:45     ` Biju Das
  2026-09-21 16:24     ` Tommaso Merciai
  1 sibling, 0 replies; 14+ messages in thread
From: Biju Das @ 2026-09-21 15:45 UTC (permalink / raw)
  To: geert, Tommaso Merciai
  Cc: Tommaso Merciai, Prabhakar Mahadev Lad, linux-renesas-soc,
	magnus.damm, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	devicetree, linux-kernel, Bartosz Golaszewski, Philipp Zabel

Hi Geert,

Thanks for the feedback.

> -----Original Message-----
> From: Geert Uytterhoeven <geert@linux-m68k.org>
> Sent: 21 September 2026 13:31
> Subject: Re: [PATCH v3 0/4] Add USB2.0 support for RZ/G3E
> 
> Hi Tommaso,
> 
> On Mon, 21 Sept 2026 at 11:57, Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com> wrote:
> > On Wed, Sep 09, 2026 at 10:54:57AM +0200, Tommaso Merciai wrote:
> > > This patch series adds USB2.0 support for the Renesas R9A09G047 SoC
> > > and enables it on the RZ/G3E SMARK II board.
> > >
> > > Kind regards,
> > > Tommaso
> > >
> > > v2->v3:
> > >  - No changes just rebased on top of next-20260908.
> > >
> > > v1->v2:
> > >  - Included PATCH v7 7/9 and PATCH v7 6/9 from
> > >    v7:
> > > https://lore.kernel.org/all/20260812141940.246223-1-biju.das.jz@bp.r
> > > enesas.com/
> > >  - Drop placeholder patch
> > >  - PATCH 4/4: Disabled RZ/G3L common nodes from r9a08g046l48-smarc.dts.
> > >    Update commit body, and improved usb20_pins, usb21_pins comments.
> > >
> > > Biju Das (2):
> > >   arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes
> > >   arm64: dts: renesas: r9a08g046: Add USB2.0 function device nodes
> > >
> > > Tommaso Merciai (2):
> > >   arm64: dts: renesas: r9a09g047: Add USB2.0 support
> > >   arm64: dts: renesas: r9a09g047e57-smarc: Enable USB2.0 support
> >
> > RZ/G3L USB2.0 binding patch is not queued.
> > Shall I send a new version without RZ/G3L patches ie, using placeholders [1] ?
> 
> That might be a good idea, to unblock mainlining of RZ/G3E USB.
> 
> However, it looks like the only missing piece is DT bindings for "renesas,r9a08g046-usbphy-ctrl"? How
> come we ended up with a driver in linux-next without bindings?

USB phy ctrl Binding + driver patches[1] are still not queued.
But dsi binding patch is queued without the driver change[2]

[1] https://lore.kernel.org/all/20260828133509.276281-1-biju.das.jz@bp.renesas.com/
[2] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=a4377c7722b824d3d2eb940eb681cf37b0969a24

Cheers,
Biju




^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH v3 0/4] Add USB2.0 support for RZ/G3E
  2026-09-21 12:31   ` Geert Uytterhoeven
  2026-09-21 15:45     ` Biju Das
@ 2026-09-21 16:24     ` Tommaso Merciai
  1 sibling, 0 replies; 14+ messages in thread
From: Tommaso Merciai @ 2026-09-21 16:24 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: tomm.merciai, prabhakar.mahadev-lad.rj, linux-renesas-soc,
	biju.das.jz, Magnus Damm, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, devicetree, linux-kernel, Bartosz Golaszewski

Hi Geert,
Thanks for your feedbacks.

On Mon, Sep 21, 2026 at 02:31:26PM +0200, Geert Uytterhoeven wrote:
> Hi Tommaso,
> 
> On Mon, 21 Sept 2026 at 11:57, Tommaso Merciai
> <tommaso.merciai.xr@bp.renesas.com> wrote:
> > On Wed, Sep 09, 2026 at 10:54:57AM +0200, Tommaso Merciai wrote:
> > > This patch series adds USB2.0 support for the Renesas R9A09G047 SoC and
> > > enables it on the RZ/G3E SMARK II board.
> > >
> > > Kind regards,
> > > Tommaso
> > >
> > > v2->v3:
> > >  - No changes just rebased on top of next-20260908.
> > >
> > > v1->v2:
> > >  - Included PATCH v7 7/9 and PATCH v7 6/9 from
> > >    v7: https://lore.kernel.org/all/20260812141940.246223-1-biju.das.jz@bp.renesas.com/
> > >  - Drop placeholder patch
> > >  - PATCH 4/4: Disabled RZ/G3L common nodes from r9a08g046l48-smarc.dts.
> > >    Update commit body, and improved usb20_pins, usb21_pins comments.
> > >
> > > Biju Das (2):
> > >   arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes
> > >   arm64: dts: renesas: r9a08g046: Add USB2.0 function device nodes
> > >
> > > Tommaso Merciai (2):
> > >   arm64: dts: renesas: r9a09g047: Add USB2.0 support
> > >   arm64: dts: renesas: r9a09g047e57-smarc: Enable USB2.0 support
> >
> > RZ/G3L USB2.0 binding patch is not queued.
> > Shall I send a new version without RZ/G3L patches ie, using placeholders [1] ?
> 
> That might be a good idea, to unblock mainlining of RZ/G3E USB.

I will send v4 then, taking old [1] + PATCH 3/4 + PATCH 4/4 from this series.
PATCH 4/4 reworked to address your comments.

Kind regards,
Tommaso

> 
> However, it looks like the only missing piece is DT bindings for
> "renesas,r9a08g046-usbphy-ctrl"? How come we ended up with
> a driver in linux-next without bindings?
> 
> > [1] https://lore.kernel.org/all/aec1fce5558c6c88725fe710a2f9e4b1eb9e8d64.1787062965.git.tommaso.merciai.xr@bp.renesas.com/
> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> -- 
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds

^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2026-09-21 16:25 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-09  8:54 [PATCH v3 0/4] Add USB2.0 support for RZ/G3E Tommaso Merciai
2026-09-09  8:54 ` [PATCH v3 1/4] arm64: dts: renesas: r9a08g046: Add USB2.0 host/phy device nodes Tommaso Merciai
2026-09-14 17:28   ` Geert Uytterhoeven
2026-09-15  9:40     ` Biju Das
2026-09-09  8:54 ` [PATCH v3 2/4] arm64: dts: renesas: r9a08g046: Add USB2.0 function " Tommaso Merciai
2026-09-09  8:55 ` [PATCH v3 3/4] arm64: dts: renesas: r9a09g047: Add USB2.0 support Tommaso Merciai
2026-09-14 17:34   ` Geert Uytterhoeven
2026-09-09  8:55 ` [PATCH v3 4/4] arm64: dts: renesas: r9a09g047e57-smarc: Enable " Tommaso Merciai
2026-09-14 17:42   ` Geert Uytterhoeven
2026-09-21 10:06     ` Tommaso Merciai
2026-09-21  9:57 ` [PATCH v3 0/4] Add USB2.0 support for RZ/G3E Tommaso Merciai
2026-09-21 12:31   ` Geert Uytterhoeven
2026-09-21 15:45     ` Biju Das
2026-09-21 16:24     ` Tommaso Merciai

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®