| Message ID | 33674270.0PFIgc82r6@aspire.rjw.lan |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | PM / core: Direct handling of DPM_FLAG_SMART_SUSPEND and DPM_FLAG_LEAVE_SUSPENDED | expand |
On Wed, Jan 03, 2018 at 01:38:27AM +0100, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > Make the PCIe port driver set DPM_FLAG_SMART_SUSPEND and > DPM_FLAG_LEAVE_SUSPENDED for the devices handled by it to benefit > from the opportunistic optimizations in the PCI layer enabled by > these flags. > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> Acked-by: Bjorn Helgaas <bhelgaas@google.com> Please merge this along with the rest of the series. > --- > drivers/pci/pcie/portdrv_pci.c | 3 +++ > 1 file changed, 3 insertions(+) > > Index: linux-pm/drivers/pci/pcie/portdrv_pci.c > =================================================================== > --- linux-pm.orig/drivers/pci/pcie/portdrv_pci.c > +++ linux-pm/drivers/pci/pcie/portdrv_pci.c > @@ -150,6 +150,9 @@ static int pcie_portdrv_probe(struct pci > > pci_save_state(dev); > > + dev_pm_set_driver_flags(&dev->dev, DPM_FLAG_SMART_SUSPEND | > + DPM_FLAG_LEAVE_SUSPENDED); > + > if (pci_bridge_d3_possible(dev)) { > /* > * Keep the port resumed 100ms to make sure things like >
On Thu, Jan 4, 2018 at 11:14 PM, Bjorn Helgaas <helgaas@kernel.org> wrote: > On Wed, Jan 03, 2018 at 01:38:27AM +0100, Rafael J. Wysocki wrote: >> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com> >> >> Make the PCIe port driver set DPM_FLAG_SMART_SUSPEND and >> DPM_FLAG_LEAVE_SUSPENDED for the devices handled by it to benefit >> from the opportunistic optimizations in the PCI layer enabled by >> these flags. >> >> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > Acked-by: Bjorn Helgaas <bhelgaas@google.com> > > Please merge this along with the rest of the series. I will, thank you!
Index: linux-pm/drivers/pci/pcie/portdrv_pci.c =================================================================== --- linux-pm.orig/drivers/pci/pcie/portdrv_pci.c +++ linux-pm/drivers/pci/pcie/portdrv_pci.c @@ -150,6 +150,9 @@ static int pcie_portdrv_probe(struct pci pci_save_state(dev); + dev_pm_set_driver_flags(&dev->dev, DPM_FLAG_SMART_SUSPEND | + DPM_FLAG_LEAVE_SUSPENDED); + if (pci_bridge_d3_possible(dev)) { /* * Keep the port resumed 100ms to make sure things like