On Wed, Jun 17, 2015 at 12:16:45PM +0200, Wolfram Sang wrote: > On Wed, Jun 17, 2015 at 12:08:38PM +0300, Mika Westerberg wrote: > > The driver calls pm_runtime_put() right before pm_runtime_disable() in its > > ->remove() hook to make sure clock is gated etc. However, it turns out that > > pm_runtime_put() only calls ->idle() hook without actually suspending > > anything. The following pm_runtime_disable() will prevent the driver from > > suspending thus leaving it "active". > > > > It is better to suspend the device synchronously to make sure it is > > actually suspended before disabling runtime PM from it. > > > > While there, undo call to pm_runtime_use_autosuspend(). > > > > Signed-off-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx> > > For current? For next? stable? > Right, sorry about that. I think it is fine to get this for next. -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
