This is an automatic generated email to let you know that the following patch 
were queued:

Subject: staging: media: atomisp: use pm_runtime_resume_and_get()
Author:  Mauro Carvalho Chehab <[email protected]>
Date:    Fri Apr 23 17:19:21 2021 +0200

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.

Besides that, the de-init order in case of css error was wrong.
This change should also fix that.

Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/staging/media/atomisp/pci/atomisp_fops.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

---

diff --git a/drivers/staging/media/atomisp/pci/atomisp_fops.c 
b/drivers/staging/media/atomisp/pci/atomisp_fops.c
index f1e6b2597853..26d05474a035 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_fops.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_fops.c
@@ -837,7 +837,7 @@ dev_init:
        }
 
        /* runtime power management, turn on ISP */
-       ret = pm_runtime_get_sync(vdev->v4l2_dev->dev);
+       ret = pm_runtime_resume_and_get(vdev->v4l2_dev->dev);
        if (ret < 0) {
                dev_err(isp->dev, "Failed to power on device\n");
                goto error;
@@ -881,9 +881,9 @@ done:
 
 css_error:
        atomisp_css_uninit(isp);
+       pm_runtime_put(vdev->v4l2_dev->dev);
 error:
        hmm_pool_unregister(HMM_POOL_TYPE_DYNAMIC);
-       pm_runtime_put(vdev->v4l2_dev->dev);
        rt_mutex_unlock(&isp->mutex);
        return ret;
 }

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to