On Mon, Jul 16, 2018 at 11:07:07AM +0100, Alexandru Gheorghe wrote:

Currently, if userspace calls drm_wait_vblank before the crtc is activated the crtc vblank_enable hook is called, which in case of malidp driver triggers some warninngs. This happens because on device init we don't inform the drm core about the vblank state by calling drm_crtc_vblank_on/off/reset which together with drm_vblank_get have some magic that prevents calling drm_vblank_enable when crtc is off.

Signed-off-by: Alexandru Gheorghe alexandru-cosmin.gheorghe@arm.com

Acked-by: Liviu Dudau liviu.dudau@arm.com

---

drivers/gpu/drm/arm/malidp_drv.c | 1 + 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/arm/malidp_drv.c b/drivers/gpu/drm/arm/malidp_drv.c index 4169a72..641d743 100644 --- a/drivers/gpu/drm/arm/malidp_drv.c +++ b/drivers/gpu/drm/arm/malidp_drv.c @@ -755,6 +755,7 @@ static int malidp_bind(struct device *dev) drm->irq_enabled = true;

ret = drm_vblank_init(drm, drm->mode_config.num_crtc);

• drm_crtc_vblank_reset(&malidp->crtc); if (ret < 0) { DRM_ERROR("failed to initialise vblank\n"); goto vblank_fail;

-- 2.7.4