Re: [PATCH 0/2] Modify USB controller nodes for USB on X1E80100

From: Val Packett

Date: Sat Nov 22 2025 - 17:18:36 EST


On 10/27/25 8:46 AM, Konrad Dybcio wrote:
On 10/19/25 1:35 PM, Krishna Kurapati PSSNV wrote:

On 10/14/2025 2:30 PM, Val Packett wrote:
Hi,

On 10/13/25 11:21 PM, Krishna Kurapati wrote:
This series aims to flatten usb dt nodes and use latest bindings.
While at it, fix a compile time warning on the HS only controller node.

Tests done:
1. On CRD, verified host mode operation of the 3 exposed Type-C ports
and enumeration of the Goodix fingerprint sensor on the multiport
usb controller.

2. Modified the dr_mode to otg for the first 3 controllers and
verified role switching and device mode operation on the 3 exposed
Type-C ports.

HS only controller was not tested on any platform.
have you tested suspend-resume?

The flattened dwc driver seems to break it for me on Latitude 7455, upon trying to resume the screen never comes back up and the system just reboots from zero in a couple seconds instead.

I've looked at the code and I couldn't find the cause so far, but it is fine with the legacy driver and not fine with this one :(

Hi Val,

 Thanks for reporting this. I did test runtime suspend resume on all 3 typec ports as mentioned. But I didn't check system suspend case.

 I will try to reproduce the issue, fix it and then resend the patches.
I can repro on the CRD..

It's the USB3_0 host that causes the issue

Removing the clk_bulk_disable_unprepare() call in dwc3_qcom_suspend()
helps..

Konrad

friendly ping.. has there been any progress on solving this properly and switching to the flattened nodes?

~val