drm: sti: sti_hqvdp: undo preparation of a clock source.
authorArvind Yadav <arvind.yadav.cs@gmail.com>
Wed, 21 Jun 2017 10:15:55 +0000 (15:45 +0530)
committerBenjamin Gaignard <benjamin.gaignard@linaro.org>
Wed, 21 Jun 2017 13:45:19 +0000 (15:45 +0200)
Undo preparation of a clock source, if sti_hqvdp_start_xp70 and
sti_hqvdp_atomic_check are not successful.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>
Link: http://patchwork.freedesktop.org/patch/msgid/7afad3012fb6e40f43a1eb5a64dc6364c38bd052.1498039961.git.arvind.yadav.cs@gmail.com
drivers/gpu/drm/sti/sti_hqvdp.c

index 1234f87bce2021b464e8226f44ada16656ab2c7e..53a46dda8bd5ed3722d512b820b2760fe8c9faa7 100644 (file)
@@ -958,6 +958,7 @@ static void sti_hqvdp_start_xp70(struct sti_hqvdp *hqvdp)
        }
        if (i == POLL_MAX_ATTEMPT) {
                DRM_ERROR("Could not reset\n");
+               clk_disable_unprepare(hqvdp->clk);
                goto out;
        }
 
@@ -994,6 +995,7 @@ static void sti_hqvdp_start_xp70(struct sti_hqvdp *hqvdp)
        }
        if (i == POLL_MAX_ATTEMPT) {
                DRM_ERROR("Could not boot\n");
+               clk_disable_unprepare(hqvdp->clk);
                goto out;
        }
 
@@ -1081,6 +1083,7 @@ static int sti_hqvdp_atomic_check(struct drm_plane *drm_plane,
                                            &hqvdp->vtg_nb,
                                            crtc)) {
                        DRM_ERROR("Cannot register VTG notifier\n");
+                       clk_disable_unprepare(hqvdp->clk_pix_main);
                        return -EINVAL;
                }
                hqvdp->vtg_registered = true;