Re: [PATCH v2] arm64: dts: rockchip: add usb typec host support to rk3588-jaguar

[Quentin Schulz <quentin.schulz@cherry.de>]

Hi Heiko,

On 2/18/25 10:10 PM, Heiko Stuebner wrote:
> From: Heiko Stuebner <heiko.stuebner@cherry.de>
> 
> Jaguar has two type-c ports connected to fusb302 controllers that can
> work both in host and device mode and can also run in display-port
> altmode.
> 
> While these ports can work in dual-role data mode, they do not support
> powering the device itself as power-sink. This causes issues because
> the current infrastructure does not cope well with dual-role data
> without dual-role power.
> 
> So add the necessary nodes for the type-c controllers as well
> as enable the relevant core usb nodes, but limit the mode to host-mode
> for now until we figure out device mode.
> 
> Signed-off-by: Heiko Stuebner <heiko.stuebner@cherry.de>

While the USB functionality does work for both ports, and the 
orientation is properly reported, it seems like there may be some issue 
with how the PHY or controller interacts with that info because I do not 
get USB3 speeds when the device is inserted in reverse orientation, 
while I do when it's in normal orientation.

I assume that's the case for the Rock 5 ITX and Orange Pi 5+ as well and 
probably has nothing to do with the DT?

Should we go still go on with trying to merge this patch knowing that? I 
mean USB2 is still better than no USB at all :)

+Cc Chen-Yu, owner of an Orange Pi 5+, who may be able to confirm the 
issue is widespread.

Cheers,
Quentin