[linux-yocto] [PATCH 29/34] i2c: i801: Remove pci_enable_device() call from i801_resume()

wei.tee.ng at intel.com wei.tee.ng at intel.com
Mon Apr 27 20:20:54 PDT 2015


From: Jarkko Nikula <jarkko.nikula at linux.intel.com>

Since pci_disable_device() is not called from i801_suspend() and power
state is set already it means that subsequent pci_enable_device() calls do
practically nothing but monotonically increase struct pci_dev enable_cnt.

Signed-off-by: Jarkko Nikula <jarkko.nikula at linux.intel.com>
Reviewed-by: Jean Delvare <jdelvare at suse.de>
Signed-off-by: Wolfram Sang <wsa at the-dreams.de>
(cherry picked from commit f85da3f5de6f354bc26eb84c013b41a8b8558c5e)

Signed-off-by: Ng Wei Tee <wei.tee.ng at intel.com>
---
 drivers/i2c/busses/i2c-i801.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c
index b1d725d..5fb3546 100644
--- a/drivers/i2c/busses/i2c-i801.c
+++ b/drivers/i2c/busses/i2c-i801.c
@@ -1324,7 +1324,7 @@ static int i801_resume(struct pci_dev *dev)
 {
 	pci_set_power_state(dev, PCI_D0);
 	pci_restore_state(dev);
-	return pci_enable_device(dev);
+	return 0;
 }
 #else
 #define i801_suspend NULL
-- 
1.7.9.5



More information about the linux-yocto mailing list