On Mon, Sep 25, 2017 at 11:10:06PM +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> > > The power management handling in dw_i2c_plat_probe() is somewhat > messy and it is rather hard to figure out the code intention for > the case when pm_disabled is set. In that case, the driver doesn't > enable runtime PM at all, but in addition to that it calls > pm_runtime_forbid() as though it wasn't sure if runtime PM might > be enabled for the device later by someone else. > > Although that concern doesn't seem to be actually valid, the > device is clearly still expected to be PM-capable even in the > pm_disabled set case, so a better approach would be to enable > runtime PM for it unconditionally and prevent it from being > runtime-suspended by using pm_runtime_get_noresume(). > > Make the driver do that. > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> Applied to for-next, thanks!
Attachment:
signature.asc
Description: PGP signature
