diff options
author | Mika Westerberg <mika.westerberg@linux.intel.com> | 2016-06-02 04:17:11 -0400 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2016-06-13 15:51:34 -0400 |
commit | 43f7f88b9362e1f26603b45932069bbd6e15a1e1 (patch) | |
tree | f07853734d5bb79a24df6d79fc3d54c7e270a996 | |
parent | 9661e783d830699a65f7d42e4b4a76e6923089eb (diff) |
PCI: Don't clear d3cold_allowed for PCIe ports
The PCI core skips bridges and ports when the system is suspended. The PCI
core checks return value of pci_has_subordinate() in pci_pm_suspend_noirq()
to skip all devices where it is non-zero (which means PCI bridges and PCIe
ports).
Since PCIe ports are never suspended in the first place, there is no need
to set d3cold_allowed for them.
Tested-by: Lukas Wunner <lukas@wunner.de>
Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r-- | drivers/pci/pcie/portdrv_pci.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/pci/pcie/portdrv_pci.c b/drivers/pci/pcie/portdrv_pci.c index be35da2e105e..6c6bb03392ea 100644 --- a/drivers/pci/pcie/portdrv_pci.c +++ b/drivers/pci/pcie/portdrv_pci.c | |||
@@ -134,11 +134,6 @@ static int pcie_portdrv_probe(struct pci_dev *dev, | |||
134 | return status; | 134 | return status; |
135 | 135 | ||
136 | pci_save_state(dev); | 136 | pci_save_state(dev); |
137 | /* | ||
138 | * D3cold may not work properly on some PCIe port, so disable | ||
139 | * it by default. | ||
140 | */ | ||
141 | dev->d3cold_allowed = false; | ||
142 | return 0; | 137 | return 0; |
143 | } | 138 | } |
144 | 139 | ||