Re: xbacklight on intel

["Khashayar Naderehvandi" <khashayar.lists@xxxxxxxxx>]

Woops, forgot to add the mailing list, and attach the patch. Sending again.

On Sat, Mar 1, 2008 at 12:06 PM, Khashayar Naderehvandi <khashayar.lists@xxxxxxxxx> wrote:

Yeah, please file a bug.  In the meantime, you can
add '/sys/class/backlight/fujitsu' to the list of backlight directories
in i830_lvds.c then rebuild your driver to make sure it'll work for
you.

Here's that bug: https://bugs.freedesktop.org/show_bug.cgi?id=14706.
And I've attached the tiny patch against the intel driver that fixes this for me.

However, there is one little issue left. If I change the backlight using the hardware controlled function keys, xbacklight -get still returns what it had set itself previously. This seems to be because /sys/class/backlight/fujitsu-laptop contains one file called 'brightness' and one called 'actual_brightness'. 'actual_brightness' is read-only, and echoing a number to 'brightness' changes the backlight brightness as well as 'actual_brightness' properly. However, when changing the backlight through the function keys, only actual_brightness changes.

Thinking of it, this might be a bug of the fujitsu-laptop kernel module... not sure.

Regards,
K.

diff -Naur xserver-xorg-video-intel-2.2.1/src/i830_lvds.c xserver-xorg-video-intel-2.2.1-new/src/i830_lvds.c
--- xserver-xorg-video-intel-2.2.1/src/i830_lvds.c	2008-02-20 22:41:25.000000000 +0100
+++ xserver-xorg-video-intel-2.2.1-new/src/i830_lvds.c	2008-03-01 11:37:09.000000000 +0100
@@ -68,6 +68,7 @@
     "thinkpad_screen",
     "acpi_video1",
     "acpi_video0",
+    "fujitsu-laptop",
     NULL,
 };
 

_______________________________________________
xorg mailing list
xorg@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/xorg