Re: [Intel-gfx] [PATCH 07/52] drm/udl: Use drmm_add_final_kfree
On Wed, Feb 19, 2020 at 2:42 PM Laurent Pinchart wrote: > > Hi Daniel, > > Thank you for the patch. > > On Wed, Feb 19, 2020 at 11:20:37AM +0100, Daniel Vetter wrote: > > With this we can drop the final kfree from the release function. > > > > v2: We need drm_dev_put to unroll the driver creation (once > > drm_dev_init and drmm_add_final_kfree suceeded), otherwise > > the drmm_ magic doesn't happen. > > > > Signed-off-by: Daniel Vetter > > Did you mean to squash this with the previous commit ? Uh yes indeed. Will squash. -Daniel > > > Cc: Dave Airlie > > Cc: Sean Paul > > Cc: Thomas Zimmermann > > Cc: Emil Velikov > > Cc: Daniel Vetter > > Cc: "Noralf Trønnes" > > Cc: Thomas Gleixner > > Cc: Sam Ravnborg > > --- > > drivers/gpu/drm/udl/udl_drv.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/udl/udl_drv.c b/drivers/gpu/drm/udl/udl_drv.c > > index d5b89711ab1e..6a5594946096 100644 > > --- a/drivers/gpu/drm/udl/udl_drv.c > > +++ b/drivers/gpu/drm/udl/udl_drv.c > > @@ -81,8 +81,7 @@ static struct udl_device *udl_driver_create(struct > > usb_interface *interface) > > > > r = udl_init(udl); > > if (r) { > > - drm_dev_fini(>drm); > > - kfree(udl); > > + drm_dev_put(>drm); > > return ERR_PTR(r); > > } > > > > -- > Regards, > > Laurent Pinchart -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH 07/52] drm/udl: Use drmm_add_final_kfree
Hi Daniel, Thank you for the patch. On Wed, Feb 19, 2020 at 11:20:37AM +0100, Daniel Vetter wrote: > With this we can drop the final kfree from the release function. > > v2: We need drm_dev_put to unroll the driver creation (once > drm_dev_init and drmm_add_final_kfree suceeded), otherwise > the drmm_ magic doesn't happen. > > Signed-off-by: Daniel Vetter Did you mean to squash this with the previous commit ? > Cc: Dave Airlie > Cc: Sean Paul > Cc: Thomas Zimmermann > Cc: Emil Velikov > Cc: Daniel Vetter > Cc: "Noralf Trønnes" > Cc: Thomas Gleixner > Cc: Sam Ravnborg > --- > drivers/gpu/drm/udl/udl_drv.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/udl/udl_drv.c b/drivers/gpu/drm/udl/udl_drv.c > index d5b89711ab1e..6a5594946096 100644 > --- a/drivers/gpu/drm/udl/udl_drv.c > +++ b/drivers/gpu/drm/udl/udl_drv.c > @@ -81,8 +81,7 @@ static struct udl_device *udl_driver_create(struct > usb_interface *interface) > > r = udl_init(udl); > if (r) { > - drm_dev_fini(>drm); > - kfree(udl); > + drm_dev_put(>drm); > return ERR_PTR(r); > } > -- Regards, Laurent Pinchart ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
[Intel-gfx] [PATCH 07/52] drm/udl: Use drmm_add_final_kfree
With this we can drop the final kfree from the release function. v2: We need drm_dev_put to unroll the driver creation (once drm_dev_init and drmm_add_final_kfree suceeded), otherwise the drmm_ magic doesn't happen. Signed-off-by: Daniel Vetter Cc: Dave Airlie Cc: Sean Paul Cc: Thomas Zimmermann Cc: Emil Velikov Cc: Daniel Vetter Cc: "Noralf Trønnes" Cc: Thomas Gleixner Cc: Sam Ravnborg --- drivers/gpu/drm/udl/udl_drv.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/udl/udl_drv.c b/drivers/gpu/drm/udl/udl_drv.c index d5b89711ab1e..6a5594946096 100644 --- a/drivers/gpu/drm/udl/udl_drv.c +++ b/drivers/gpu/drm/udl/udl_drv.c @@ -81,8 +81,7 @@ static struct udl_device *udl_driver_create(struct usb_interface *interface) r = udl_init(udl); if (r) { - drm_dev_fini(>drm); - kfree(udl); + drm_dev_put(>drm); return ERR_PTR(r); } -- 2.24.1 ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx