[PATCH v2 6/6] drm/rockchip: Enable vblank without event

Yakir Yang ykk at rock-chips.com
Wed Aug 17 03:27:31 UTC 2016


Sean,

On 08/17/2016 09:11 AM, Sean Paul wrote:
> vblank should be enabled regardless of whether an event
> is expected back. This is especially important for a cursor
> plane.
Yep, I also found that sometimes vblank haven't been enabled when I move 
the mouse lightly, that would cause eDP panel wound't exit from PSR 
active state, and then nothing would be updated on panel. After apply 
this patch, things work rightly now, thanks for fixing.

Reviewed-by: Yakir Yang <ykk at rock-chips.com>
Tested-by: Yakir Yang <ykk at rock-chip.com>
> Signed-off-by: Sean Paul <seanpaul at chromium.org>
> ---
>
> Changes in v2:
> 	- Rebased on https://cgit.freedesktop.org/~seanpaul/dogwood/log/?h=for-next
>
>   drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 15 ++++++++++-----
>   1 file changed, 10 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
> index d1e0e06..1787084 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
> @@ -112,6 +112,7 @@ struct vop {
>   	struct device *dev;
>   	struct drm_device *drm_dev;
>   	bool is_enabled;
> +	bool vblank_active;
>   
>   	/* mutex vsync_ work */
>   	struct mutex vsync_mutex;
> @@ -1107,10 +1108,11 @@ static void vop_crtc_atomic_begin(struct drm_crtc *crtc,
>   	struct vop *vop = to_vop(crtc);
>   
>   	spin_lock_irq(&crtc->dev->event_lock);
> -	if (crtc->state->event) {
> -		WARN_ON(drm_crtc_vblank_get(crtc) != 0);
> -		WARN_ON(vop->event);
> +	vop->vblank_active = true;
> +	WARN_ON(drm_crtc_vblank_get(crtc) != 0);
> +	WARN_ON(vop->event);
>   
> +	if (crtc->state->event) {
>   		vop->event = crtc->state->event;
>   		crtc->state->event = NULL;
>   	}
> @@ -1197,12 +1199,14 @@ static void vop_handle_vblank(struct vop *vop)
>   
>   	spin_lock_irqsave(&drm->event_lock, flags);
>   	if (vop->event) {
> -
>   		drm_crtc_send_vblank_event(crtc, vop->event);
> -		drm_crtc_vblank_put(crtc);
>   		vop->event = NULL;
>   
>   	}
> +	if (vop->vblank_active) {
> +		vop->vblank_active = false;
> +		drm_crtc_vblank_put(crtc);
> +	}
>   	spin_unlock_irqrestore(&drm->event_lock, flags);
>   
>   	if (!completion_done(&vop->wait_update_complete))
> @@ -1472,6 +1476,7 @@ static int vop_initial(struct vop *vop)
>   	clk_disable(vop->aclk);
>   
>   	vop->is_enabled = false;
> +	vop->vblank_active = false;
>   
>   	return 0;
>   




More information about the dri-devel mailing list