diff options
author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2018-07-05 06:47:16 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2018-07-05 06:47:16 -0400 |
commit | 88b96088e94ec66f6c9015eee28cb9d053be69c0 (patch) | |
tree | 5f655566320676b20452266a416660866a98442b | |
parent | 895b66129ad8c562865b64306032bdb378f4484f (diff) | |
parent | 26112ddc254c98681b224aa9ededefc01b6e02d2 (diff) |
Merge branch 'pm-pci'
Merge a PCI power management regression fix.
* pm-pci:
PCI / ACPI / PM: Resume bridges w/o drivers on suspend-to-RAM
-rw-r--r-- | drivers/pci/pci-acpi.c | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/drivers/pci/pci-acpi.c b/drivers/pci/pci-acpi.c index 65113b6eed14..89ee6a2b6eb8 100644 --- a/drivers/pci/pci-acpi.c +++ b/drivers/pci/pci-acpi.c | |||
@@ -629,6 +629,18 @@ static bool acpi_pci_need_resume(struct pci_dev *dev) | |||
629 | { | 629 | { |
630 | struct acpi_device *adev = ACPI_COMPANION(&dev->dev); | 630 | struct acpi_device *adev = ACPI_COMPANION(&dev->dev); |
631 | 631 | ||
632 | /* | ||
633 | * In some cases (eg. Samsung 305V4A) leaving a bridge in suspend over | ||
634 | * system-wide suspend/resume confuses the platform firmware, so avoid | ||
635 | * doing that, unless the bridge has a driver that should take care of | ||
636 | * the PM handling. According to Section 16.1.6 of ACPI 6.2, endpoint | ||
637 | * devices are expected to be in D3 before invoking the S3 entry path | ||
638 | * from the firmware, so they should not be affected by this issue. | ||
639 | */ | ||
640 | if (pci_is_bridge(dev) && !dev->driver && | ||
641 | acpi_target_system_state() != ACPI_STATE_S0) | ||
642 | return true; | ||
643 | |||
632 | if (!adev || !acpi_device_power_manageable(adev)) | 644 | if (!adev || !acpi_device_power_manageable(adev)) |
633 | return false; | 645 | return false; |
634 | 646 | ||