gnome-power-manager r3107 - in branches/gnome-2-24: . src

[rhughes svn gnome org]

Author: rhughes
Date: Tue Dec  2 08:47:40 2008
New Revision: 3107
URL: http://svn.gnome.org/viewvc/gnome-power-manager?rev=3107&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:
   branches/gnome-2-24/ChangeLog
   branches/gnome-2-24/src/gpm-brightness-hal.c

Modified: branches/gnome-2-24/src/gpm-brightness-hal.c
==============================================================================
--- branches/gnome-2-24/src/gpm-brightness-hal.c	(original)
+++ branches/gnome-2-24/src/gpm-brightness-hal.c	Tue Dec  2 08:47:40 2008
@@ -322,6 +322,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);
@@ -364,7 +365,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);