From: Mauro Carvalho Chehab Date: Fri, 23 Apr 2021 15:19:12 +0000 (+0200) Subject: media: i2c: dw9807-vcm: use pm_runtime_resume_and_get() X-Git-Tag: accepted/tizen/unified/20230118.172025~6722^2~313 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d5e75e8b4a24715fddd4adf3a4c0bf90e36546c5;p=platform%2Fkernel%2Flinux-rpi.git media: i2c: dw9807-vcm: 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 --- diff --git a/drivers/media/i2c/dw9807-vcm.c b/drivers/media/i2c/dw9807-vcm.c index 438a44b..95e06f1 100644 --- a/drivers/media/i2c/dw9807-vcm.c +++ b/drivers/media/i2c/dw9807-vcm.c @@ -130,15 +130,7 @@ static const struct v4l2_ctrl_ops dw9807_vcm_ctrl_ops = { static int dw9807_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 dw9807_close(struct v4l2_subdev *sd, struct v4l2_subdev_fh *fh)