drm/ast: switch to drm_*_get(), drm_*_put() helpers
authorCihangir Akturk <cakturk@gmail.com>
Fri, 11 Aug 2017 12:32:50 +0000 (15:32 +0300)
committerSean Paul <seanpaul@chromium.org>
Fri, 11 Aug 2017 15:38:47 +0000 (11:38 -0400)
Use drm_*_get() and drm_*_put() helpers instead of drm_*_reference()
and drm_*_unreference() helpers.

drm_*_reference() and drm_*_unreference() functions are just
compatibility alias for drm_*_get() and drm_*_put() and should not be
used by new code. So convert all users of compatibility functions to
use the new APIs.

Generated by: scripts/coccinelle/api/drm-get-put.cocci

Signed-off-by: Cihangir Akturk <cakturk@gmail.com>
Acked-by: Joel Stanley <joel@jms.id.au>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Link: https://patchwork.freedesktop.org/patch/msgid/1502454794-28558-5-git-send-email-cakturk@gmail.com
drivers/gpu/drm/ast/ast_fb.c
drivers/gpu/drm/ast/ast_main.c
drivers/gpu/drm/ast/ast_mode.c

index 9052ebeae8d0ac79aae1c8b13cb66abdd3a7a57d..0cd827e11fa20d8af7f038cecb8ea12465014523 100644 (file)
@@ -266,7 +266,7 @@ static void ast_fbdev_destroy(struct drm_device *dev,
        drm_fb_helper_unregister_fbi(&afbdev->helper);
 
        if (afb->obj) {
-               drm_gem_object_unreference_unlocked(afb->obj);
+               drm_gem_object_put_unlocked(afb->obj);
                afb->obj = NULL;
        }
        drm_fb_helper_fini(&afbdev->helper);
index 262c2c0e43b4b162fae7530f88b143814abe8949..cb05e8e79eb973f3e9349caf02e2e6c2c9b8884e 100644 (file)
@@ -387,7 +387,7 @@ static void ast_user_framebuffer_destroy(struct drm_framebuffer *fb)
 {
        struct ast_framebuffer *ast_fb = to_ast_framebuffer(fb);
 
-       drm_gem_object_unreference_unlocked(ast_fb->obj);
+       drm_gem_object_put_unlocked(ast_fb->obj);
        drm_framebuffer_cleanup(fb);
        kfree(fb);
 }
@@ -429,13 +429,13 @@ ast_user_framebuffer_create(struct drm_device *dev,
 
        ast_fb = kzalloc(sizeof(*ast_fb), GFP_KERNEL);
        if (!ast_fb) {
-               drm_gem_object_unreference_unlocked(obj);
+               drm_gem_object_put_unlocked(obj);
                return ERR_PTR(-ENOMEM);
        }
 
        ret = ast_framebuffer_init(dev, ast_fb, mode_cmd, obj);
        if (ret) {
-               drm_gem_object_unreference_unlocked(obj);
+               drm_gem_object_put_unlocked(obj);
                kfree(ast_fb);
                return ERR_PTR(ret);
        }
@@ -627,7 +627,7 @@ int ast_dumb_create(struct drm_file *file,
                return ret;
 
        ret = drm_gem_handle_create(file, gobj, &handle);
-       drm_gem_object_unreference_unlocked(gobj);
+       drm_gem_object_put_unlocked(gobj);
        if (ret)
                return ret;
 
@@ -675,7 +675,7 @@ ast_dumb_mmap_offset(struct drm_file *file,
        bo = gem_to_ast_bo(obj);
        *offset = ast_bo_mmap_offset(bo);
 
-       drm_gem_object_unreference_unlocked(obj);
+       drm_gem_object_put_unlocked(obj);
 
        return 0;
 
index 724c16bb6a621b5b90d031a7a2d565792f00be82..fea790b0edfac2086065f2afec993e623a036dda 100644 (file)
@@ -934,7 +934,7 @@ static void ast_cursor_fini(struct drm_device *dev)
 {
        struct ast_private *ast = dev->dev_private;
        ttm_bo_kunmap(&ast->cache_kmap);
-       drm_gem_object_unreference_unlocked(ast->cursor_cache);
+       drm_gem_object_put_unlocked(ast->cursor_cache);
 }
 
 int ast_mode_init(struct drm_device *dev)
@@ -1199,10 +1199,10 @@ static int ast_cursor_set(struct drm_crtc *crtc,
 
        ast_show_cursor(crtc);
 
-       drm_gem_object_unreference_unlocked(obj);
+       drm_gem_object_put_unlocked(obj);
        return 0;
 fail:
-       drm_gem_object_unreference_unlocked(obj);
+       drm_gem_object_put_unlocked(obj);
        return ret;
 }