Revert "drm/exynos/decon5433: wait for two vblanks before framebuffer removal"
authorAndrzej Hajda <a.hajda@samsung.com>
Tue, 9 Aug 2016 09:07:47 +0000 (11:07 +0200)
committerSeung-Woo Kim <sw0312.kim@samsung.com>
Wed, 14 Dec 2016 04:52:23 +0000 (13:52 +0900)
Since previous patch fixes strange DECON behavior we do not
need to wait for two vblanks before freeing buffers.

This reverts commit 661aaf94623c1de5d96503aaefdfae4b9105cb1a.

Change-Id: I3cbd68f9af5748f74c5a878de34b6b531aa26075
Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
drivers/gpu/drm/exynos/exynos5433_drm_decon.c

index e3cc259..e3a2794 100644 (file)
@@ -94,7 +94,7 @@ static void decon_wait_for_vblank(struct exynos_drm_crtc *crtc)
        if (ctx->suspended)
                return;
 
-       atomic_set(&ctx->wait_vsync_event, 2);
+       atomic_set(&ctx->wait_vsync_event, 1);
 
        /*
         * wait for FIMD to signal VSYNC interrupt or return after
@@ -667,7 +667,7 @@ static void decon_clear_channel(struct decon_context *ctx)
 
        decon_update(ctx);
 
-       atomic_set(&ctx->wait_vsync_event, 2);
+       atomic_set(&ctx->wait_vsync_event, 1);
 
        /*
         * wait for FIMD to signal VSYNC interrupt or return after
@@ -762,7 +762,7 @@ static irqreturn_t decon_irq_handler(int irq, void *dev_id)
 
 out:
        if (atomic_read(&ctx->wait_vsync_event)) {
-               atomic_dec_if_positive(&ctx->wait_vsync_event);
+               atomic_set(&ctx->wait_vsync_event, 0);
                wake_up(&ctx->wait_vsync_queue);
        }