]> Pileus Git - ~andy/linux/commitdiff
[SCSI] isci: add CONFIG_PM_SLEEP to suspend/resume functions
authorJingoo Han <jg1.han@samsung.com>
Tue, 26 Mar 2013 07:01:38 +0000 (16:01 +0900)
committerJames Bottomley <JBottomley@Parallels.com>
Thu, 11 Apr 2013 23:01:53 +0000 (16:01 -0700)
Add CONFIG_PM_SLEEP to suspend/resume functions to fix the following
build warning when CONFIG_PM_SLEEP is not selected. This is because
sleep PM callbacks defined by SIMPLE_DEV_PM_OPS are only used when
the CONFIG_PM_SLEEP is enabled.

drivers/scsi/isci/init.c:725:12: warning: 'isci_suspend' defined but not used [-Wunused-function]
drivers/scsi/isci/init.c:743:12: warning: 'isci_resume' defined but not used [-Wunused-function]

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lukasz Dorau <lukasz.dorau@intel.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
drivers/scsi/isci/init.c

index 2839baa82a5a458d0035d59e8a5666ac07690779..d25d0d859f05aff55cf70cf8b7c476984f93147d 100644 (file)
@@ -721,7 +721,7 @@ static void isci_pci_remove(struct pci_dev *pdev)
        }
 }
 
-#ifdef CONFIG_PM
+#ifdef CONFIG_PM_SLEEP
 static int isci_suspend(struct device *dev)
 {
        struct pci_dev *pdev = to_pci_dev(dev);
@@ -770,18 +770,16 @@ static int isci_resume(struct device *dev)
 
        return 0;
 }
+#endif
 
 static SIMPLE_DEV_PM_OPS(isci_pm_ops, isci_suspend, isci_resume);
-#endif
 
 static struct pci_driver isci_pci_driver = {
        .name           = DRV_NAME,
        .id_table       = isci_id_table,
        .probe          = isci_pci_probe,
        .remove         = isci_pci_remove,
-#ifdef CONFIG_PM
        .driver.pm      = &isci_pm_ops,
-#endif
 };
 
 static __init int isci_init(void)