media: platform: sti: Fix runtime PM imbalance in regs_show

pm_runtime_get_sync() will increase the runtime PM counter
even it returns an error. Thus a pairing decrement is needed
to prevent refcount leak. Fix this by replacing this API with
pm_runtime_resume_and_get(), which will not change the runtime
PM counter on error.

Signed-off-by: Dinghao Liu <dinghao.liu@zju.edu.cn>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>

diff --git a/drivers/media/platform/sti/bdisp/bdisp-debug.c b/drivers/media/platform/sti/bdisp/bdisp-debug.c
index 2b270093009c750499007ff91030ef5c6a7db83e..a27f638df11c6bced68d02eacaec31c31d832793 100644 (file)
--- a/drivers/media/platform/sti/bdisp/bdisp-debug.c
+++ b/drivers/media/platform/sti/bdisp/bdisp-debug.c
@@ -480,7 +480,7 @@ static int regs_show(struct seq_file *s, void *data)
        int ret;
        unsigned int i;
 
-       ret = pm_runtime_get_sync(bdisp->dev);
+       ret = pm_runtime_resume_and_get(bdisp->dev);
        if (ret < 0) {
                seq_puts(s, "Cannot wake up IP\n");
                return 0;