Commit 5984de0b authored by Bjorn Helgaas's avatar Bjorn Helgaas
Browse files

PCI/PM: Remove unused 'state' parameter to pci_legacy_suspend_late() 

1a1daf09 ("PCI/PM: Remove unused pci_driver.suspend_late() hook")
removed the legacy .suspend_late() hook, which was the only user of the
"state" parameter to pci_legacy_suspend_late(), but it neglected to remove
the parameter.

Remove the unused "state" parameter to pci_legacy_suspend_late().

Link: https://lore.kernel.org/r/20221025193502.669091-1-helgaas@kernel.org


Signed-off-by: default avatarBjorn Helgaas <bhelgaas@google.com>
Acked-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 9abf2313

drivers/pci/pci-driver.c

+4 −4
Original line number Diff line number Diff line
@@ -646,7 +646,7 @@ static int pci_legacy_suspend(struct device *dev, pm_message_t state) 
	return 0;

}



static int pci_legacy_suspend_late(struct device *dev, pm_message_t state)

static int pci_legacy_suspend_late(struct device *dev)

{

	struct pci_dev *pci_dev = to_pci_dev(dev);
@@ -848,7 +848,7 @@ static int pci_pm_suspend_noirq(struct device *dev) 
		return 0;



	if (pci_has_legacy_pm_support(pci_dev))

		return pci_legacy_suspend_late(dev, PMSG_SUSPEND);

		return pci_legacy_suspend_late(dev);



	if (!pm) {

		pci_save_state(pci_dev);
@@ -1060,7 +1060,7 @@ static int pci_pm_freeze_noirq(struct device *dev) 
	const struct dev_pm_ops *pm = dev->driver ? dev->driver->pm : NULL;



	if (pci_has_legacy_pm_support(pci_dev))

		return pci_legacy_suspend_late(dev, PMSG_FREEZE);

		return pci_legacy_suspend_late(dev);



	if (pm && pm->freeze_noirq) {

		int error;
@@ -1179,7 +1179,7 @@ static int pci_pm_poweroff_noirq(struct device *dev) 
		return 0;



	if (pci_has_legacy_pm_support(pci_dev))

		return pci_legacy_suspend_late(dev, PMSG_HIBERNATE);

		return pci_legacy_suspend_late(dev);



	if (!pm) {

		pci_fixup_device(pci_fixup_suspend_late, pci_dev);