On Tue, May 28, 2019 at 11:03:04AM +0200, Daniel Vetter wrote: > With all the work I've done on replacing fb notifier calls with direct > calls into fbcon the backlight/lcd notifier is the only user left. > > It will only receive events now that it cares about, hence we can > remove this check. > > Signed-off-by: Daniel Vetter <[email protected]> > Reviewed-by: Sam Ravnborg <[email protected]> > Reviewed-by: Maarten Lankhorst <[email protected]> > Cc: Lee Jones <[email protected]> > Cc: Daniel Thompson <[email protected]> > Cc: Jingoo Han <[email protected]>
Acked-by: Daniel Thompson <[email protected]> > --- > drivers/video/backlight/lcd.c | 11 ----------- > 1 file changed, 11 deletions(-) > > diff --git a/drivers/video/backlight/lcd.c b/drivers/video/backlight/lcd.c > index ecdda06989d0..d6b653aa4ee9 100644 > --- a/drivers/video/backlight/lcd.c > +++ b/drivers/video/backlight/lcd.c > @@ -30,17 +30,6 @@ static int fb_notifier_callback(struct notifier_block > *self, > struct lcd_device *ld; > struct fb_event *evdata = data; > > - /* If we aren't interested in this event, skip it immediately ... */ > - switch (event) { > - case FB_EVENT_BLANK: > - case FB_EVENT_MODE_CHANGE: > - case FB_EARLY_EVENT_BLANK: > - case FB_R_EARLY_EVENT_BLANK: > - break; > - default: > - return 0; > - } > - > ld = container_of(self, struct lcd_device, fb_notif); > if (!ld->ops) > return 0; > -- > 2.20.1 >
