On Wed, Mar 14, 2018 at 12:48:49PM +0100, Rafael J. Wysocki wrote: > Have you considered putting these things into the ->suspend_late and > ->resume_early callbacks, respectively? > > That might be slightly better as runtime resume is still enabled when > the ->suspend and ->resume callbacks run. There is no ->suspend_late or ->resume_early callbacks in struct pcie_port_service_driver so I followed what drivers/pci/pcie/pme.c is doing. I guess we could add those callbacks as well if you think they are better suited here.