On 10/05/2016 04:20 PM, Gustavo Padovan wrote: > Hi Marek, > > 2016-10-05 Marek Vasut <marex@xxxxxxx>: > >> On 10/05/2016 03:21 PM, Daniel Vetter wrote: >>> On Wed, Oct 05, 2016 at 11:03:20AM +0200, Gustavo Padovan wrote: >>>> Hi Marek, >>>> >>>> 2016-10-05 Marek Vasut <marex@xxxxxxx>: >>>> >>>>> Remove the unnecessary wrapper functions around drm_connector_cleanup(). >>>>> >>>>> Signed-off-by: Marek Vasut <marex@xxxxxxx> >>>>> Cc: Daniel Vetter <daniel.vetter@xxxxxxxx> >>>>> --- >>>>> drivers/gpu/drm/bridge/analogix-anx78xx.c | 7 +------ >>>>> drivers/gpu/drm/bridge/nxp-ptn3460.c | 7 +------ >>>>> drivers/gpu/drm/bridge/parade-ps8622.c | 7 +------ >>>>> 3 files changed, 3 insertions(+), 18 deletions(-) >>>> >>>> Reviewed-by: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx> >>> >>> Yeah, nice little cleanup. Applied to drm-misc. >> >> So while we're at it, I also found drivers with code like [1]. Is >> calling the drm_connector_unregister() necessary in there at all ? >> Or shall we convert it the same way as the three drivers above ? > > drm_connector_register/unregister() are not necessary anymore as this is > handled by drm_dev_register() now. So these drivers can be clean up as > well. OK, got it. Patch is out. -- Best regards, Marek Vasut _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
