Re: [PATCH 03/12] arm64: dts: qcom: qcs6490-radxa-dragon-q6a: Enable USB 3.0 and HDMI ports

[Xilin Wu]

On 4/8/2026 5:03 PM, Konrad Dybcio wrote:
> On 4/7/26 5:19 PM, Xilin Wu wrote:
> > This board doesn't feature a regular Type-C port. The usb_1_qmpphy's
>
> I guess the receptacle on board is power-only?

Yes. The Type-C port is power only.

> > RX1/TX1 pair is statically connected to the USB-A port, while its RX0/TX0
> > pair is connected to the RA620 DP-to-HDMI bridge.
> >
> > Add and enable the nodes for the features to work.
> >
> > Signed-off-by: Xilin Wu <sophon@xxxxxxxxx>
> > ---
> >   .../boot/dts/qcom/qcs6490-radxa-dragon-q6a.dts     | 152 +++++++++++++++++++++
> >   1 file changed, 152 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/qcs6490-radxa-dragon-q6a.dts b/arch/arm64/boot/dts/qcom/qcs6490-radxa-dragon-q6a.dts
> > index c961d3ec625f..8d649b3a1cfa 100644
> > --- a/arch/arm64/boot/dts/qcom/qcs6490-radxa-dragon-q6a.dts
> > +++ b/arch/arm64/boot/dts/qcom/qcs6490-radxa-dragon-q6a.dts
> > @@ -111,6 +111,71 @@ usb2_3_connector: endpoint {
> >   		};
> >   	};
> >   
> > +	usb3_con: connector {
>
> This label is unused

Ack. I will remove the label.

> > +		compatible = "usb-a-connector";
>
> No vbus-supply?

It's supplied by vcc_5v_peri. I'll add it in v2.

> [...]
>
> > +&mdss_dp {
> > +	sound-name-prefix = "Display Port0";
>
> Hmmmmm.. other platforms call it "DisplayPort0" (without a space)..
> But I suppose this name needs to match UCM..
>
> We'd also normally push this property to the SoC DTSI

Actually I don't think the name is used in UCM. I can rename it and push 
the property to SoC DTSI if necessary.

> Konrad


--
Best regards,
Xilin Wu <sophon@xxxxxxxxx>