drm/omap: gem: Replace struct_mutex usage with omap_obj private lock

The DRM device struct_mutex is used to protect against concurrent GEM
object operations that deal with memory allocation and pinning. All
those operations are local to a GEM object and don't need to be
serialized across different GEM objects. Replace the struct_mutex with
a local omap_obj.lock or drop it altogether where not needed.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

1 files changed, 1 insertions, 7 deletions

diff --git a/drivers/gpu/drm/omapdrm/omap_fbdev.c b/drivers/gpu/drm/omapdrm/omap_fbdev.c
index 0f66c74a54b0..d958cc813a94 100644
--- a/drivers/gpu/drm/omapdrm/omap_fbdev.c
+++ b/drivers/gpu/drm/omapdrm/omap_fbdev.c
@@ -170,13 +170,11 @@ static int omap_fbdev_create(struct drm_fb_helper *helper,
                 goto fail;
         }
 
-        mutex_lock(&dev->struct_mutex);
-
         fbi = drm_fb_helper_alloc_fbi(helper);
         if (IS_ERR(fbi)) {
                 dev_err(dev->dev, "failed to allocate fb info\n");
                 ret = PTR_ERR(fbi);
-                goto fail_unlock;
+                goto fail;
         }
 
         DBG("fbi=%p, dev=%p", fbi, dev);
@@ -212,12 +210,8 @@ static int omap_fbdev_create(struct drm_fb_helper *helper,
         DBG("par=%p, %dx%d", fbi->par, fbi->var.xres, fbi->var.yres);
         DBG("allocated %dx%d fb", fbdev->fb->width, fbdev->fb->height);
 
-        mutex_unlock(&dev->struct_mutex);
-
         return 0;
 
-fail_unlock:
-        mutex_unlock(&dev->struct_mutex);
 fail:
 
         if (ret) {