drm/sti: run hqvdp init sequence only once

Do not rely on plane->status to define whether this is the first update
but rather check for hqvdp->xp70_initialized bit status.
This avoids multiple and unwanted calls to sti_vtg_register_client()
which breaks the kernel scheduler.

Signed-off-by: Fabien Dessenne <fabien.dessenne@st.com>
Acked-by: Vincent Abriou <vincent.abriou@st.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/drivers/gpu/drm/sti/sti_hqvdp.c b/drivers/gpu/drm/sti/sti_hqvdp.c
index 04546ebb75d7..c6b25f4fad6a 100644
--- a/drivers/gpu/drm/sti/sti_hqvdp.c
+++ b/drivers/gpu/drm/sti/sti_hqvdp.c
@@ -771,6 +771,7 @@ static void sti_hqvdp_disable(struct sti_hqvdp *hqvdp)
                 DRM_ERROR("XP70 could not revert to idle\n");
 
         hqvdp->plane.status = STI_PLANE_DISABLED;
+        hqvdp->xp70_initialized = false;
 }
 
 /**
@@ -1013,7 +1014,6 @@ static int sti_hqvdp_atomic_check(struct drm_plane *drm_plane,
         struct sti_hqvdp *hqvdp = to_sti_hqvdp(plane);
         struct drm_crtc *crtc = state->crtc;
         struct drm_framebuffer *fb = state->fb;
-        bool first_prepare = plane->status == STI_PLANE_DISABLED ? true : false;
         struct drm_crtc_state *crtc_state;
         struct drm_display_mode *mode;
         int dst_x, dst_y, dst_w, dst_h;
@@ -1064,7 +1064,7 @@ static int sti_hqvdp_atomic_check(struct drm_plane *drm_plane,
                 return -EINVAL;
         }
 
-        if (first_prepare) {
+        if (!hqvdp->xp70_initialized) {
                 /* Start HQVDP XP70 coprocessor */
                 sti_hqvdp_start_xp70(hqvdp);