On 12/03/2018 21:15, Christophe JAILLET wrote: > According to 'drivers/gpu/drm/drm_drv.c', 'drm_dev_unref()' is just a > compatibility alias for 'drm_dev_put()'. So use the latter instead. > > Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> > --- > drivers/gpu/drm/meson/meson_drv.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/meson/meson_drv.c b/drivers/gpu/drm/meson/meson_drv.c > index 02b0886debc0..6ee3dd7fa450 100644 > --- a/drivers/gpu/drm/meson/meson_drv.c > +++ b/drivers/gpu/drm/meson/meson_drv.c > @@ -284,7 +284,7 @@ static int meson_drv_bind_master(struct device *dev, bool has_components) > return 0; > > free_drm: > - drm_dev_unref(drm); > + drm_dev_put(drm); > > return ret; > } > @@ -303,7 +303,7 @@ static void meson_drv_unbind(struct device *dev) > drm_kms_helper_poll_fini(drm); > drm_fbdev_cma_fini(priv->fbdev); > drm_mode_config_cleanup(drm); > - drm_dev_unref(drm); > + drm_dev_put(drm); > > } > > It's not useless, there is a clear statement : * This is a compatibility alias for drm_dev_put() and should not be used by new * code. Acked-by: Neil Armstrong <narmstrong@xxxxxxxxxxxx> Thanks, Neil _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
