]> git.proxmox.com Git - mirror_ubuntu-focal-kernel.git/commitdiff
i2c: i801: fix unused-function warning
authorAnders Roxell <anders.roxell@linaro.org>
Mon, 14 May 2018 09:33:26 +0000 (11:33 +0200)
committerWolfram Sang <wsa@the-dreams.de>
Thu, 17 May 2018 13:54:12 +0000 (15:54 +0200)
With CONFIG_PM, we get a harmless build warning:
drivers/i2c/busses/i2c-i801.c:1723:12: warning: ‘i801_resume’ defined but not used [-Wunused-function]
 static int i801_resume(struct device *dev)
            ^~~~~~~~~~~
drivers/i2c/busses/i2c-i801.c:1714:12: warning: ‘i801_suspend’ defined but not used [-Wunused-function]
 static int i801_suspend(struct device *dev)
            ^~~~~~~~~~~~

Follow design pattern from other drivers like i2c-brcmstb, i2c-mpc,
i2c-ocores, i2c-pnx, i2c-puv3, i2c-st, i2c-stu300 and i2c-mux-pca954x
and changing the ifdef CONFIG_PM to CONFIG_PM_SLEEP.

Fixes: a9c8088c7988 ("i2c: i801: Don't restore config registers on runtime PM")
Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
Reviewed-by: Jean Delvare <jdelvare@suse.de>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
drivers/i2c/busses/i2c-i801.c

index ed07f9002710eced6a2bce173c91243d5a9515e8..954fb3f3b7fcbd6cb0ccd5c22b76ea9f941089a5 100644 (file)
@@ -1710,7 +1710,7 @@ static void i801_shutdown(struct pci_dev *dev)
        pci_write_config_byte(dev, SMBHSTCFG, priv->original_hstcfg);
 }
 
-#ifdef CONFIG_PM
+#ifdef CONFIG_PM_SLEEP
 static int i801_suspend(struct device *dev)
 {
        struct pci_dev *pci_dev = to_pci_dev(dev);