linux-kernel - Re: [PATCH v5 00/12] Fix USB suspend on TI J7200 (cdns3-ti, cdns3, xhci)

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: <7h7cctw8ja.fsf@baylibre.com>
Date: Tue, 06 Aug 2024 16:12:09 -0700
From: Kevin Hilman <khilman@...nel.org>
To: Théo Lebrun <theo.lebrun@...tlin.com>, Roger Quadros
 <rogerq@...nel.org>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, Rob
 Herring <robh@...nel.org>, Krzysztof
 Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Peter
 Chen <peter.chen@...nel.org>, Pawel Laszczak <pawell@...ence.com>, Mathias
 Nyman <mathias.nyman@...el.com>, Nishanth
 Menon <nm@...com>, Vignesh Raghavendra <vigneshr@...com>, Tero Kristo
 <kristo@...nel.org>
Cc: linux-usb@...r.kernel.org, devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
 Grégory Clement <gregory.clement@...tlin.com>, Thomas
 Petazzoni
 <thomas.petazzoni@...tlin.com>, Conor Dooley <conor.dooley@...rochip.com>
Subject: Re: [PATCH v5 00/12] Fix USB suspend on TI J7200 (cdns3-ti, cdns3,
 xhci)

Théo Lebrun <theo.lebrun@...tlin.com> writes:

[...]

> Some TI platforms cannot tell, before going to suspend, if their USB
> controller will reset. Suspend behavior is defined by (at least) two
> features:
>
>  - Power domains. See arch/arm64/boot/dts/ti/k3-j7200-main.dtsi:
>
>    usbss0: cdns-usb@...4000 {
>       compatible = "ti,j7200-usb", "ti,j721e-usb";
>       // ...
>       power-domains = <&k3_pds 288 TI_SCI_PD_EXCLUSIVE>;
>       // ...
>    };
>
>    This `power-domains` property implies that even s2idle will reset
>    the controller.

minor but important clarification: not *will* reset, but *might* reset the controller.

Kevin