drm/omap: Stop using drm_framebuffer_unregister_private
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 27 Dec 2016 10:49:24 +0000 (11:49 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 12 Jan 2017 10:03:35 +0000 (11:03 +0100)
This is the deprecated function for when you embedded the framebuffer
somewhere else (which breaks refcounting). But omapdrm is using
drm_framebuffer_remove and a free-standing fb, so this is rendundant.

Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1482835765-12044-4-git-send-email-daniel.vetter@ffwll.ch
drivers/gpu/drm/omapdrm/omap_fbdev.c

index aed99a0fc44bb7a6210d5e723652d14fecf76091..2a839956dae620df7ecb1c4d223fa1429641680c 100644 (file)
@@ -225,10 +225,8 @@ fail:
 
                drm_fb_helper_release_fbi(helper);
 
-               if (fb) {
-                       drm_framebuffer_unregister_private(fb);
+               if (fb)
                        drm_framebuffer_remove(fb);
-               }
        }
 
        return ret;
@@ -314,10 +312,8 @@ void omap_fbdev_free(struct drm_device *dev)
        omap_gem_put_paddr(fbdev->bo);
 
        /* this will free the backing object */
-       if (fbdev->fb) {
-               drm_framebuffer_unregister_private(fbdev->fb);
+       if (fbdev->fb)
                drm_framebuffer_remove(fbdev->fb);
-       }
 
        kfree(fbdev);