diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2016-05-18 15:47:38 -0400 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2016-05-20 17:03:31 -0400 |
commit | fcee59065e58498682c60069cc6fb90694bab27e (patch) | |
tree | bea115621d1291a80d6bb57c2af3a36e366b84f5 /drivers/gpu/drm/imx | |
parent | fc7fedc20b7c819b9ef003afa16a64dc9e5ac30e (diff) |
drm: Nuke ->vblank_disable_allowed
This was added in
commit 0a3e67a4caac273a3bfc4ced3da364830b1ab241
Author: Jesse Barnes <jbarnes@virtuousgeek.org>
Date: Tue Sep 30 12:14:26 2008 -0700
drm: Rework vblank-wait handling to allow interrupt reduction.
to stay backwards-compatible with old UMS code that didn't even tell
the kernel when it did a modeset, so that the kernel could
save/restore vblank counters. At worst this means vblanks will be
somewhat funky on a setup that very likely no one still runs.
So let's just nuke it.
Plan B would be to set it unconditionally in drm_vblank_init for kms
drivers, instead of in each driver separately. So if this patch breaks
anything please only restore the hunks in drmP.h and drm_irq.c, plus
add a check for DRIVER_MODESET in drm_vblank_init.
Stumbled over this in a discussion on irc with Chris.
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: Liviu Dudau <liviu.dudau@arm.com>
Cc: Russell King <rmk+kernel@arm.linux.org.uk>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Eric Anholt <eric@anholt.net>
Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Cc: Inki Dae <inki.dae@samsung.com>
Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
Cc: Mark Yao <mark.yao@rock-chips.com>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Philipp Zabel <p.zabel@pengutronix.de>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Acked-by: Liviu Dudau <Liviu.Dudau@arm.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Tested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu/drm/imx')
-rw-r--r-- | drivers/gpu/drm/imx/imx-drm-core.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/drivers/gpu/drm/imx/imx-drm-core.c b/drivers/gpu/drm/imx/imx-drm-core.c index 2453fb1c68a7..1080019e7b17 100644 --- a/drivers/gpu/drm/imx/imx-drm-core.c +++ b/drivers/gpu/drm/imx/imx-drm-core.c | |||
@@ -252,13 +252,6 @@ static int imx_drm_driver_load(struct drm_device *drm, unsigned long flags) | |||
252 | if (ret) | 252 | if (ret) |
253 | goto err_kms; | 253 | goto err_kms; |
254 | 254 | ||
255 | /* | ||
256 | * with vblank_disable_allowed = true, vblank interrupt will be | ||
257 | * disabled by drm timer once a current process gives up ownership | ||
258 | * of vblank event. (after drm_vblank_put function is called) | ||
259 | */ | ||
260 | drm->vblank_disable_allowed = true; | ||
261 | |||
262 | platform_set_drvdata(drm->platformdev, drm); | 255 | platform_set_drvdata(drm->platformdev, drm); |
263 | 256 | ||
264 | /* Now try and bind all our sub-components */ | 257 | /* Now try and bind all our sub-components */ |