Re: [PATCH v2 0/2] arm64: dts: qcom: x1e80100: fix USB OTG regressions

From: Johan Hovold
Date: Mon Dec 09 2024 - 10:30:41 EST


On Mon, Dec 09, 2024 at 03:23:05PM +0200, Dmitry Baryshkov wrote:
> On Mon, Dec 09, 2024 at 12:19:03PM +0100, Johan Hovold wrote:
> > A recent change enabling OTG mode on the Lenovo ThinkPad T14s USB-C
> > ports can break SuperSpeed device hotplugging.
> >
> > Abel noticed that the corresponding commit for the CRD also triggers a
> > hard reset during resume from suspend.
> >
> > With retimer (and orientation detection) support not even merged yet,
> > let's revert at least until we have stable host mode in mainline.
> >
> > Note that Stephan and Dmitry have already identified other problems with
> > the offending commits here:
> >
> > https://lore.kernel.org/all/ZxZO6Prrm2ITUZMQ@xxxxxxxxxx/
> > https://lore.kernel.org/all/hw2pdof4ajadjsjrb44f2q4cz4yh5qcqz5d3l7gjt2koycqs3k@xx5xvd26uyef

> > Changes in v2
> > - revert also the corresponding patch for the CRD which breaks suspend
>
> As you are reverting two commits, please revert the third one too, it
> breaks pmic-glink.

Can you be more specific?

I was gonna say that pmic_glink works since hotplug and orientation
detection still works, but I tested now with DP altmode and that is
indeed broken unless I revert the third commit (f042bc234c2e ("arm64:
dts: qcom: x1e80100: enable OTG on USB-C controllers")).

Was that what you had in mind? Can you explain why that breaks?

I'll respin with a v3, but please answer the above first.

> > Johan Hovold (2):
> > Revert "arm64: dts: qcom: x1e78100-t14s: enable otg on usb-c ports"
> > Revert "arm64: dts: qcom: x1e80100-crd: enable otg on usb ports"

Johan