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





------- Comment #36 from [EMAIL PROTECTED]  2008-01-22 16:39 -------
I alredy have applied the patch in comment#11 (As far as I understand this
patch is already in .24-rc8)
With this patch I
1) can control brightness via /proc and /sys
2) I can`t use fn keys - theq does`t work
Without this patch I
1) can not control brightness vi /sys or /proc (Only can see current value, but
I can not set it)
2) I can use fn keys

In comment#32 I have post some piece of code that allow me to use fn keys and
to control brightness via /proc (not /sys) in the same time.

On my machine there is two problems with video.c acpi driver, when patch from
comment#11 installed:
1) When I press fn key acpi_video_device_notify() with right event. So sequence
fo brightness correction called and brightness changes. BUT when from
acpi_video_device_notify() calls acpi_bus_generate_proc_event() something
hapens and, as result, acpi_video_set_brightness() calls with previous value of
brightness. As result nothing happens with brightness. And fn keys doesn`t
works. I have traced all this calls, by addin printk functon.

I can`t understand where  is  there acpi_video_set_brightness()  call with old
value of brightness. I simply disabled this call in my patch, by changing
function parametr. This is not a good idea.

2) When video.c calculates next brightness level on my laptop it does it not in
right way. The _BQC method already return next not current level! So it skip
some levels. I don`t know is it bug of driver or of my notebook... In my patch
I have fixed this for me. I don`t know how it will be work on different
machines.

>For issue 3, please make sure the value you set must be one of those gotton
from /proc/acpi/video/GFX0/DD02/brightness.
Yes it is from the list.


-- 
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