diff options
| author | Mauro Carvalho Chehab | 2021-04-23 17:19:11 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab | 2021-05-19 09:51:40 +0200 |
| commit | 7917f27941c3c1289aad2be27bf329844125d13e (patch) | |
| tree | c8db0427f2b3456634d42da18ea614e712875be3 | |
| parent | 401b0e5dcab62196c56aa7c2536b0cacfc506e0e (diff) | |
media: i2c: dw9714: use pm_runtime_resume_and_get()
Commit dd8088d5a896 ("PM: runtime: Add pm_runtime_resume_and_get to deal with usage counter")
added pm_runtime_resume_and_get() in order to automatically handle
dev->power.usage_count decrement on errors.
Use the new API, in order to cleanup the error check logic.
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
| -rw-r--r-- | drivers/media/i2c/dw9714.c | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/drivers/media/i2c/dw9714.c b/drivers/media/i2c/dw9714.c index 3f0b082f863f..c8b4292512dc 100644 --- a/drivers/media/i2c/dw9714.c +++ b/drivers/media/i2c/dw9714.c @@ -85,15 +85,7 @@ static const struct v4l2_ctrl_ops dw9714_vcm_ctrl_ops = { static int dw9714_open(struct v4l2_subdev *sd, struct v4l2_subdev_fh *fh) { - int rval; - - rval = pm_runtime_get_sync(sd->dev); - if (rval < 0) { - pm_runtime_put_noidle(sd->dev); - return rval; - } - - return 0; + return pm_runtime_resume_and_get(sd->dev); } static int dw9714_close(struct v4l2_subdev *sd, struct v4l2_subdev_fh *fh) |