http://bugzilla.kernel.org/show_bug.cgi?id=9724





------- Comment #41 from [EMAIL PROTECTED]  2008-01-24 17:59 -------
(In reply to comment #39)
> At first when I kill acpid I can control brightness both via fn keys.
> when I use fn keys!
> There is a sequence of brightness levels when I use fn keys to change
> brightness from the lowest to higest and to lowest level: (yoyr patch was
> applied to the vanilla .24-rc8 kernel, my patch wasn`t applied)
> MISHA1:/usr/src# cat /proc/acpi/video/GFX0/DD02/brightness
> levels:  100 42 0 14 28 42 56 70 84 100
> current: 0
...
> MISHA1:/usr/src# cat /proc/acpi/video/GFX0/DD02/brightness
> levels:  100 42 0 14 28 42 56 70 84 100
> current: 0
You said you can control the brightness via fn keys.
so this is done with acpid killed, right?
It seems that there are two acpi events for a fn key press, or one acpi event
but it is handled twice, one in acpi video driver, and another in userspace.
And sure maybe you're right that _BQC returns the next brightness level.
Let's debug further.

> 2. echo 70 > /proc/.../brightness;
> 3. attach the result of "cat /proc/acpi/.../brightness"
> I have 
> levels:  100 42 0 14 28 42 56 70 84 100
> current: 70
With my patch applied, cat /proc/.../brightness will return the brightness
level gotton from _BQC.
This suggest that the _BQC method return the correct brightness level rather
than the next not current level.

Mihail,
can you please add a debug message at the beginning of
acpi_video_device_lcd_set_level:
+       dump_stack();
+       printk("DEBUG: acpi_video_device_lcd_set_level: %d\n", level);
and attach the dmesg output after you press the fn keys for _ONE_ time.
There should be two debug messages if the brightness levels is changed twice.


-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
acpi-bugzilla mailing list
acpi-bugzilla@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to