]> git.baikalelectronics.ru Git - kernel.git/commit
i2c: designware: Remove needless pm_runtime_put_noidle() call
authorJarkko Nikula <jarkko.nikula@linux.intel.com>
Tue, 15 Aug 2017 14:34:44 +0000 (17:34 +0300)
committerWolfram Sang <wsa@the-dreams.de>
Thu, 17 Aug 2017 15:55:20 +0000 (17:55 +0200)
commit36978ff00b2bb06b6cb9f42ebc2c34ac4942ffd3
tree1c3e47dca92efd9efbe2aefca1f7f57949dfe643
parent2a9a8ade3355c9510368a33d3bd3d1babccfbc13
i2c: designware: Remove needless pm_runtime_put_noidle() call

I guess pm_runtime_put_noidle() call in i2c_dw_probe_slave() was copied
by accident from similar master mode adapter registration code. It is
unbalanced due missing pm_runtime_get_noresume() but harmless since it
doesn't decrease dev->power.usage_count below zero.

In theory we can hit similar needless runtime suspend/resume cycle
during slave mode adapter registration that was happening when
registering the master mode adapter. See commit fc99fb08d2e3 ("i2c:
designware: Prevent runtime suspend during adapter registration").

However, since we are slave, we can consider it as a wrong configuration
if we have other slaves attached under this adapter and can omit the
pm_runtime_get_noresume()/pm_runtime_put_noidle() calls for simplicity.

Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
drivers/i2c/busses/i2c-designware-slave.c