Re: [PATCH V2 1/2] PCI: Add ACPI StorageD3Enable _DSD support

2020-06-25 Thread Bjorn Helgaas
On Thu, Jun 25, 2020 at 01:30:53PM +0200, Rafael J. Wysocki wrote: > On Wed, Jun 24, 2020 at 11:15 PM Bjorn Helgaas wrote: > > On Fri, Jun 12, 2020 at 01:48:19PM -0700, David E. Box wrote: > > > StorageD3Enable is a boolean property that indicates that the > > > platform wants to use D3 for PCIe

Re: [PATCH V2 1/2] PCI: Add ACPI StorageD3Enable _DSD support

2020-06-25 Thread David E. Box
On Thu, 2020-06-25 at 13:30 +0200, Rafael J. Wysocki wrote: > On Wed, Jun 24, 2020 at 11:15 PM Bjorn Helgaas > wrote: > > On Fri, Jun 12, 2020 at 01:48:19PM -0700, David E. Box wrote: > > > StorageD3Enable is a boolean property that indicates that the > > > platform > > > wants to use D3 for PCIe

Re: [PATCH V2 1/2] PCI: Add ACPI StorageD3Enable _DSD support

2020-06-25 Thread Mika Westerberg
On Thu, Jun 25, 2020 at 01:30:53PM +0200, Rafael J. Wysocki wrote: > It is not necessary to call acpi_pci_find_companion() from > pci_acpi_storage_d3() as long as that function is required to be > called by the target device's driver probe or later. > > Ths acpi_pci_bridge_d3() case is different,

Re: [PATCH V2 1/2] PCI: Add ACPI StorageD3Enable _DSD support

2020-06-25 Thread Rafael J. Wysocki
On Wed, Jun 24, 2020 at 11:15 PM Bjorn Helgaas wrote: > > On Fri, Jun 12, 2020 at 01:48:19PM -0700, David E. Box wrote: > > StorageD3Enable is a boolean property that indicates that the platform > > wants to use D3 for PCIe storage drives during suspend-to-idle. It is a > > BIOS work around that

Re: [PATCH V2 1/2] PCI: Add ACPI StorageD3Enable _DSD support

2020-06-24 Thread David E. Box
On Wed, 2020-06-24 at 16:37 -0500, Bjorn Helgaas wrote: > On Fri, Jun 12, 2020 at 01:48:19PM -0700, David E. Box wrote: > > StorageD3Enable is a boolean property that indicates that the > > platform > > wants to use D3 for PCIe storage drives during suspend-to-idle. > > Is this something that

Re: [PATCH V2 1/2] PCI: Add ACPI StorageD3Enable _DSD support

2020-06-24 Thread Bjorn Helgaas
On Fri, Jun 12, 2020 at 01:48:19PM -0700, David E. Box wrote: > StorageD3Enable is a boolean property that indicates that the platform > wants to use D3 for PCIe storage drives during suspend-to-idle. Is this something that should apply to plug-in drives, or does this only apply to soldered-in

Re: [PATCH V2 1/2] PCI: Add ACPI StorageD3Enable _DSD support

2020-06-24 Thread Bjorn Helgaas
On Fri, Jun 12, 2020 at 01:48:19PM -0700, David E. Box wrote: > StorageD3Enable is a boolean property that indicates that the platform > wants to use D3 for PCIe storage drives during suspend-to-idle. It is a > BIOS work around that is currently in use on shipping systems like some > Intel Comet

[PATCH V2 1/2] PCI: Add ACPI StorageD3Enable _DSD support

2020-06-12 Thread David E. Box
StorageD3Enable is a boolean property that indicates that the platform wants to use D3 for PCIe storage drives during suspend-to-idle. It is a BIOS work around that is currently in use on shipping systems like some Intel Comet Lake platforms. It is meant to change default driver policy for suspend