[PATCH v3 1/2] nvme: Look for StorageD3Enable on companion ACPI device instead
Christoph Hellwig
hch at lst.de
Sun May 30 23:46:55 PDT 2021
On Fri, May 28, 2021 at 11:02:34AM -0500, Mario Limonciello wrote:
> The documentation around the StorageD3Enable property hints that it
> should be made on the PCI device. This is where newer AMD systems set
> the property and it's required for S0i3 support.
>
> So rather than look for nodes of the root port only present on Intel
> systems, switch to the companion ACPI device for all systems.
> David Box from Intel indicated this should work on Intel as well.
I think we need to wait for the confirmation from David. This looks good,
but I'd like to see testing. I also wonder how many of the simple suspend
quirks we can drop with this.
Shyjumon and Jon, can you retests the platforms quirked in
1fae37accfc5 ("nvme/pci: Add sleep quirk for Samsung and Toshiba drives")
with this fix?
More information about the Linux-nvme
mailing list