arm64: dts: qcom: qcs8300-ride: Enable second USB controller on QCS8300 Ride

[PATCH] arm64: dts: qcom: qcs8300-ride: Enable second USB controller on QCS8300 Ride

Posted by Manish Nagar 9 months, 2 weeks ago

Enable secondary USB controller on QCS8300 Ride platform. Since it is a
Type-A port, the dr_mode has been set to "host". The VBUS to connected
peripherals is provided by TPS2559QWDRCTQ1 regulator connected to the
port. The regulator has an enable pin controlled by PMM8650. Model it as
fixed regulator and keep it Always-On at boot, since the regulator is
GPIO controlled regulator.

Co-developed-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
Signed-off-by: Manish Nagar <quic_mnagar@quicinc.com>
---
 arch/arm64/boot/dts/qcom/qcs8300-ride.dts | 35 +++++++++++++++++++++++
 1 file changed, 35 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/qcs8300-ride.dts b/arch/arm64/boot/dts/qcom/qcs8300-ride.dts
index 916d4e6da922..7947e48f6a95 100644
--- a/arch/arm64/boot/dts/qcom/qcs8300-ride.dts
+++ b/arch/arm64/boot/dts/qcom/qcs8300-ride.dts
@@ -22,6 +22,16 @@ aliases {
 	chosen {
 		stdout-path = "serial0:115200n8";
 	};
+
+	regulator-usb2-vbus {
+		compatible = "regulator-fixed";
+		regulator-name = "USB2_VBUS";
+		gpio = <&pmm8650au_1_gpios 7 GPIO_ACTIVE_HIGH>;
+		pinctrl-0 = <&usb2_en>;
+		pinctrl-names = "default";
+		enable-active-high;
+		regulator-always-on;
+	};
 };
 
 &apps_rsc {
@@ -286,6 +296,15 @@ queue3 {
 	};
 };
 
+&pmm8650au_1_gpios {
+	usb2_en: usb2-en-state {
+		pins = "gpio7";
+		function = "normal";
+		output-enable;
+		power-source = <0>;
+	};
+};
+
 &qupv3_id_0 {
 	status = "okay";
 };
@@ -355,6 +374,14 @@ &usb_1_hsphy {
 	status = "okay";
 };
 
+&usb_2_hsphy {
+	vdda-pll-supply = <&vreg_l7a>;
+	vdda18-supply = <&vreg_l7c>;
+	vdda33-supply = <&vreg_l9a>;
+
+	status = "okay";
+};
+
 &usb_qmpphy {
 	vdda-phy-supply = <&vreg_l7a>;
 	vdda-pll-supply = <&vreg_l5a>;
@@ -369,3 +396,11 @@ &usb_1 {
 &usb_1_dwc3 {
 	dr_mode = "peripheral";
 };
+
+&usb_2 {
+	status = "okay";
+};
+
+&usb_2_dwc3 {
+	dr_mode = "host";
+};
-- 
2.25.1

Re: [PATCH] arm64: dts: qcom: qcs8300-ride: Enable second USB controller on QCS8300 Ride

Posted by Bjorn Andersson 9 months, 1 week ago

On Mon, 10 Mar 2025 16:17:43 +0530, Manish Nagar wrote:
> Enable secondary USB controller on QCS8300 Ride platform. Since it is a
> Type-A port, the dr_mode has been set to "host". The VBUS to connected
> peripherals is provided by TPS2559QWDRCTQ1 regulator connected to the
> port. The regulator has an enable pin controlled by PMM8650. Model it as
> fixed regulator and keep it Always-On at boot, since the regulator is
> GPIO controlled regulator.
> 
> [...]

Applied, thanks!

[1/1] arm64: dts: qcom: qcs8300-ride: Enable second USB controller on QCS8300 Ride
      commit: 64f8541e7a2c5e91498f297bce89a88ab9a34fd3

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>

Re: [PATCH] arm64: dts: qcom: qcs8300-ride: Enable second USB controller on QCS8300 Ride

Posted by Konrad Dybcio 9 months, 2 weeks ago

On 10.03.2025 11:47 AM, Manish Nagar wrote:
> Enable secondary USB controller on QCS8300 Ride platform. Since it is a
> Type-A port, the dr_mode has been set to "host". The VBUS to connected
> peripherals is provided by TPS2559QWDRCTQ1 regulator connected to the
> port. The regulator has an enable pin controlled by PMM8650. Model it as
> fixed regulator and keep it Always-On at boot, since the regulator is
> GPIO controlled regulator.
> 
> Co-developed-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
> Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
> Signed-off-by: Manish Nagar <quic_mnagar@quicinc.com>
> ---

Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

Konrad