Commit 0cd90a54 authored by Laurent Pinchart's avatar Laurent Pinchart

drm: rcar-du: Turn vblank on/off when enabling/disabling CRTC

The DRM core vblank handling mechanism requires drivers to forcefully
turn vblank reporting off when disabling the CRTC, and to restore the
vblank reporting status when enabling the CRTC.

Implement this using the drm_crtc_vblank_on/off helpers. When disabling
vblank we must first wait for page flips to complete, so implement page
flip completion wait as well.

Finally, drm_crtc_vblank_off() must be called at startup to synchronize
the state of the vblank core code with the hardware, which is initially
disabled. This is performed at CRTC creation time, requiring vertical
blanking to be initialized before creating CRTCs.
Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
parent 36693f3c
...@@ -279,7 +279,7 @@ void rcar_du_crtc_cancel_page_flip(struct rcar_du_crtc *rcrtc, ...@@ -279,7 +279,7 @@ void rcar_du_crtc_cancel_page_flip(struct rcar_du_crtc *rcrtc,
if (event && event->base.file_priv == file) { if (event && event->base.file_priv == file) {
rcrtc->event = NULL; rcrtc->event = NULL;
event->base.destroy(&event->base); event->base.destroy(&event->base);
drm_vblank_put(dev, rcrtc->index); drm_crtc_vblank_put(&rcrtc->crtc);
} }
spin_unlock_irqrestore(&dev->event_lock, flags); spin_unlock_irqrestore(&dev->event_lock, flags);
} }
...@@ -303,7 +303,7 @@ static void rcar_du_crtc_finish_page_flip(struct rcar_du_crtc *rcrtc) ...@@ -303,7 +303,7 @@ static void rcar_du_crtc_finish_page_flip(struct rcar_du_crtc *rcrtc)
wake_up(&rcrtc->flip_wait); wake_up(&rcrtc->flip_wait);
spin_unlock_irqrestore(&dev->event_lock, flags); spin_unlock_irqrestore(&dev->event_lock, flags);
drm_vblank_put(dev, rcrtc->index); drm_crtc_vblank_put(&rcrtc->crtc);
} }
static bool rcar_du_crtc_page_flip_pending(struct rcar_du_crtc *rcrtc) static bool rcar_du_crtc_page_flip_pending(struct rcar_du_crtc *rcrtc)
...@@ -383,6 +383,9 @@ static void rcar_du_crtc_start(struct rcar_du_crtc *rcrtc) ...@@ -383,6 +383,9 @@ static void rcar_du_crtc_start(struct rcar_du_crtc *rcrtc)
rcar_du_group_start_stop(rcrtc->group, true); rcar_du_group_start_stop(rcrtc->group, true);
/* Turn vertical blanking interrupt reporting back on. */
drm_crtc_vblank_on(crtc);
rcrtc->started = true; rcrtc->started = true;
} }
...@@ -393,10 +396,12 @@ static void rcar_du_crtc_stop(struct rcar_du_crtc *rcrtc) ...@@ -393,10 +396,12 @@ static void rcar_du_crtc_stop(struct rcar_du_crtc *rcrtc)
if (!rcrtc->started) if (!rcrtc->started)
return; return;
/* Wait for page flip completion before stopping the CRTC as userspace /* Disable vertical blanking interrupt reporting. We first need to wait
* excepts page flips to eventually complete. * for page flip completion before stopping the CRTC as userspace
* expects page flips to eventually complete.
*/ */
rcar_du_crtc_wait_page_flip(rcrtc); rcar_du_crtc_wait_page_flip(rcrtc);
drm_crtc_vblank_off(crtc);
mutex_lock(&rcrtc->group->planes.lock); mutex_lock(&rcrtc->group->planes.lock);
rcrtc->plane->enabled = false; rcrtc->plane->enabled = false;
...@@ -596,7 +601,7 @@ static int rcar_du_crtc_page_flip(struct drm_crtc *crtc, ...@@ -596,7 +601,7 @@ static int rcar_du_crtc_page_flip(struct drm_crtc *crtc,
if (event) { if (event) {
event->pipe = rcrtc->index; event->pipe = rcrtc->index;
drm_vblank_get(dev, rcrtc->index); drm_crtc_vblank_get(crtc);
spin_lock_irqsave(&dev->event_lock, flags); spin_lock_irqsave(&dev->event_lock, flags);
rcrtc->event = event; rcrtc->event = event;
spin_unlock_irqrestore(&dev->event_lock, flags); spin_unlock_irqrestore(&dev->event_lock, flags);
...@@ -693,6 +698,9 @@ int rcar_du_crtc_create(struct rcar_du_group *rgrp, unsigned int index) ...@@ -693,6 +698,9 @@ int rcar_du_crtc_create(struct rcar_du_group *rgrp, unsigned int index)
drm_crtc_helper_add(crtc, &crtc_helper_funcs); drm_crtc_helper_add(crtc, &crtc_helper_funcs);
/* Start with vertical blanking interrupt reporting disabled. */
drm_crtc_vblank_off(crtc);
/* Register the interrupt handler. */ /* Register the interrupt handler. */
if (rcar_du_has(rcdu, RCAR_DU_FEATURE_CRTC_IRQ_CLOCK)) { if (rcar_du_has(rcdu, RCAR_DU_FEATURE_CRTC_IRQ_CLOCK)) {
irq = platform_get_irq(pdev, index); irq = platform_get_irq(pdev, index);
......
...@@ -175,17 +175,19 @@ static int rcar_du_load(struct drm_device *dev, unsigned long flags) ...@@ -175,17 +175,19 @@ static int rcar_du_load(struct drm_device *dev, unsigned long flags)
if (IS_ERR(rcdu->mmio)) if (IS_ERR(rcdu->mmio))
return PTR_ERR(rcdu->mmio); return PTR_ERR(rcdu->mmio);
/* DRM/KMS objects */ /* Initialize vertical blanking interrupts handling. Start with vblank
ret = rcar_du_modeset_init(rcdu); * disabled for all CRTCs.
*/
ret = drm_vblank_init(dev, (1 << rcdu->info->num_crtcs) - 1);
if (ret < 0) { if (ret < 0) {
dev_err(&pdev->dev, "failed to initialize DRM/KMS\n"); dev_err(&pdev->dev, "failed to initialize vblank\n");
goto done; goto done;
} }
/* vblank handling */ /* DRM/KMS objects */
ret = drm_vblank_init(dev, (1 << rcdu->num_crtcs) - 1); ret = rcar_du_modeset_init(rcdu);
if (ret < 0) { if (ret < 0) {
dev_err(&pdev->dev, "failed to initialize vblank\n"); dev_err(&pdev->dev, "failed to initialize DRM/KMS\n");
goto done; goto done;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment