On Fri, Aug 11, 2017 at 03:33:06PM +0300, Cihangir Akturk wrote:
> 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 <cakt...@gmail.com>

Applied to drm-misc-next, thank you!

Sean

> ---
>  drivers/gpu/drm/rockchip/rockchip_drm_fb.c    | 6 +++---
>  drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c | 2 +-
>  drivers/gpu/drm/rockchip/rockchip_drm_gem.c   | 2 +-
>  drivers/gpu/drm/rockchip/rockchip_drm_vop.c   | 4 ++--
>  4 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_fb.c 
> b/drivers/gpu/drm/rockchip/rockchip_drm_fb.c
> index 8a0f756..7077304 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_fb.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_fb.c
> @@ -48,7 +48,7 @@ static void rockchip_drm_fb_destroy(struct drm_framebuffer 
> *fb)
>       int i;
>  
>       for (i = 0; i < ROCKCHIP_MAX_FB_BUFFER; i++)
> -             drm_gem_object_unreference_unlocked(rockchip_fb->obj[i]);
> +             drm_gem_object_put_unlocked(rockchip_fb->obj[i]);
>  
>       drm_framebuffer_cleanup(fb);
>       kfree(rockchip_fb);
> @@ -144,7 +144,7 @@ rockchip_user_fb_create(struct drm_device *dev, struct 
> drm_file *file_priv,
>                       width * drm_format_plane_cpp(mode_cmd->pixel_format, i);
>  
>               if (obj->size < min_size) {
> -                     drm_gem_object_unreference_unlocked(obj);
> +                     drm_gem_object_put_unlocked(obj);
>                       ret = -EINVAL;
>                       goto err_gem_object_unreference;
>               }
> @@ -161,7 +161,7 @@ rockchip_user_fb_create(struct drm_device *dev, struct 
> drm_file *file_priv,
>  
>  err_gem_object_unreference:
>       for (i--; i >= 0; i--)
> -             drm_gem_object_unreference_unlocked(objs[i]);
> +             drm_gem_object_put_unlocked(objs[i]);
>       return ERR_PTR(ret);
>  }
>  
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c 
> b/drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c
> index ce946b9..724579e 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c
> @@ -173,7 +173,7 @@ void rockchip_drm_fbdev_fini(struct drm_device *dev)
>       drm_fb_helper_unregister_fbi(helper);
>  
>       if (helper->fb)
> -             drm_framebuffer_unreference(helper->fb);
> +             drm_framebuffer_put(helper->fb);
>  
>       drm_fb_helper_fini(helper);
>  }
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c 
> b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> index f74333e..1869c8b 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> @@ -383,7 +383,7 @@ rockchip_gem_create_with_handle(struct drm_file 
> *file_priv,
>               goto err_handle_create;
>  
>       /* drop reference from allocate - handle holds it now. */
> -     drm_gem_object_unreference_unlocked(obj);
> +     drm_gem_object_put_unlocked(obj);
>  
>       return rk_obj;
>  
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c 
> b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
> index 948719d..bf9ed0e 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
> @@ -1026,7 +1026,7 @@ static void vop_crtc_atomic_flush(struct drm_crtc *crtc,
>               if (old_plane_state->fb == new_plane_state->fb)
>                       continue;
>  
> -             drm_framebuffer_reference(old_plane_state->fb);
> +             drm_framebuffer_get(old_plane_state->fb);
>               drm_flip_work_queue(&vop->fb_unref_work, old_plane_state->fb);
>               set_bit(VOP_PENDING_FB_UNREF, &vop->pending);
>               WARN_ON(drm_crtc_vblank_get(crtc) != 0);
> @@ -1150,7 +1150,7 @@ static void vop_fb_unref_worker(struct drm_flip_work 
> *work, void *val)
>       struct drm_framebuffer *fb = val;
>  
>       drm_crtc_vblank_put(&vop->crtc);
> -     drm_framebuffer_unreference(fb);
> +     drm_framebuffer_put(fb);
>  }
>  
>  static void vop_handle_vblank(struct vop *vop)
> -- 
> 2.7.4
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-ker...@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

-- 
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to