Re: [PATCH v3 23/28] drm/udl: switch to drm_*_get(), drm_*_put() helpers
On Fri, Aug 11, 2017 at 03:33:09PM +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 AkturkApplied to drm-misc-next, thank you! Sean > --- > drivers/gpu/drm/udl/udl_fb.c | 6 +++--- > drivers/gpu/drm/udl/udl_gem.c | 4 ++-- > 2 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/udl/udl_fb.c b/drivers/gpu/drm/udl/udl_fb.c > index 77a2c59..b7ca90d 100644 > --- a/drivers/gpu/drm/udl/udl_fb.c > +++ b/drivers/gpu/drm/udl/udl_fb.c > @@ -309,7 +309,7 @@ static void udl_user_framebuffer_destroy(struct > drm_framebuffer *fb) > struct udl_framebuffer *ufb = to_udl_fb(fb); > > if (ufb->obj) > - drm_gem_object_unreference_unlocked(>obj->base); > + drm_gem_object_put_unlocked(>obj->base); > > drm_framebuffer_cleanup(fb); > kfree(ufb); > @@ -403,7 +403,7 @@ static int udlfb_create(struct drm_fb_helper *helper, > > return ret; > out_gfree: > - drm_gem_object_unreference_unlocked(>ufb.obj->base); > + drm_gem_object_put_unlocked(>ufb.obj->base); > out: > return ret; > } > @@ -419,7 +419,7 @@ static void udl_fbdev_destroy(struct drm_device *dev, > drm_fb_helper_fini(>helper); > drm_framebuffer_unregister_private(>ufb.base); > drm_framebuffer_cleanup(>ufb.base); > - drm_gem_object_unreference_unlocked(>ufb.obj->base); > + drm_gem_object_put_unlocked(>ufb.obj->base); > } > > int udl_fbdev_init(struct drm_device *dev) > diff --git a/drivers/gpu/drm/udl/udl_gem.c b/drivers/gpu/drm/udl/udl_gem.c > index db9cece..dee6bd9 100644 > --- a/drivers/gpu/drm/udl/udl_gem.c > +++ b/drivers/gpu/drm/udl/udl_gem.c > @@ -52,7 +52,7 @@ udl_gem_create(struct drm_file *file, > return ret; > } > > - drm_gem_object_unreference_unlocked(>base); > + drm_gem_object_put_unlocked(>base); > *handle_p = handle; > return 0; > } > @@ -234,7 +234,7 @@ int udl_gem_mmap(struct drm_file *file, struct drm_device > *dev, > *offset = drm_vma_node_offset_addr(>base.vma_node); > > out: > - drm_gem_object_unreference(>base); > + drm_gem_object_put(>base); > unlock: > mutex_unlock(>struct_mutex); > return ret; > -- > 2.7.4 > -- Sean Paul, Software Engineer, Google / Chromium OS
Re: [PATCH v3 23/28] drm/udl: switch to drm_*_get(), drm_*_put() helpers
On Fri, Aug 11, 2017 at 03:33:09PM +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 Applied to drm-misc-next, thank you! Sean > --- > drivers/gpu/drm/udl/udl_fb.c | 6 +++--- > drivers/gpu/drm/udl/udl_gem.c | 4 ++-- > 2 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/udl/udl_fb.c b/drivers/gpu/drm/udl/udl_fb.c > index 77a2c59..b7ca90d 100644 > --- a/drivers/gpu/drm/udl/udl_fb.c > +++ b/drivers/gpu/drm/udl/udl_fb.c > @@ -309,7 +309,7 @@ static void udl_user_framebuffer_destroy(struct > drm_framebuffer *fb) > struct udl_framebuffer *ufb = to_udl_fb(fb); > > if (ufb->obj) > - drm_gem_object_unreference_unlocked(>obj->base); > + drm_gem_object_put_unlocked(>obj->base); > > drm_framebuffer_cleanup(fb); > kfree(ufb); > @@ -403,7 +403,7 @@ static int udlfb_create(struct drm_fb_helper *helper, > > return ret; > out_gfree: > - drm_gem_object_unreference_unlocked(>ufb.obj->base); > + drm_gem_object_put_unlocked(>ufb.obj->base); > out: > return ret; > } > @@ -419,7 +419,7 @@ static void udl_fbdev_destroy(struct drm_device *dev, > drm_fb_helper_fini(>helper); > drm_framebuffer_unregister_private(>ufb.base); > drm_framebuffer_cleanup(>ufb.base); > - drm_gem_object_unreference_unlocked(>ufb.obj->base); > + drm_gem_object_put_unlocked(>ufb.obj->base); > } > > int udl_fbdev_init(struct drm_device *dev) > diff --git a/drivers/gpu/drm/udl/udl_gem.c b/drivers/gpu/drm/udl/udl_gem.c > index db9cece..dee6bd9 100644 > --- a/drivers/gpu/drm/udl/udl_gem.c > +++ b/drivers/gpu/drm/udl/udl_gem.c > @@ -52,7 +52,7 @@ udl_gem_create(struct drm_file *file, > return ret; > } > > - drm_gem_object_unreference_unlocked(>base); > + drm_gem_object_put_unlocked(>base); > *handle_p = handle; > return 0; > } > @@ -234,7 +234,7 @@ int udl_gem_mmap(struct drm_file *file, struct drm_device > *dev, > *offset = drm_vma_node_offset_addr(>base.vma_node); > > out: > - drm_gem_object_unreference(>base); > + drm_gem_object_put(>base); > unlock: > mutex_unlock(>struct_mutex); > return ret; > -- > 2.7.4 > -- Sean Paul, Software Engineer, Google / Chromium OS
[PATCH v3 23/28] drm/udl: switch to drm_*_get(), drm_*_put() helpers
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--- drivers/gpu/drm/udl/udl_fb.c | 6 +++--- drivers/gpu/drm/udl/udl_gem.c | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/udl/udl_fb.c b/drivers/gpu/drm/udl/udl_fb.c index 77a2c59..b7ca90d 100644 --- a/drivers/gpu/drm/udl/udl_fb.c +++ b/drivers/gpu/drm/udl/udl_fb.c @@ -309,7 +309,7 @@ static void udl_user_framebuffer_destroy(struct drm_framebuffer *fb) struct udl_framebuffer *ufb = to_udl_fb(fb); if (ufb->obj) - drm_gem_object_unreference_unlocked(>obj->base); + drm_gem_object_put_unlocked(>obj->base); drm_framebuffer_cleanup(fb); kfree(ufb); @@ -403,7 +403,7 @@ static int udlfb_create(struct drm_fb_helper *helper, return ret; out_gfree: - drm_gem_object_unreference_unlocked(>ufb.obj->base); + drm_gem_object_put_unlocked(>ufb.obj->base); out: return ret; } @@ -419,7 +419,7 @@ static void udl_fbdev_destroy(struct drm_device *dev, drm_fb_helper_fini(>helper); drm_framebuffer_unregister_private(>ufb.base); drm_framebuffer_cleanup(>ufb.base); - drm_gem_object_unreference_unlocked(>ufb.obj->base); + drm_gem_object_put_unlocked(>ufb.obj->base); } int udl_fbdev_init(struct drm_device *dev) diff --git a/drivers/gpu/drm/udl/udl_gem.c b/drivers/gpu/drm/udl/udl_gem.c index db9cece..dee6bd9 100644 --- a/drivers/gpu/drm/udl/udl_gem.c +++ b/drivers/gpu/drm/udl/udl_gem.c @@ -52,7 +52,7 @@ udl_gem_create(struct drm_file *file, return ret; } - drm_gem_object_unreference_unlocked(>base); + drm_gem_object_put_unlocked(>base); *handle_p = handle; return 0; } @@ -234,7 +234,7 @@ int udl_gem_mmap(struct drm_file *file, struct drm_device *dev, *offset = drm_vma_node_offset_addr(>base.vma_node); out: - drm_gem_object_unreference(>base); + drm_gem_object_put(>base); unlock: mutex_unlock(>struct_mutex); return ret; -- 2.7.4
[PATCH v3 23/28] drm/udl: switch to drm_*_get(), drm_*_put() helpers
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 --- drivers/gpu/drm/udl/udl_fb.c | 6 +++--- drivers/gpu/drm/udl/udl_gem.c | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/udl/udl_fb.c b/drivers/gpu/drm/udl/udl_fb.c index 77a2c59..b7ca90d 100644 --- a/drivers/gpu/drm/udl/udl_fb.c +++ b/drivers/gpu/drm/udl/udl_fb.c @@ -309,7 +309,7 @@ static void udl_user_framebuffer_destroy(struct drm_framebuffer *fb) struct udl_framebuffer *ufb = to_udl_fb(fb); if (ufb->obj) - drm_gem_object_unreference_unlocked(>obj->base); + drm_gem_object_put_unlocked(>obj->base); drm_framebuffer_cleanup(fb); kfree(ufb); @@ -403,7 +403,7 @@ static int udlfb_create(struct drm_fb_helper *helper, return ret; out_gfree: - drm_gem_object_unreference_unlocked(>ufb.obj->base); + drm_gem_object_put_unlocked(>ufb.obj->base); out: return ret; } @@ -419,7 +419,7 @@ static void udl_fbdev_destroy(struct drm_device *dev, drm_fb_helper_fini(>helper); drm_framebuffer_unregister_private(>ufb.base); drm_framebuffer_cleanup(>ufb.base); - drm_gem_object_unreference_unlocked(>ufb.obj->base); + drm_gem_object_put_unlocked(>ufb.obj->base); } int udl_fbdev_init(struct drm_device *dev) diff --git a/drivers/gpu/drm/udl/udl_gem.c b/drivers/gpu/drm/udl/udl_gem.c index db9cece..dee6bd9 100644 --- a/drivers/gpu/drm/udl/udl_gem.c +++ b/drivers/gpu/drm/udl/udl_gem.c @@ -52,7 +52,7 @@ udl_gem_create(struct drm_file *file, return ret; } - drm_gem_object_unreference_unlocked(>base); + drm_gem_object_put_unlocked(>base); *handle_p = handle; return 0; } @@ -234,7 +234,7 @@ int udl_gem_mmap(struct drm_file *file, struct drm_device *dev, *offset = drm_vma_node_offset_addr(>base.vma_node); out: - drm_gem_object_unreference(>base); + drm_gem_object_put(>base); unlock: mutex_unlock(>struct_mutex); return ret; -- 2.7.4