On Wed, May 14, 2025 at 02:34:05PM -0500, Mario Limonciello wrote: > Adjust the pci_pm_poweroff_noirq() to follow the same flow as > pci_pm_suspend_noirq() in that PCIe ports that are power manageable should
Nit: s/should// > without downstream devices in D0 should be put into their appropriate > sleep state. This leads to a lot of code duplication between pci_pm_suspend_noirq() and pci_pm_poweroff_noirq(). Can the common portion of the code be moved to a helper invoked by both functions so that it's easier to follow the logic and understand common and differing parts of the suspend flow? Thanks, Lukas