From: Gustavo Padovan <gustavo.pado...@collabora.co.uk>

on resume (or enable()) do not call disable_plane() on planes that
are already disabled.

Signed-off-by: Gustavo Padovan <gustavo.padovan at collabora.co.uk>
---
 drivers/gpu/drm/exynos/exynos_drm_fimd.c | 15 +--------------
 1 file changed, 1 insertion(+), 14 deletions(-)

diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c 
b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
index e55a8b0..7dae3a7 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
@@ -786,23 +786,10 @@ static void fimd_window_resume(struct fimd_context *ctx)
                plane = &ctx->planes[i];
                plane->enabled = plane->resume;
                plane->resume = false;
-       }
-}

-static void fimd_apply(struct fimd_context *ctx)
-{
-       struct exynos_drm_plane *plane;
-       int i;
-
-       for (i = 0; i < WINDOWS_NR; i++) {
-               plane = &ctx->planes[i];
                if (plane->enabled)
                        fimd_update_plane(ctx->crtc, i);
-               else
-                       fimd_disable_plane(ctx->crtc, i);
        }
-
-       fimd_commit(ctx->crtc);
 }

 static int fimd_enable(struct exynos_drm_crtc *crtc)
@@ -840,7 +827,7 @@ static int fimd_enable(struct exynos_drm_crtc *crtc)

        fimd_window_resume(ctx);

-       fimd_apply(ctx);
+       fimd_commit(ctx->crtc);

        return 0;

-- 
2.1.0

Reply via email to