[PATCH 09/14] drm/mgag200: Add missing drm_connector_unregister()

Daniel Vetter daniel at ffwll.ch
Thu Jul 20 08:15:12 UTC 2017


On Wed, Jul 19, 2017 at 10:44:15AM +0200, Takashi Iwai wrote:
> On Tue, 18 Jul 2017 16:43:15 +0200,
> Takashi Iwai wrote:
> > 
> > From: Egbert Eich <eich at suse.de>
> > 
> > When destroying connector unregister it.
> > 
> > Signed-off-by: Egbert Eich <eich at suse.de>
> > Signed-off-by: Takashi Iwai <tiwai at suse.de>
> 
> This one should be also superfluous as Daniel pointed for the similar
> patch for cirrus.  Please disregard this one.
> 
> 
> BTW, I guess the patch was inspired from ast driver code, but then it
> implies that the call in ast_mode.c is also superfluous.
> Shall I cook up a patch?

Would be good. The entire drm unload code paths are in fairly bad shape,
with lots of cargo-culted copypasta going on ...
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch


More information about the dri-devel mailing list