[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ftqb2uxp6yk73djyo3psehhqq5wamimtissgfehhziwbkprl4c@phftum3m3sdy>
Date: Tue, 13 Jan 2026 13:29:08 +0200
From: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
To: Swati Agarwal <swati.agarwal@....qualcomm.com>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Icenowy Zheng <uwu@...nowy.me>,
Matthias Kaehlcke <mka@...omium.org>,
J . Neuschäfer <j.ne@...teo.net>,
Jens Glathe <jens.glathe@...schoolsolutions.biz>,
Mike Looijmans <mike.looijmans@...ic.nl>,
Catalin Popescu <catalin.popescu@...ca-geosystems.com>,
Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konradybcio@...nel.org>,
Pin-yen Lin <treapking@...omium.org>, linux-usb@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-arm-msm@...r.kernel.org
Subject: Re: [PATCH v3 4/4] arm64: dts: qcom: lemans-evk: Enable secondary
USB controller in host mode
On Tue, Jan 13, 2026 at 01:32:41PM +0530, Swati Agarwal wrote:
> On Sat, Dec 20, 2025 at 9:47 PM Dmitry Baryshkov
> <dmitry.baryshkov@....qualcomm.com> wrote:
> >
> > On Sat, Dec 20, 2025 at 12:05:37PM +0530, Swati Agarwal wrote:
> > > Enable secondary USB controller in host mode on lemans EVK Platform.
> > >
> > > For secondary USB Typec port, there is a genesys USB HUB GL3590 having 4
> > > ports sitting in between SOC and HD3SS3220 Type-C port controller and SS
> > > lines run from the SoC through the hub to the Port controller. Mark the
> > > second USB controller as host only capable.
> > >
> > > Add HD3SS3220 Type-C port controller along with Type-c connector for
> > > controlling vbus supply.
> > >
> > > Signed-off-by: Swati Agarwal <swati.agarwal@....qualcomm.com>
> > > ---
> > > arch/arm64/boot/dts/qcom/lemans-evk.dts | 158 ++++++++++++++++++++++++
> > > 1 file changed, 158 insertions(+)
> > >
> > > diff --git a/arch/arm64/boot/dts/qcom/lemans-evk.dts b/arch/arm64/boot/dts/qcom/lemans-evk.dts
> > > index 70d85b6ba772..d72639479d75 100644
> > > --- a/arch/arm64/boot/dts/qcom/lemans-evk.dts
> > > +++ b/arch/arm64/boot/dts/qcom/lemans-evk.dts
> > > @@ -67,6 +67,47 @@ usb0_con_ss_ep: endpoint {
> > > };
> > > };
> > >
> > > + connector-1 {
> > > + compatible = "usb-c-connector";
> > > + label = "USB1-Type-C";
> > > + data-role = "host";
> > > + power-role = "dual";
> > > + try-power-role = "source";
> > > +
> > > + vbus-supply = <&vbus_supply_regulator_1>;
> > > +
> > > + ports {
> > > + #address-cells = <1>;
> > > + #size-cells = <0>;
> > > +
> > > + port@0 {
> > > + reg = <0>;
> > > +
> > > + usb1_con_ss_ep: endpoint {
> > > + remote-endpoint = <&hd3ss3220_1_in_ep>;
> > > + };
> > > + };
> > > +
> > > + port@1 {
> > > + reg = <1>;
> > > +
> > > + usb1_hs_in: endpoint {
> > > + remote-endpoint = <&usb_hub_2_1>;
> > > + };
> > > +
> > > + };
> > > +
> > > + port@2 {
> > > + reg = <2>;
> > > +
> > > + usb1_ss_in: endpoint {
> > > + remote-endpoint = <&usb_hub_3_1>;
> > > + };
> > > +
> > > + };
> > > + };
> > > + };
> > > +
> > > edp0-connector {
> > > compatible = "dp-connector";
> > > label = "EDP0";
> > > @@ -140,6 +181,16 @@ vbus_supply_regulator_0: regulator-vbus-supply-0 {
> > > enable-active-high;
> > > };
> > >
> > > + vbus_supply_regulator_1: regulator-vbus-supply-1 {
> > > + compatible = "regulator-fixed";
> > > + regulator-name = "vbus_supply_1";
> > > + gpio = <&expander1 3 GPIO_ACTIVE_HIGH>;
> > > + regulator-min-microvolt = <5000000>;
> > > + regulator-max-microvolt = <5000000>;
> > > + regulator-boot-on;
> > > + enable-active-high;
> > > + };
> > > +
> > > vmmc_sdc: regulator-vmmc-sdc {
> > > compatible = "regulator-fixed";
> > >
> > > @@ -527,6 +578,33 @@ hd3ss3220_0_out_ep: endpoint {
> > > };
> > > };
> > > };
> > > +
> > > + usb-typec@47 {
> > > + compatible = "ti,hd3ss3220";
> > > + reg = <0x47>;
> > > +
> > > + interrupts-extended = <&pmm8654au_2_gpios 6 IRQ_TYPE_EDGE_FALLING>;
> > > +
> > > + id-gpios = <&tlmm 51 GPIO_ACTIVE_HIGH>;
> > > +
> > > + pinctrl-0 = <&usb1_id>, <&usb1_intr>;
> > > + pinctrl-names = "default";
> > > +
> > > + ports {
> > > + #address-cells = <1>;
> > > + #size-cells = <0>;
> > > +
> > > + port@0 {
> > > + reg = <0>;
> > > +
> > > + hd3ss3220_1_in_ep: endpoint {
> > > + remote-endpoint = <&usb1_con_ss_ep>;
> > > + };
> > > + };
> > > +
> > > + };
> > > + };
> > > +
> > > };
> > >
> > > &i2c18 {
> > > @@ -690,6 +768,14 @@ usb0_intr_state: usb0-intr-state {
> > > bias-pull-up;
> > > power-source = <0>;
> > > };
> > > +
> > > + usb1_intr: usb1-intr-state {
> > > + pins = "gpio6";
> > > + function = "normal";
> > > + input-enable;
> > > + bias-pull-up;
> > > + power-source = <0>;
> > > + };
> > > };
> > >
> > > &qup_i2c19_default {
> > > @@ -849,6 +935,12 @@ usb_id: usb-id-state {
> > > function = "gpio";
> > > bias-pull-up;
> > > };
> > > +
> > > + usb1_id: usb1-id-state {
> > > + pins = "gpio51";
> > > + function = "gpio";
> > > + bias-pull-up;
> > > + };
> > > };
> > >
> > > &uart10 {
> > > @@ -903,6 +995,72 @@ &usb_0_qmpphy {
> > > status = "okay";
> > > };
> > >
> > > +&usb_1 {
> > > + dr_mode = "host";
> > > +
> > > + #address-cells = <1>;
> > > + #size-cells = <0>;
> > > +
> > > + status = "okay";
> > > +
> > > + usb_hub_2_x: hub@1 {
> > > + compatible = "usb5e3,610";
> > > + reg = <1>;
> > > + peer-hub = <&usb_hub_3_x>;
> > > + #address-cells = <1>;
> > > + #size-cells = <0>;
> > > +
> > > + ports {
> > > + #address-cells = <1>;
> > > + #size-cells = <0>;
> > > +
> > > + port@1 {
> > > + reg = <1>;
> > > +
> > > + usb_hub_2_1: endpoint {
> > > + remote-endpoint = <&usb1_hs_in>;
> > > + };
> >
> > Are all other ports disconnected? If so, why do we need a hub?
> Hi Dmitry,
> I didn't understand your query, can you give more context to it?
You have described one port of the hub. How are other ports routed? Are
they connected to outer ports? To some other devices? Unconnected?
--
With best wishes
Dmitry
Powered by blists - more mailing lists