diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-11-24 12:00:36 +0300 |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2015-12-06 00:44:07 +0300 |
commit | 0b8ebeacf5ef43a467c7ec5400ccc1ffc3fbdfba (patch) | |
tree | ae367b9b8dfa91535a38ec8a62d20184b804d58d /drivers/gpu/drm/armada/armada_drm.h | |
parent | 4bd3fd443aa4bc3cede05de201e9cf3277059477 (diff) | |
download | linux-0b8ebeacf5ef43a467c7ec5400ccc1ffc3fbdfba.tar.xz |
drm/armada: use a private mutex to protect priv->linear
Reusing the Big DRM Lock just leaks, and the few things left that
dev->struct_mutex protected are very well contained - it's just the
linear drm_mm manager.
With this armada is completely struct_mutex free!
v2: Convert things properly and also take the lock in
armada_gem_free_object, and remove the stale comment (Russell).
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'drivers/gpu/drm/armada/armada_drm.h')
-rw-r--r-- | drivers/gpu/drm/armada/armada_drm.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/gpu/drm/armada/armada_drm.h b/drivers/gpu/drm/armada/armada_drm.h index 4df6f2af2b21..3b2bb6128d40 100644 --- a/drivers/gpu/drm/armada/armada_drm.h +++ b/drivers/gpu/drm/armada/armada_drm.h @@ -57,7 +57,8 @@ struct armada_private { DECLARE_KFIFO(fb_unref, struct drm_framebuffer *, 8); struct drm_fb_helper *fbdev; struct armada_crtc *dcrtc[2]; - struct drm_mm linear; + struct drm_mm linear; /* protected by linear_lock */ + struct mutex linear_lock; struct drm_property *csc_yuv_prop; struct drm_property *csc_rgb_prop; struct drm_property *colorkey_prop; |