On Sat, Feb 15, 2020 at 08:09:40PM +0100, Daniel Vetter wrote: > On Sat, Feb 15, 2020 at 6:33 PM Sam Ravnborg <sam@xxxxxxxxxxxx> wrote: > > > > struct drm_encoder_helper_funcs included a callback > > named drm_crtc. > > > > There are no users left - so drop it. > > There was one reference in drm_crtc_helper.c, > > which checked if the value was not NULL. > > As it was never assigned this check could be dropped. > > > > Signed-off-by: Sam Ravnborg <sam@xxxxxxxxxxxx> > > Cc: Daniel Vetter <daniel.vetter@xxxxxxxx> > > --- > > > > Stumbled upon this and dediced to check if it was used. > > Build tested with allyesconfig, allmodconfig for relevant architectures. > > Nice catch! > > Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx> Thanks for the quick review. Applied and pushed out. Sam _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
