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.westerb...@linux.intel.com>
> 
> 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to