media: rkisp1: Disable runtime PM in probe error path
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Tue, 14 Jun 2022 19:10:36 +0000 (20:10 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Sun, 17 Jul 2022 10:24:53 +0000 (11:24 +0100)
If the v4l2_device_register() call fails, runtime PM is left enabled.
Fix it.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Dafna Hirschfeld <dafna@fastmail.com>
Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/platform/rockchip/rkisp1/rkisp1-dev.c

index 97d569968285adeeb40a825c697f7344d3d62b16..248f0172ca628fb5246ab461b540db9af83a6a8a 100644 (file)
@@ -523,7 +523,7 @@ static int rkisp1_probe(struct platform_device *pdev)
 
        ret = v4l2_device_register(rkisp1->dev, &rkisp1->v4l2_dev);
        if (ret)
-               return ret;
+               goto err_pm_runtime_disable;
 
        ret = media_device_register(&rkisp1->media_dev);
        if (ret) {
@@ -543,6 +543,7 @@ err_unreg_media_dev:
        media_device_unregister(&rkisp1->media_dev);
 err_unreg_v4l2_dev:
        v4l2_device_unregister(&rkisp1->v4l2_dev);
+err_pm_runtime_disable:
        pm_runtime_disable(&pdev->dev);
        return ret;
 }