Repository: qualcomm-linux/meta-qcom
Pull request #799: rb3gen2: power up the xHCI host controller 
Author: login: ldts
Created: 2025-03-13T09:29:58Z | Merged: 2025-03-24T08:38:16Z

This patch is **Pending** - not sure if we want to hold merging it in until it has been posted in the final form. In any case, here it is for reference.

[Documentation](https://docs.qualcomm.com/bundle/publicresource/topics/80-70017-8/pcie.html?vproduct=1601111740013072&version=1.3&facet=Device%20Interfaces#pcie-software-support-for-usb)

**Notes**

-  Requires the xHCI Renesas firmware in `/lib/firmware/renesas_usb_fw.mem`

**Test for my particular use case (important bit, notice the Gigabit Ethernet device)**
```
root@qcs6490-rb3gen2-core-kit:~# lsusb                                                                                                                                                                                                                                                                                                                                                  
Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub                                                                                                                                                                                                                                                                                                                          
Bus 001 Device 002: ID 05e3:0610 Genesys Logic, Inc. Hub                                                                                                                                                                                                                                                                                                                                
Bus 001 Device 003: ID 2109:2812 VIA Labs, Inc. VL812 Hub                                                                                                                                                                                                                                                                                                                               
Bus 001 Device 004: ID 24f0:0140 Metadot Das Keyboard 4                                                                                                                                                                                                                                                                                                                                 
Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub                                                                                                                                                                                                                                                                                                                          
Bus 002 Device 002: ID 05e3:0625 Genesys Logic, Inc. USB3.2 Hub                                                                                                                                                                                                                                                                                                                         
Bus 002 Device 003: ID 0b95:1790 ASIX Electronics Corp. AX88179 Gigabit Ethernet                                                                                                                                                                                                                                                                                                        
Bus 002 Device 004: ID 2109:0812 VIA Labs, Inc. VL812 Hub
``` 



Diff:
diff --git a/recipes-kernel/linux/linux-yocto-dev.bbappend b/recipes-kernel/linux/linux-yocto-dev.bbappend
index eb195bf6..dc259a43 100644
--- a/recipes-kernel/linux/linux-yocto-dev.bbappend
+++ b/recipes-kernel/linux/linux-yocto-dev.bbappend
@@ -12,6 +12,7 @@ SRC_URI:append:qcom = " \
     file://qcm6490-board-dts/0001-arm64-dts-qcom-sc7280-Add-support-for-camss.patch \
     file://qcm6490-board-dts/0002-arm64-dts-qcom-qcs6490-rb3gen2-vision-mezzanine-Add-.patch \
     file://qcm6490-board-dts/0002-media-dt-bindings-update-clocks-for-sc7280-camss.patch \
+    file://qcm6490-board-dts/0001-PENDING-enable-xHCI.patch \
     file://workarounds/0001-QCLINUX-arm64-dts-qcom-qcm6490-disable-sdhc1-for-ufs.patch \
     file://workarounds/0001-PENDING-arm64-dts-qcom-Remove-voltage-vote-support-f.patch \
     file://drivers/0003-PCI-Add-new-start_link-stop_link-function-ops.patch \
diff --git a/recipes-kernel/linux/linux-yocto-dev/configs/qcom.cfg b/recipes-kernel/linux/linux-yocto-dev/configs/qcom.cfg
index 45293eb2..ebca3261 100644
--- a/recipes-kernel/linux/linux-yocto-dev/configs/qcom.cfg
+++ b/recipes-kernel/linux/linux-yocto-dev/configs/qcom.cfg
@@ -8,3 +8,4 @@ CONFIG_CLK_QCM2290_GPUCC=m
 # RB3
 CONFIG_PCI_PWRCTL=y
 CONFIG_PCI_PWRCTRL_TC956X=m
+CONFIG_USB_XHCI_PCI_RENESAS=m
diff --git a/recipes-kernel/linux/linux-yocto-dev/qcm6490-board-dts/0001-PENDING-enable-xHCI.patch b/recipes-kernel/linux/linux-yocto-dev/qcm6490-board-dts/0001-PENDING-enable-xHCI.patch
new file mode 100644
index 00000000..c95f8404
--- /dev/null
+++ b/recipes-kernel/linux/linux-yocto-dev/qcm6490-board-dts/0001-PENDING-enable-xHCI.patch
@@ -0,0 +1,133 @@
+From 6364f1eef332977bec795bd5237c8b1aa33bb88b Mon Sep 17 00:00:00 2001
+From: Jorge Ramirez-Ortiz <jorge.ramirez@oss.qualcomm.com>
+Date: Wed, 5 Mar 2025 11:11:22 +0100
+Subject: [PATCH] dts: rb3gen2: enable Renesas xHCI
+
+Make sure relevant power is on and resets are let go of, to make the USB
+Ethernet adapter show up.
+
+Ported  to 6.14 from an original patch by Bjorn Andersson.
+
+Link: https://github.com/quic-bjorande/linux/commit/ef0d4badf252c10883a369e8fb217799868158b0
+Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
+Upstream-Status: Pending
+---
+ arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 76 ++++++++++++++++++++
+ 1 file changed, 76 insertions(+)
+
+diff --git a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
+index 13dbb24a3179..73959e5095bf 100644
+--- a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
++++ b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
+@@ -243,6 +243,31 @@ vdd_ntn_1p8: regulator-vdd-ntn-1p8 {
+ 		regulator-enable-ramp-delay = <10000>;
+ 	};
+ 
++        vreg-pcie0-1p05v {
++		compatible = "regulator-fixed";
++		regulator-name = "PCIE0_1.05V";
++		gpio = <&pm7250b_gpios 4 GPIO_ACTIVE_HIGH>;
++		regulator-min-microvolt = <1050000>;
++		regulator-max-microvolt = <1050000>;
++		enable-active-high;
++		pinctrl-0 = <&upd_pwr_en2_state>;
++		pinctrl-names = "default";
++
++		regulator-always-on;
++	};
++
++	vreg-pcie0_3p3v-dual {
++		compatible = "regulator-fixed";
++		regulator-name = "PCIE0_3.3V_Dual";
++		gpio = <&pm7250b_gpios 1 GPIO_ACTIVE_HIGH>;
++		regulator-min-microvolt = <3300000>;
++		regulator-max-microvolt = <3300000>;
++		enable-active-high;
++		pinctrl-0 = <&upd_pwr_en1_state>;
++		pinctrl-names = "default";
++
++		regulator-always-on;
++	};
+ };
+ 
+ &apps_rsc {
+@@ -780,6 +805,9 @@ pcie@0,0 {
+ 
+ 		i2c-parent = <&i2c0 0x77>;
+ 
++		pinctrl-0 = <&tc956x_rst_state>, <&upd_hub_rst_state>, <&usb_hub_rst_state>;
++		pinctrl-names = "default";
++
+ 		reset-gpios = <&pm8350c_gpios 1 GPIO_ACTIVE_LOW>;
+ 
+ 		pcie@1,0 {
+@@ -953,6 +981,27 @@ ntn_1p8_en: ntn-1p8-en-state {
+ 		output-enable;
+ 		power-source = <0>;
+ 	};
++
++	upd_hub_rst_state: upd-hub-rst-state {
++		pins = "gpio4";
++		function = "normal";
++
++		bias-disable;
++		input-disable;
++		output-enable;
++		output-high;
++		power-source = <0>;
++	};
++
++	tc956x_rst_state: tc956x-rst-state {
++		pins = "gpio1";
++		function = "normal";
++
++		bias-disable;
++		input-disable;
++		output-enable;
++		power-source = <0>;
++	};
+ };
+ 
+ &tlmm {
+@@ -1048,6 +1097,24 @@ &edp_hot_plug_det {
+ };
+ 
+ &pm7250b_gpios {
++	upd_pwr_en1_state: upd-pwr-en1-state {
++		pins = "gpio1";
++		function = "normal";
++
++		output-enable;
++		input-disable;
++		power-source = <0>;
++	};
++
++	upd_pwr_en2_state: upd-pwr-en2-state {
++		pins = "gpio4";
++		function = "normal";
++
++		output-enable;
++		input-disable;
++		power-source = <0>;
++	};
++
+ 	lt9611_rst_pin: lt9611-rst-state {
+ 		pins = "gpio2";
+ 		function = "normal";
+@@ -1101,4 +1168,13 @@ sd_cd: sd-cd-state {
+ 		function = "gpio";
+ 		bias-pull-up;
+ 	};
++
++	usb_hub_rst_state: usb-hub-rst-state {
++		pins = "gpio162";
++		function = "gpio";
++
++		drive-strength = <2>;
++		output-low;
++		bias-disable;
++       };
+ };
+-- 
+2.34.1
+


Reviews:
user: login: lumag
state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-03-13T14:36:31Z
user: login: lumag
state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-03-13T14:38:30Z
user: login: lumag
state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-03-13T14:39:20Z
user: login: ricardosalveti
state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-03-13T15:44:21Z
user: login: ldts
state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-03-13T15:48:22Z
user: login: ldts
state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-03-13T15:53:26Z
user: login: ldts
state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-03-13T15:54:59Z
user: login: lumag
state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-03-13T16:35:34Z
user: login: ldts
state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-03-13T16:58:47Z
user: login: ricardosalveti
state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-03-13T20:56:45Z
user: login: lumag
state: APPROVED
author_association: CONTRIBUTOR
submitted_at: 2025-03-14T07:40:51Z
user: login: lumag
state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-03-14T07:44:26Z
user: login: ndechesne
state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-03-14T17:00:42Z
user: login: lumag
state: COMMENTED
author_association: CONTRIBUTOR
submitted_at: 2025-03-14T20:47:19Z
user: login: ricardosalveti
state: APPROVED
author_association: CONTRIBUTOR
submitted_at: 2025-03-14T21:04:38Z
user: login: ndechesne
state: APPROVED
author_association: CONTRIBUTOR
submitted_at: 2025-03-24T08:38:14Z

Review comments:
diff_hunk: @@ -0,0 +1,133 @@
+From 6364f1eef332977bec795bd5237c8b1aa33bb88b Mon Sep 17 00:00:00 2001
+From: Jorge Ramirez-Ortiz <jorge.ramirez@oss.qualcomm.com>
+Date: Wed, 5 Mar 2025 11:11:22 +0100
+Subject: [PATCH] dts: rb3gen2: enable Renesas xHCI
+
+Make sure relevant power is on and resets are let go of, to make the USB
+Ethernet adapter show up.
+
+Ported  to 6.14 from an original patch by Bjorn Andersson.
+
+Link: https://github.com/quic-bjorande/linux/commit/ef0d4badf252c10883a369e8fb217799868158b0
+Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
+Upstream-Status: Pending
+---
+ arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 76 ++++++++++++++++++++
+ 1 file changed, 76 insertions(+)
+
+diff --git a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
+index 13dbb24a3179..73959e5095bf 100644
+--- a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
++++ b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
+@@ -243,6 +243,31 @@ vdd_ntn_1p8: regulator-vdd-ntn-1p8 {
+ 		regulator-enable-ramp-delay = <10000>;
+ 	};
+ 
++        vreg-pcie0-1p05v {
++		compatible = "regulator-fixed";
++		regulator-name = "PCIE0_1.05V";
++		gpio = <&pm7250b_gpios 4 GPIO_ACTIVE_HIGH>;
++		regulator-min-microvolt = <1050000>;
++		regulator-max-microvolt = <1050000>;
++		enable-active-high;
++		pinctrl-0 = <&upd_pwr_en2_state>;
++		pinctrl-names = "default";
++
++		regulator-always-on;
path: recipes-kernel/linux/linux-yocto-dev/qcm6490-board-dts/0001-PENDING-enable-xHCI.patch
user: login: lumag
body: What is being powered by these regulators? Are they powering up the hub? the PCIe bridge? The host?
created_at: 2025-03-13T14:36:31Z
author_association: CONTRIBUTOR
diff_hunk: @@ -0,0 +1,133 @@
+From 6364f1eef332977bec795bd5237c8b1aa33bb88b Mon Sep 17 00:00:00 2001
+From: Jorge Ramirez-Ortiz <jorge.ramirez@oss.qualcomm.com>
+Date: Wed, 5 Mar 2025 11:11:22 +0100
+Subject: [PATCH] dts: rb3gen2: enable Renesas xHCI
+
+Make sure relevant power is on and resets are let go of, to make the USB
+Ethernet adapter show up.
+
+Ported  to 6.14 from an original patch by Bjorn Andersson.
+
+Link: https://github.com/quic-bjorande/linux/commit/ef0d4badf252c10883a369e8fb217799868158b0
+Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
+Upstream-Status: Pending
+---
+ arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 76 ++++++++++++++++++++
+ 1 file changed, 76 insertions(+)
+
+diff --git a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
+index 13dbb24a3179..73959e5095bf 100644
+--- a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
++++ b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
+@@ -243,6 +243,31 @@ vdd_ntn_1p8: regulator-vdd-ntn-1p8 {
+ 		regulator-enable-ramp-delay = <10000>;
+ 	};
+ 
++        vreg-pcie0-1p05v {
++		compatible = "regulator-fixed";
++		regulator-name = "PCIE0_1.05V";
++		gpio = <&pm7250b_gpios 4 GPIO_ACTIVE_HIGH>;
++		regulator-min-microvolt = <1050000>;
++		regulator-max-microvolt = <1050000>;
++		enable-active-high;
++		pinctrl-0 = <&upd_pwr_en2_state>;
++		pinctrl-names = "default";
++
++		regulator-always-on;
++	};
++
++	vreg-pcie0_3p3v-dual {
++		compatible = "regulator-fixed";
++		regulator-name = "PCIE0_3.3V_Dual";
++		gpio = <&pm7250b_gpios 1 GPIO_ACTIVE_HIGH>;
++		regulator-min-microvolt = <3300000>;
++		regulator-max-microvolt = <3300000>;
++		enable-active-high;
++		pinctrl-0 = <&upd_pwr_en1_state>;
++		pinctrl-names = "default";
++
++		regulator-always-on;
++	};
+ };
+ 
+ &apps_rsc {
+@@ -780,6 +805,9 @@ pcie@0,0 {
+ 
+ 		i2c-parent = <&i2c0 0x77>;
+ 
++		pinctrl-0 = <&tc956x_rst_state>, <&upd_hub_rst_state>, <&usb_hub_rst_state>;
++		pinctrl-names = "default";
path: recipes-kernel/linux/linux-yocto-dev/qcm6490-board-dts/0001-PENDING-enable-xHCI.patch
user: login: lumag
body: Is it really enough to hold those pins in the corresponding state? Or should we be toggling those pins in order to reset the hub? Why isn't the hub being properly described as a PCIe device which uses those pins? 
created_at: 2025-03-13T14:38:30Z
author_association: CONTRIBUTOR
diff_hunk: @@ -8,3 +8,4 @@ CONFIG_CLK_QCM2290_GPUCC=m
 # RB3
 CONFIG_PCI_PWRCTL=y
 CONFIG_PCI_PWRCTRL_TC956X=m
+CONFIG_USB_XHCI_PCI_RENESAS=m
path: recipes-kernel/linux/linux-yocto-dev/configs/qcom.cfg
user: login: lumag
body: At least the commit message should be pointing to the Renesas website.
created_at: 2025-03-13T14:39:20Z
author_association: CONTRIBUTOR
diff_hunk: @@ -0,0 +1,133 @@
+From 6364f1eef332977bec795bd5237c8b1aa33bb88b Mon Sep 17 00:00:00 2001
+From: Jorge Ramirez-Ortiz <jorge.ramirez@oss.qualcomm.com>
+Date: Wed, 5 Mar 2025 11:11:22 +0100
+Subject: [PATCH] dts: rb3gen2: enable Renesas xHCI
+
+Make sure relevant power is on and resets are let go of, to make the USB
+Ethernet adapter show up.
+
+Ported  to 6.14 from an original patch by Bjorn Andersson.
+
+Link: https://github.com/quic-bjorande/linux/commit/ef0d4badf252c10883a369e8fb217799868158b0
+Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
+Upstream-Status: Pending
path: recipes-kernel/linux/linux-yocto-dev/qcm6490-board-dts/0001-PENDING-enable-xHCI.patch
user: login: ricardosalveti
body: Why is it still pending?
created_at: 2025-03-13T15:44:20Z
author_association: CONTRIBUTOR
diff_hunk: @@ -0,0 +1,133 @@
+From 6364f1eef332977bec795bd5237c8b1aa33bb88b Mon Sep 17 00:00:00 2001
+From: Jorge Ramirez-Ortiz <jorge.ramirez@oss.qualcomm.com>
+Date: Wed, 5 Mar 2025 11:11:22 +0100
+Subject: [PATCH] dts: rb3gen2: enable Renesas xHCI
+
+Make sure relevant power is on and resets are let go of, to make the USB
+Ethernet adapter show up.
+
+Ported  to 6.14 from an original patch by Bjorn Andersson.
+
+Link: https://github.com/quic-bjorande/linux/commit/ef0d4badf252c10883a369e8fb217799868158b0
+Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
+Upstream-Status: Pending
path: recipes-kernel/linux/linux-yocto-dev/qcm6490-board-dts/0001-PENDING-enable-xHCI.patch
user: login: ldts
body: I dont know - something pending on Bjorn's list. I pinged him a couple of times this week already before posting - he might be out and I thought it would make sense have the functionality ready after EW25 (hence this PR).
we can drop
as I said, I am not sure how do we balance functionality vs upstream status yet.

created_at: 2025-03-13T15:48:22Z
author_association: CONTRIBUTOR
diff_hunk: @@ -8,3 +8,4 @@ CONFIG_CLK_QCM2290_GPUCC=m
 # RB3
 CONFIG_PCI_PWRCTL=y
 CONFIG_PCI_PWRCTRL_TC956X=m
+CONFIG_USB_XHCI_PCI_RENESAS=m
path: recipes-kernel/linux/linux-yocto-dev/configs/qcom.cfg
user: login: ldts
body: bear with me while I align to your granularity please

so the kconfig says
```
config USB_XHCI_PCI_RENESAS                                                                                             
        tristate "Support for additional Renesas xHCI controller with firmware"                                         
        depends on USB_XHCI_PCI                                                                                         
        help                                                                                                            
          Say 'Y' to enable the support for the Renesas xHCI controller with                                            
          firmware. Make sure you have the firmware for the device and                                                  
          installed on your system for this device to work.                                                             
          If unsure, say 'N'.   
```
if at that level, there are no links to Reneasas, why do we need here? what is the information we want to convey?
created_at: 2025-03-13T15:53:26Z
author_association: CONTRIBUTOR
diff_hunk: @@ -0,0 +1,133 @@
+From 6364f1eef332977bec795bd5237c8b1aa33bb88b Mon Sep 17 00:00:00 2001
+From: Jorge Ramirez-Ortiz <jorge.ramirez@oss.qualcomm.com>
+Date: Wed, 5 Mar 2025 11:11:22 +0100
+Subject: [PATCH] dts: rb3gen2: enable Renesas xHCI
+
+Make sure relevant power is on and resets are let go of, to make the USB
+Ethernet adapter show up.
+
+Ported  to 6.14 from an original patch by Bjorn Andersson.
+
+Link: https://github.com/quic-bjorande/linux/commit/ef0d4badf252c10883a369e8fb217799868158b0
+Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
+Upstream-Status: Pending
+---
+ arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 76 ++++++++++++++++++++
+ 1 file changed, 76 insertions(+)
+
+diff --git a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
+index 13dbb24a3179..73959e5095bf 100644
+--- a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
++++ b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
+@@ -243,6 +243,31 @@ vdd_ntn_1p8: regulator-vdd-ntn-1p8 {
+ 		regulator-enable-ramp-delay = <10000>;
+ 	};
+ 
++        vreg-pcie0-1p05v {
++		compatible = "regulator-fixed";
++		regulator-name = "PCIE0_1.05V";
++		gpio = <&pm7250b_gpios 4 GPIO_ACTIVE_HIGH>;
++		regulator-min-microvolt = <1050000>;
++		regulator-max-microvolt = <1050000>;
++		enable-active-high;
++		pinctrl-0 = <&upd_pwr_en2_state>;
++		pinctrl-names = "default";
++
++		regulator-always-on;
path: recipes-kernel/linux/linux-yocto-dev/qcm6490-board-dts/0001-PENDING-enable-xHCI.patch
user: login: ldts
body: dunno, didnt look into it. we can wait for Bjorn's to post to have a properly documented commit.
created_at: 2025-03-13T15:54:59Z
author_association: CONTRIBUTOR
diff_hunk: @@ -8,3 +8,4 @@ CONFIG_CLK_QCM2290_GPUCC=m
 # RB3
 CONFIG_PCI_PWRCTL=y
 CONFIG_PCI_PWRCTRL_TC956X=m
+CONFIG_USB_XHCI_PCI_RENESAS=m
path: recipes-kernel/linux/linux-yocto-dev/configs/qcom.cfg
user: login: lumag
body: I was thinking about a link to the firmware: https://www.renesas.com/en/document/swo/pd720201-pd720202-firmware
created_at: 2025-03-13T16:35:34Z
author_association: CONTRIBUTOR
diff_hunk: @@ -8,3 +8,4 @@ CONFIG_CLK_QCM2290_GPUCC=m
 # RB3
 CONFIG_PCI_PWRCTL=y
 CONFIG_PCI_PWRCTRL_TC956X=m
+CONFIG_USB_XHCI_PCI_RENESAS=m
path: recipes-kernel/linux/linux-yocto-dev/configs/qcom.cfg
user: login: ldts
body: ok, i'll update. 
we should then think about the audio topology file as well on the lpass PR.
created_at: 2025-03-13T16:58:47Z
author_association: CONTRIBUTOR
diff_hunk: @@ -8,3 +8,4 @@ CONFIG_CLK_QCM2290_GPUCC=m
 # RB3
 CONFIG_PCI_PWRCTL=y
 CONFIG_PCI_PWRCTRL_TC956X=m
+CONFIG_USB_XHCI_PCI_RENESAS=m
path: recipes-kernel/linux/linux-yocto-dev/configs/qcom.cfg
user: login: ricardosalveti
body: Yeah, just so the user can find where the firmware is located (we have this documented in qclinux and other places as well).
created_at: 2025-03-13T20:56:45Z
author_association: CONTRIBUTOR
diff_hunk: @@ -0,0 +1,133 @@
+From 6364f1eef332977bec795bd5237c8b1aa33bb88b Mon Sep 17 00:00:00 2001
+From: Jorge Ramirez-Ortiz <jorge.ramirez@oss.qualcomm.com>
+Date: Wed, 5 Mar 2025 11:11:22 +0100
+Subject: [PATCH] dts: rb3gen2: enable Renesas xHCI
+
+Make sure relevant power is on and resets are let go of, to make the USB
+Ethernet adapter show up.
+
+Ported  to 6.14 from an original patch by Bjorn Andersson.
+
+Link: https://github.com/quic-bjorande/linux/commit/ef0d4badf252c10883a369e8fb217799868158b0
+Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
+Upstream-Status: Pending
path: recipes-kernel/linux/linux-yocto-dev/qcm6490-board-dts/0001-PENDING-enable-xHCI.patch
user: login: lumag
body: I took a glance, there are several items which need to be implemented as a way to implement this "properly". We need a pwrseq driver for the hub and ideally, we need to implement a driver for multiple regulators being driven by the same GPIO. The estimate is about 1-2 MW. In the meantime, I'd suggest merging this patch, it is not completely optimal, not suitable for upstream, but more or less correct.
created_at: 2025-03-14T07:44:26Z
author_association: CONTRIBUTOR
diff_hunk: @@ -0,0 +1,133 @@
+From 6364f1eef332977bec795bd5237c8b1aa33bb88b Mon Sep 17 00:00:00 2001
+From: Jorge Ramirez-Ortiz <jorge.ramirez@oss.qualcomm.com>
+Date: Wed, 5 Mar 2025 11:11:22 +0100
+Subject: [PATCH] dts: rb3gen2: enable Renesas xHCI
+
+Make sure relevant power is on and resets are let go of, to make the USB
+Ethernet adapter show up.
+
+Ported  to 6.14 from an original patch by Bjorn Andersson.
+
+Link: https://github.com/quic-bjorande/linux/commit/ef0d4badf252c10883a369e8fb217799868158b0
+Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
+Upstream-Status: Pending
path: recipes-kernel/linux/linux-yocto-dev/qcm6490-board-dts/0001-PENDING-enable-xHCI.patch
user: login: ndechesne
body: @lumag does your last comment imply you are going to work on that, shortly? 
created_at: 2025-03-14T17:00:42Z
author_association: CONTRIBUTOR
diff_hunk: @@ -0,0 +1,133 @@
+From 6364f1eef332977bec795bd5237c8b1aa33bb88b Mon Sep 17 00:00:00 2001
+From: Jorge Ramirez-Ortiz <jorge.ramirez@oss.qualcomm.com>
+Date: Wed, 5 Mar 2025 11:11:22 +0100
+Subject: [PATCH] dts: rb3gen2: enable Renesas xHCI
+
+Make sure relevant power is on and resets are let go of, to make the USB
+Ethernet adapter show up.
+
+Ported  to 6.14 from an original patch by Bjorn Andersson.
+
+Link: https://github.com/quic-bjorande/linux/commit/ef0d4badf252c10883a369e8fb217799868158b0
+Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
+Upstream-Status: Pending
path: recipes-kernel/linux/linux-yocto-dev/qcm6490-board-dts/0001-PENDING-enable-xHCI.patch
user: login: lumag
body: Yes, I've added it to my ToDo list if nobody else is going to pick it up.
created_at: 2025-03-14T20:47:19Z
author_association: CONTRIBUTOR

PR comments:
user: login: github-actions[bot]
created_at: 2025-03-13T15:42:17Z
body: ## Test Results
 2 files  ±0   4 suites  ±0   1m 13s ⏱️ ±0s
19 tests ±0  19 ✅ ±0  0 💤 ±0  0 ❌ ±0 
38 runs  ±0  38 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit c748ea1f. ± Comparison against base commit 33a0ecf7.

[test-results]:data:application/gzip;base64,H4sIADBP02cC/12Myw7CIBQFf6Vh7QLKpaX+jLnySIhtMTxWjf8uJa2iuzNzktmIdbOJ5Nr1l47E7FIFKKBzwOT8WnDkhcuT9o9NJ9xiVurPPNyzGPoRFt38I0wIPhwm5HUvcnnsM9iIb69yk6vc1pRfFpcKEDWCNMgskzBQC1oLRBSSgRzgbu3UUwMCFSevN0hNpUAAAQAA

:recycle: This comment has been updated with latest results.
author_association: NONE
user: login: github-actions[bot]
created_at: 2025-03-13T15:42:18Z
body: ## Test jobs for commit 47c2470770c03298c55711f169073098707e2c7b
 * [Job 66053 on qcs6490](https://lava.infra.foundries.io/scheduler/job/66053)
 * [Job 66052 on qcs9100](https://lava.infra.foundries.io/scheduler/job/66052)

author_association: NONE
user: login: github-actions[bot]
created_at: 2025-03-13T21:33:39Z
body: ## Test jobs for commit c748ea1f18460f4dd5aaa5814864bff920e45ac3
 * [Job 66079 on qcs9100](https://lava.infra.foundries.io/scheduler/job/66079)
 * [Job 66078 on qcs6490](https://lava.infra.foundries.io/scheduler/job/66078)

author_association: NONE
user: login: ldts
created_at: 2025-03-24T06:44:18Z
body: @ndechesne is this functionality still required?
author_association: CONTRIBUTOR