summaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm/armada
diff options
context:
space:
mode:
authorRussell King <rmk+kernel@arm.linux.org.uk>2015-07-15 18:11:24 +0100
committerRussell King <rmk+kernel@arm.linux.org.uk>2015-10-01 14:32:52 +0100
commit66377efa3fd468283a092f17692e81c2344b03ed (patch)
tree602c3b33b5c330dc20495cba33179ce7c53563ee /drivers/gpu/drm/armada
parentfecfdb2db8b5fc2e6cb731a714889de5e43c2380 (diff)
downloadblackbird-op-linux-66377efa3fd468283a092f17692e81c2344b03ed.tar.gz
blackbird-op-linux-66377efa3fd468283a092f17692e81c2344b03ed.zip
drm/armada: use xchg() to atomically update dplane->old_fb
Rather than using a spinlock, use xchg() to atomically update dplane->old_fb. This allows us to eliminate dplane->lock. Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'drivers/gpu/drm/armada')
-rw-r--r--drivers/gpu/drm/armada/armada_overlay.c12
1 files changed, 2 insertions, 10 deletions
diff --git a/drivers/gpu/drm/armada/armada_overlay.c b/drivers/gpu/drm/armada/armada_overlay.c
index 093c2d4f2b79..5e2dcecbf6ef 100644
--- a/drivers/gpu/drm/armada/armada_overlay.c
+++ b/drivers/gpu/drm/armada/armada_overlay.c
@@ -31,7 +31,6 @@ struct armada_ovl_plane_properties {
struct armada_ovl_plane {
struct drm_plane base;
- spinlock_t lock;
struct drm_framebuffer *old_fb;
uint32_t src_hw;
uint32_t dst_hw;
@@ -76,10 +75,7 @@ static void armada_ovl_retire_fb(struct armada_ovl_plane *dplane,
{
struct drm_framebuffer *old_fb;
- spin_lock(&dplane->lock);
- old_fb = dplane->old_fb;
- dplane->old_fb = fb;
- spin_unlock(&dplane->lock);
+ old_fb = xchg(&dplane->old_fb, fb);
if (old_fb)
armada_drm_queue_unref_work(dplane->base.dev, old_fb);
@@ -289,10 +285,7 @@ static int armada_ovl_plane_disable(struct drm_plane *plane)
if (plane->fb)
drm_framebuffer_unreference(plane->fb);
- spin_lock_irq(&dplane->lock);
- fb = dplane->old_fb;
- dplane->old_fb = NULL;
- spin_unlock_irq(&dplane->lock);
+ fb = xchg(&dplane->old_fb, NULL);
if (fb)
drm_framebuffer_unreference(fb);
@@ -464,7 +457,6 @@ int armada_overlay_plane_create(struct drm_device *dev, unsigned long crtcs)
if (!dplane)
return -ENOMEM;
- spin_lock_init(&dplane->lock);
init_waitqueue_head(&dplane->vbl.wait);
armada_drm_vbl_event_init(&dplane->vbl.update, armada_ovl_plane_vbl,
dplane);
OpenPOWER on IntegriCloud