On Sunday, February 05, 2017 12:34:43 AM Lukas Wunner wrote: > On Sat, Feb 04, 2017 at 01:44:34PM -0800, Yinghai Lu wrote: > > On Sat, Feb 4, 2017 at 10:56 AM, Lukas Wunner <lukas@xxxxxxxxx> wrote: > > > On Sat, Feb 04, 2017 at 09:12:54AM +0100, Lukas Wunner wrote: > > > Section 6.7.3.4 of the PCIe Base spec seems to support the theory above, > > > so here's a tentative patch. > > > > > > > > > -- >8 -- > > > Subject: [PATCH] PCI: pciehp: Don't enable PME on runtime suspend > > > > it works: > > Thanks a lot for the report and for testing the patch! > > > @Rafael: > Bjorn usually wants you to ack PM-related patches. Looking at the > patch again I'm now thinking it might be cleaner to move the check > for is_hotplug_bridge into pci_dev_run_wake(). Thoughts? Yes, pci_dev_run_wake() would be a better place. Thanks, Rafael