Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Heiko Stuebner <heiko@sntech.de>
To: Chukun Pan <amadeus@jmu.edu.cn>
Cc: jonas@kwiboo.se, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org,
	Chukun Pan <amadeus@jmu.edu.cn>
Subject: Re: [PATCH v2 2/5] arm64: dts: rockchip: Enable USB 2.0 ports on Radxa E20C
Date: Thu, 28 May 2026 07:57:45 +0200	[thread overview]
Message-ID: <3879225.zToM8qfIzz@phil> (raw)
In-Reply-To: <20260525065005.1056845-1-amadeus@jmu.edu.cn>

Hi,

Am Montag, 25. Mai 2026, 08:50:05 Mitteleuropäische Sommerzeit schrieb Chukun Pan:
> > +&usb_host0_xhci {
> > +	extcon = <&usb2phy>;
> > +	maximum-speed = "high-speed";
> > +	phys = <&usb2phy_otg>;
> > +	phy-names = "usb2-phy";
> 
> I received this warning for this otg port:
> [    0.163307] dwc3 fe500000.usb: Configuration mismatch. dr_mode forced to host

Interesting, because if I read the schematics correctly, this is the
controller labeled "usb2.0 otg0" .
It even has a vbus-detect line.

If I understand the schematics and product page correctly, this
actually is an outward facing type-c-port, making the device a
peripheral?

usb_host0_xhci -> port1 of FE1.1s_QFN (device1 connected to the hub)
uart0 -> ch340 -> port2 of FE1.1s_QFN (device2 connected to the hub)

FE1.1s_QFN host dm+dp going to the type-c port of the board.

So if anything, that controller should probably have dr_mode=peripheral
I think?


> 
> > +	snps,dis_u2_susphy_quirk;
> 
> This property is missing in the sige1 patch. Since this property
> is related to phy vbus detection, can we move it to rk3528.dtsi?

yeah, that sounds sensible ... looking at the other socs using that
quirk, it really seems to be a property of the controller.


Heiko




  reply	other threads:[~2026-05-28  5:58 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-05-05 17:12 [PATCH v2 0/5] rockchip: add USB nodes for RK3528 Heiko Stuebner
2026-05-05 17:12 ` [PATCH v2 1/5] arm64: dts: rockchip: Add " Heiko Stuebner
2026-05-05 17:12 ` [PATCH v2 2/5] arm64: dts: rockchip: Enable USB 2.0 ports on Radxa E20C Heiko Stuebner
2026-05-25  6:50   ` Chukun Pan
2026-05-28  5:57     ` Heiko Stuebner [this message]
2026-05-05 17:12 ` [PATCH v2 3/5] arm64: dts: rockchip: Enable USB ports on Radxa ROCK 2A/2F Heiko Stuebner
2026-05-05 17:12 ` [PATCH v2 4/5] arm64: dts: rockchip: Enable USB 2.0 ports on ArmSoM Sige1 Heiko Stuebner
2026-05-25  7:00   ` Chukun Pan
2026-05-05 17:12 ` [PATCH v2 5/5] arm64: dts: rockchip: Enable USB 2.0 ports on NanoPi Zero2 Heiko Stuebner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3879225.zToM8qfIzz@phil \
    --to=heiko@sntech.de \
    --cc=amadeus@jmu.edu.cn \
    --cc=jonas@kwiboo.se \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rockchip@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox