i2c: Fix bus-level power management callbacks
authorRafael J. Wysocki <rjw@sisk.pl>
Mon, 10 May 2010 21:09:30 +0000 (23:09 +0200)
committerRafael J. Wysocki <rjw@sisk.pl>
Mon, 10 May 2010 21:09:30 +0000 (23:09 +0200)
commit2f60ba706bd9af84c4eab704243b262e69556f2e
tree9e8b18893983e0dac392efa3642660c1c16ea17f
parented77134bfccf5e75b6cbadab268e559dbe6a4ebb
i2c: Fix bus-level power management callbacks

There are three issues with the i2c bus type's power management
callbacks at the moment.  First, they don't include any hibernate
callbacks, although they should at least include the .restore()
callback (there's no guarantee that the driver will be present in
memory before loading the image kernel and we must restore the
pre-hibernation state of the device).  Second, the "legacy"
callbacks are not going to be invoked by the PM core since the bus
type's pm object is not NULL.  Finally, the system sleep PM
(ie. suspend/resume) callbacks don't check if the device has been
already suspended at run time, in which case they should skip
suspending it.  Also, it looks like the i2c bus type can use the
generic subsystem-level runtime PM callbacks.

For these reasons, rework the system sleep PM callbacks provided by
the i2c bus type to handle hibernation correctly and to invoke the
"legacy" callbacks for drivers that provide them.  In addition to
that make the i2c bus type use the generic subsystem-level runtime
PM callbacks.

Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Acked-by: Jean Delvare <khali@linux-fr.org>
drivers/i2c/i2c-core.c
include/linux/pm_runtime.h