Thread (7 messages) 7 messages, 4 authors, 2021-06-03

Re: [PATCH v3 1/2] nvme: Look for StorageD3Enable on companion ACPI device instead

From: "David E. Box" <david.e.box@linux.intel.com>
Date: 2021-06-02 22:52:10
Also in: linux-nvme

Hi Mario,

The patch works for systems that worked on the current implementation.
Thanks.

Reviewed-by: David E. Box <david.e.box@linux.intel.com>

On Tue, 2021-06-01 at 10:15 -0500, Limonciello, Mario wrote:
quoted
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.
David, do you think you can get this soonish or are you similarly 
blocked like Shyjumon by remote work restrictions?  As this is causing 
see this series land for 5.14.  If you can't get to it in time, I would
prefer that we switch this patch back to how it was conservatively done
in v1 for 5.14 (and redo 2/2 on top of that):
https://lore.kernel.org/linux-nvme/20210527113751.GB17266@lst.de/T/#t (local)

and let you get back to the list when you can to optimize it "later".
quoted
Shyjumon and Jon, can you retests the platforms quirked in
1fae37accfc5 ("nvme/pci: Add sleep quirk for Samsung and Toshiba
drives") with this fix?

Shyjumon>> Yes, I do agree we need to test this also. However the
boards where which I had these issues are not in my remote access now
(as the work frequency on this boards are less and also due to Covid
situation),
                         It might  take some time for me to test. I
will update as soon as I can.
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help