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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <d0c18094-7234-450d-bd9c-e9f545de31e2@packett.cool>
Date: Tue, 14 Oct 2025 06:00:54 -0300
From: Val Packett <val@...kett.cool>
To: Krishna Kurapati <krishna.kurapati@....qualcomm.com>,
 Bjorn Andersson <andersson@...nel.org>,
 Konrad Dybcio <konradybcio@...nel.org>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Abel Vesa <abel.vesa@...aro.org>,
 Rajendra Nayak <quic_rjendra@...cinc.com>,
 Sibi Sankar <sibi.sankar@....qualcomm.com>
Cc: linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/2] Modify USB controller nodes for USB on X1E80100

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 :(


Thanks,
~val