On Tue, Aug 19, 2014 at 04:04:50PM -0700, Clint Taylor wrote:
> On 08/12/2014 07:11 AM, Jani Nikula wrote:
> >Make backlight class sysfs brightness 0 value switch off the backlight
> >for connectors that have the backlight_power callback defined. For eDP,
> >this has the similar caveats regarding power savings as bl_power as only
> >the power sequencer backlight control is switched off.
> >
> >Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx>
> >---
> > drivers/gpu/drm/i915/intel_panel.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> >diff --git a/drivers/gpu/drm/i915/intel_panel.c b/drivers/gpu/drm/i915/intel_panel.c
> >index c365f2a57c75..574690afadb3 100644
> >--- a/drivers/gpu/drm/i915/intel_panel.c
> >+++ b/drivers/gpu/drm/i915/intel_panel.c
> >@@ -986,7 +986,8 @@ static int intel_backlight_device_update_status(struct backlight_device *bd)
> > */
> > if (panel->backlight.enabled) {
> > if (panel->backlight_power) {
> >- bool enable = bd->props.power == FB_BLANK_UNBLANK;
> >+ bool enable = bd->props.power == FB_BLANK_UNBLANK &&
> >+ bd->props.brightness != 0;
> > panel->backlight_power(connector, enable);
> > }
> > } else {
> >
> Didn't get a chance to test this on nightly.
>
> Reviewed_by: Clinton Taylor <Clinton.A.Taylor@xxxxxxxxx>
All 4 patches merged, thanks.
Aside: I really liked that your review here didn't just amount to an r-b
tag, but had some comments and questions - that gives my a nice warm&fuzzy
feeling that a real review has actually been done and not just some patch
rubber stamping.
-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
