| Message ID | 20240322-typec-fix-sm8250-v1-2-1ac22b333ea9@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | arm64: dts: qcom: sm8250: fix description of the Type-C signals | expand |
On 22/03/2024 11:58, Dmitry Baryshkov wrote: > The SuperSpeed signals originate from the DWC3 host controller and then > are routed through the Combo QMP PHY, where they are multiplexed with > the DisplayPort signals. Add corresponding OF graph link. > > Reported-by: Luca Weiss <luca.weiss@fairphone.com> > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > --- > arch/arm64/boot/dts/qcom/sm8250.dtsi | 24 ++++++++++++++++++++++-- > 1 file changed, 22 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi > index d57039a4c3aa..e551e733ab94 100644 > --- a/arch/arm64/boot/dts/qcom/sm8250.dtsi > +++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi > @@ -3917,6 +3917,10 @@ port@0 { > > port@1 { > reg = <1>; > + > + usb_1_qmpphy_usb_ss_in: endpoint { > + remote-endpoint = <&usb_1_dwc3_ss_out>; > + }; > }; > > port@2 { > @@ -4195,8 +4199,24 @@ usb_1_dwc3: usb@a600000 { > phys = <&usb_1_hsphy>, <&usb_1_qmpphy QMP_USB43DP_USB3_PHY>; > phy-names = "usb2-phy", "usb3-phy"; > > - port { > - usb_1_dwc3_hs_out: endpoint {}; > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + > + usb_1_dwc3_hs_out: endpoint { > + }; > + }; > + > + port@1 { > + reg = <1>; > + > + usb_1_dwc3_ss_out: endpoint { > + remote-endpoint = <&usb_1_qmpphy_usb_ss_in>; > + }; > + }; > }; > }; > }; > I think these should go into platform definitions, there's nothing at the SoC level that imposes the port constraint. --- bod
On 22.03.2024 14:19, Dmitry Baryshkov wrote: > On Fri, 22 Mar 2024 at 14:11, Bryan O'Donoghue > <bryan.odonoghue@linaro.org> wrote: >> >> On 22/03/2024 11:58, Dmitry Baryshkov wrote: >>> The SuperSpeed signals originate from the DWC3 host controller and then >>> are routed through the Combo QMP PHY, where they are multiplexed with >>> the DisplayPort signals. Add corresponding OF graph link. >>> >>> Reported-by: Luca Weiss <luca.weiss@fairphone.com> >>> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> >>> --- >>> arch/arm64/boot/dts/qcom/sm8250.dtsi | 24 ++++++++++++++++++++++-- >>> 1 file changed, 22 insertions(+), 2 deletions(-) >>> >>> diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi >>> index d57039a4c3aa..e551e733ab94 100644 >>> --- a/arch/arm64/boot/dts/qcom/sm8250.dtsi >>> +++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi >>> @@ -3917,6 +3917,10 @@ port@0 { >>> >>> port@1 { >>> reg = <1>; >>> + >>> + usb_1_qmpphy_usb_ss_in: endpoint { >>> + remote-endpoint = <&usb_1_dwc3_ss_out>; >>> + }; >>> }; >>> >>> port@2 { >>> @@ -4195,8 +4199,24 @@ usb_1_dwc3: usb@a600000 { >>> phys = <&usb_1_hsphy>, <&usb_1_qmpphy QMP_USB43DP_USB3_PHY>; >>> phy-names = "usb2-phy", "usb3-phy"; >>> >>> - port { >>> - usb_1_dwc3_hs_out: endpoint {}; >>> + ports { >>> + #address-cells = <1>; >>> + #size-cells = <0>; >>> + >>> + port@0 { >>> + reg = <0>; >>> + >>> + usb_1_dwc3_hs_out: endpoint { >>> + }; >>> + }; >>> + >>> + port@1 { >>> + reg = <1>; >>> + >>> + usb_1_dwc3_ss_out: endpoint { >>> + remote-endpoint = <&usb_1_qmpphy_usb_ss_in>; >>> + }; >>> + }; >>> }; >>> }; >>> }; >>> >> >> I think these should go into platform definitions, there's nothing at >> the SoC level that imposes the port constraint. > > The link between DWC3 and QMP PHY is fixed in the SoC, if I remember correctly. Yes, I believe so too Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org> Konrad >
diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi index d57039a4c3aa..e551e733ab94 100644 --- a/arch/arm64/boot/dts/qcom/sm8250.dtsi +++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi @@ -3917,6 +3917,10 @@ port@0 { port@1 { reg = <1>; + + usb_1_qmpphy_usb_ss_in: endpoint { + remote-endpoint = <&usb_1_dwc3_ss_out>; + }; }; port@2 { @@ -4195,8 +4199,24 @@ usb_1_dwc3: usb@a600000 { phys = <&usb_1_hsphy>, <&usb_1_qmpphy QMP_USB43DP_USB3_PHY>; phy-names = "usb2-phy", "usb3-phy"; - port { - usb_1_dwc3_hs_out: endpoint {}; + ports { + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + reg = <0>; + + usb_1_dwc3_hs_out: endpoint { + }; + }; + + port@1 { + reg = <1>; + + usb_1_dwc3_ss_out: endpoint { + remote-endpoint = <&usb_1_qmpphy_usb_ss_in>; + }; + }; }; }; };
The SuperSpeed signals originate from the DWC3 host controller and then are routed through the Combo QMP PHY, where they are multiplexed with the DisplayPort signals. Add corresponding OF graph link. Reported-by: Luca Weiss <luca.weiss@fairphone.com> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- arch/arm64/boot/dts/qcom/sm8250.dtsi | 24 ++++++++++++++++++++++-- 1 file changed, 22 insertions(+), 2 deletions(-)