On Thu, Jan 16, 2014 at 05:55:48PM +0100, Daniel Vetter wrote: > 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 ... My assumption has ben that the intel backlight stuff isn't hooked up to anything. But I guess I could give it a go. -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx