diff options
author | Ben Skeggs <bskeggs@redhat.com> | 2011-07-04 12:52:27 +1000 |
---|---|---|
committer | Ben Skeggs <bskeggs@redhat.com> | 2011-11-10 08:57:55 +1000 |
commit | bde4889aaa2d59c1febfeadd7f690be0ed143ed5 (patch) | |
tree | 2a7a3d175bb1995040eea6b6e5b14ff736d970de /drivers/gpu | |
parent | 1ea6b8f48918282bdca0b32a34095504ee65bab5 (diff) | |
download | op-kernel-dev-bde4889aaa2d59c1febfeadd7f690be0ed143ed5.zip op-kernel-dev-bde4889aaa2d59c1febfeadd7f690be0ed143ed5.tar.gz |
drm: make sure drm_vblank_init() has been called before touching vbl_lock
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Diffstat (limited to 'drivers/gpu')
-rw-r--r-- | drivers/gpu/drm/drm_irq.c | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c index cb3794a..4d3760c 100644 --- a/drivers/gpu/drm/drm_irq.c +++ b/drivers/gpu/drm/drm_irq.c @@ -407,13 +407,16 @@ int drm_irq_uninstall(struct drm_device *dev) /* * Wake up any waiters so they don't hang. */ - spin_lock_irqsave(&dev->vbl_lock, irqflags); - for (i = 0; i < dev->num_crtcs; i++) { - DRM_WAKEUP(&dev->vbl_queue[i]); - dev->vblank_enabled[i] = 0; - dev->last_vblank[i] = dev->driver->get_vblank_counter(dev, i); + if (dev->num_crtcs) { + spin_lock_irqsave(&dev->vbl_lock, irqflags); + for (i = 0; i < dev->num_crtcs; i++) { + DRM_WAKEUP(&dev->vbl_queue[i]); + dev->vblank_enabled[i] = 0; + dev->last_vblank[i] = + dev->driver->get_vblank_counter(dev, i); + } + spin_unlock_irqrestore(&dev->vbl_lock, irqflags); } - spin_unlock_irqrestore(&dev->vbl_lock, irqflags); if (!irq_enabled) return -EINVAL; |