i2c: i801: Don't restore config registers on runtime PM

jdelvare · Wolfram Sang · commit a9c8088c7988 · 2018-04-30T10:39:28.000+02:00
Restoring configuration registers is only needed when we hand control
to the firmware. This is never the case with runtime power
management. The device will autosuspend whenever not used, so avoid
useless register writes by defining suspend/resume only, and not
runtime_suspend/runtime_resume.

Signed-off-by: Jean Delvare &lt;jdelvare@suse.de&gt;
Reviewed-by: Jarkko Nikula &lt;jarkko.nikula@linux.intel.com&gt;
Signed-off-by: Wolfram Sang &lt;wsa@the-dreams.de&gt;
diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c
@@ -1731,8 +1731,7 @@ static int i801_resume(struct device *dev)
 }
 #endif
 
-static UNIVERSAL_DEV_PM_OPS(i801_pm_ops, i801_suspend,
-			    i801_resume, NULL);
+static SIMPLE_DEV_PM_OPS(i801_pm_ops, i801_suspend, i801_resume);
 
 static struct pci_driver i801_driver = {
 	.name		= "i801_smbus",

Original file line number	Diff line number	Diff line change
`@@ -1731,8 +1731,7 @@ static int i801_resume(struct device *dev)`
`1731`	`1731`	`}`
`1732`	`1732`	`#endif`
`1733`	`1733`
`1734`		`-static UNIVERSAL_DEV_PM_OPS(i801_pm_ops, i801_suspend,`
`1735`		`- i801_resume, NULL);`
	`1734`	`+static SIMPLE_DEV_PM_OPS(i801_pm_ops, i801_suspend, i801_resume);`
`1736`	`1735`
`1737`	`1736`	`static struct pci_driver i801_driver = {`
`1738`	`1737`	`.name = "i801_smbus",`