On Thu, Jan 16, 2014 at 5:27 PM, <ville.syrjala@xxxxxxxxxxxxxxx> wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > My 855gm doesn't register the intel backlight but it still ends up > calling the backlight code to enable/disable the backlight via the > LVDS code. This leads to some WARNs due to backlight.max being 0. > > Let's have intel_panel_enable_backlight() and intel_panel_disable_backlight() > check whether there's a backlight present or not. > > Also move the backlight.present check from asle_set_backlight() into > intel_panel_set_backlight() for some extra symmetry. > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> Wouldn't that be a perfect machine to test Jani's init-backlight-from-vbt patch? Iirc all the infrastructure is now there ... -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx