Currently, the vblank support is not correctly implemented in MXSFB_DRM
driver. The call to drm_vblank_init is made with mode_config.num_crtc
which at that time is 0. Because of this, vblank is not activated, so
there won't be any vblank event submitted.

Signed-off-by: Robert Chiras <robert.chi...@nxp.com>
---
 drivers/gpu/drm/mxsfb/mxsfb_drv.c | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/mxsfb/mxsfb_drv.c 
b/drivers/gpu/drm/mxsfb/mxsfb_drv.c
index 76230d7..aa35c43 100644
--- a/drivers/gpu/drm/mxsfb/mxsfb_drv.c
+++ b/drivers/gpu/drm/mxsfb/mxsfb_drv.c
@@ -43,6 +43,9 @@
 #include "mxsfb_drv.h"
 #include "mxsfb_regs.h"
 
+/* The eLCDIF max possible CRTCs */
+#define MAX_CRTCS 1
+
 enum mxsfb_devtype {
        MXSFB_V3,
        MXSFB_V4,
@@ -140,6 +143,8 @@ static void mxsfb_pipe_enable(struct 
drm_simple_display_pipe *pipe,
                mxsfb->connector = &mxsfb->panel_connector;
        }
 
+       drm_crtc_vblank_on(&mxsfb->pipe.crtc);
+
        pm_runtime_get_sync(drm->dev);
        drm_panel_prepare(mxsfb->panel);
        mxsfb_crtc_enable(mxsfb);
@@ -249,7 +254,7 @@ static int mxsfb_load(struct drm_device *drm, unsigned long 
flags)
 
        pm_runtime_enable(drm->dev);
 
-       ret = drm_vblank_init(drm, drm->mode_config.num_crtc);
+       ret = drm_vblank_init(drm, MAX_CRTCS);
        if (ret < 0) {
                dev_err(drm->dev, "Failed to initialise vblank\n");
                goto err_vblank;
@@ -272,6 +277,8 @@ static int mxsfb_load(struct drm_device *drm, unsigned long 
flags)
                goto err_vblank;
        }
 
+       drm_crtc_vblank_off(&mxsfb->pipe.crtc);
+
        /*
         * Attach panel only if there is one.
         * If there is no panel attach, it must be a bridge. In this case, we
-- 
2.7.4

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to