Don't ignore backlight level change to the same level.

Commit Message

Matthias Hopf Aug. 18, 2009, 5:07 p.m. UTC

If set externally to a different level, this would result in a no-op.
OTOH if the display is switched off (DPMS) you do not want the change to take
place immediately, but rather to be saved and set later when the display is
active again.
---
 src/drmmode_display.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

Patch

diff --git a/src/drmmode_display.c b/src/drmmode_display.c
index f41b978..64b28ef 100644
--- a/src/drmmode_display.c
+++ b/src/drmmode_display.c
@@ -1096,10 +1096,9 @@  drmmode_output_set_property(xf86OutputPtr output, Atom property,
 	if (val < 0 || val > drmmode_output->backlight_max)
 	    return FALSE;
 
-	if (val != drmmode_output->backlight_active_level) {
+	if (drmmode_output->dpms_mode == DPMSModeOn)
 	    drmmode_backlight_set(output, val);
-	    drmmode_output->backlight_active_level = val;
-	}
+	drmmode_output->backlight_active_level = val;
 	return TRUE;
     }