Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 9e266ece authored by Aaron Lu's avatar Aaron Lu Committed by Rusty Russell
Browse files

virtio_pci: pm: Use CONFIG_PM_SLEEP instead of CONFIG_PM 



The virtio_pci_freeze/restore are defined under CONFIG_PM but is used
by SET_SYSTEM_SLEEP_PM_OPS macro, which is defined under
CONFIG_PM_SLEEP. So if CONFIG_PM_SLEEP is not cofigured but
CONFIG_PM_RUNTIME is, the following warning message appeared:

drivers/virtio/virtio_pci.c:770:12: warning: ‘virtio_pci_freeze’ defined but not used [-Wunused-function]
 static int virtio_pci_freeze(struct device *dev)
            ^
drivers/virtio/virtio_pci.c:790:12: warning: ‘virtio_pci_restore’ defined but not used [-Wunused-function]
 static int virtio_pci_restore(struct device *dev)
            ^
Fix it by changing CONFIG_PM to CONFIG_PM_SLEEP.

Signed-off-by: default avatarAaron Lu <aaron.lu@intel.com>
Reviewed-by: default avatarAmit Shah <amit.shah@redhat.com>
Signed-off-by: default avatarRusty Russell <rusty@rustcorp.com.au>
parent aa96a3c6

drivers/virtio/virtio_pci.c

+2 −2
Original line number Diff line number Diff line
@@ -766,7 +766,7 @@ static void virtio_pci_remove(struct pci_dev *pci_dev) 
	kfree(vp_dev);

}



#ifdef CONFIG_PM

#ifdef CONFIG_PM_SLEEP

static int virtio_pci_freeze(struct device *dev)

{

	struct pci_dev *pci_dev = to_pci_dev(dev);
@@ -824,7 +824,7 @@ static struct pci_driver virtio_pci_driver = { 
	.id_table	= virtio_pci_id_table,

	.probe		= virtio_pci_probe,

	.remove		= virtio_pci_remove,

#ifdef CONFIG_PM

#ifdef CONFIG_PM_SLEEP

	.driver.pm	= &virtio_pci_pm_ops,

#endif

};