gnome-power-manager r3108 - in trunk: . src

[rhughes svn gnome org]

Author: rhughes
Date: Tue Dec  2 08:48:35 2008
New Revision: 3108
URL: http://svn.gnome.org/viewvc/gnome-power-manager?rev=3108&view=rev

Log:
2008-12-02  Richard Hughes  <richard hughsie com>

* src/gpm-brightness-hal.c: (gpm_brightness_hal_up),
(gpm_brightness_hal_down):
The Dell Studio 1535 and 1537 both support reading the brightness via
HAL and setting via HAL, but handles the keypress in firmware.
There is a lack in the logic for handling the OSD in this case.
Fixes #562836

Patch from Mario Limonciello <mario_limonciello dell com>, many thanks.


Modified:
   trunk/ChangeLog
   trunk/src/gpm-brightness-hal.c

Modified: trunk/src/gpm-brightness-hal.c
==============================================================================
--- trunk/src/gpm-brightness-hal.c	(original)
+++ trunk/src/gpm-brightness-hal.c	Tue Dec  2 08:48:35 2008
@@ -324,6 +324,7 @@
 	if (current_hw != brightness->priv->last_set_hw || 
             brightness->priv->does_own_updates) {
 		brightness->priv->last_set_hw = current_hw;
+		ret = TRUE;
 	} else {
 		/* macbook pro has a bazzillion brightness levels, be a bit clever */
 		step = gpm_brightness_get_step (brightness->priv->levels);
@@ -366,7 +367,8 @@
 	/* the panel has been updated in firmware */
  	if (current_hw != brightness->priv->last_set_hw ||
               brightness->priv->does_own_updates) {
-		gpm_brightness_hal_get_hw (brightness, &brightness->priv->last_set_hw);
+		brightness->priv->last_set_hw = current_hw;
+		ret = TRUE;
 	} else {
 		/* macbook pro has a bazzillion brightness levels, be a bit clever */
 		step = gpm_brightness_get_step (brightness->priv->levels);