[Debian-eeepc-devel] Bug#509865: eeepc-acpi-scripts: Display brightness restore requires actual value change on 900

[Björn Wiberg]

Package: eeepc-acpi-scripts
Version: 1.0.9
Severity: normal

On my EEE 900, /etc/acpi/actions/suspend.sh needs to actually *change* the brightness value in order for the screen brightness to get restored. Just writing a value identical to the current value is 
not sufficient; the screen remains blank.

Attaching suggested patch below, tested OK here.


sprinkle:~# dmidecode -s bios-version
0906


*** suspend.sh.dpkg-dist	2008-10-14 19:58:59.000000000 +0200
--- suspend.sh	2008-11-02 08:33:43.000000000 +0100
***************
*** 20,23 ****
--- 20,27 ----
  
  brightness=$(cat $brn_control)
  pm-suspend --quirk-s3-bios
+ 
+ # CUSTOM: Value needs to be changed, not only set
+ echo $((brightness<=0?brightness+1:brightness-1)) > $brn_control
+ 
  echo $brightness > $brn_control


-- System Information:
Debian Release: 5.0
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: i386 (i686)

Kernel: Linux 2.6.26-1-686 (SMP w/1 CPU core)
Locale: LANG=sv_SE.UTF-8, LC_CTYPE=sv_SE.UTF-8 (charmap=UTF-8) (ignored: LC_ALL set to sv_SE.UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages eeepc-acpi-scripts depends on:
ii  acpi-support-base             0.109-10   scripts for handling base ACPI eve
ii  acpid                         1.0.8-1    Utilities for using ACPI power man
ii  pm-utils                      1.1.2.4-1  utilities and scripts for power ma

Versions of packages eeepc-acpi-scripts recommends:
ii  alsa-utils                    1.0.16-2   ALSA utilities

Versions of packages eeepc-acpi-scripts suggests:
pn  aosd-cat                      <none>     (no description available)
pn  gnome-osd                     <none>     (no description available)
ii  ttf-dejavu                    2.25-3     Metapackage to pull in ttf-dejavu-

-- no debconf information