Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 30 Mar 2010 09:47:12 +0100
From:      Rui Paulo <rpaulo@freebsd.org>
To:        Dan Lukes <dan@obluda.cz>
Cc:        freebsd-acpi@freebsd.org
Subject:   Re: Brightness change on notebook that follow ACPI specification (par. B.7)
Message-ID:  <8D820BAA-C1B9-408C-AAF3-EA69612D2C46@freebsd.org>
In-Reply-To: <4BB170E7.3030407@obluda.cz>
References:  <4BB170E7.3030407@obluda.cz>

next in thread | previous in thread | raw e-mail | index | archive | help


On 30 Mar 2010, at 04:32, Dan Lukes wrote:

> 
> Most notebooks have special keys (mostly Fn+something) to change brightness of LCD display.
> 
> Some of them (my notebok, for example) follows the ACPI specification (paragraph B.7) how to announce the user request for brightness change to OS.
> 
> I implemented such handling as part of acpi-video module.
> 
> It's verified to work on my HP Mini 5101 & FreeBSD 8.0.
> 
> No test on other notebook done as I have no other notebook.
> 
> No test on other OS version, but verified that patched module can be compiled on 6.4-R, 7.2-R, 8.0-R. I assume (not verified) it's compilable on all 6.x-8.x. I assume (not verified) it should work on all of these versions.
> 
> If someone want to test it then the patch is here:
> http://www.freebsd.cz/~dan/patch-acpi-video
> 
> installation:
> cd /usr/src ; patch < patch-acpi-video
> then compile the module:
> http://www.freebsd.org/doc/handbook/acpi-debug.html#ACPI-DEBUGOUTPUT
> then kldload acpi_video
> then try keys for brightness change.
> 
> I'm interested to know on which notebook and FreeBSD version is verified to be (not)working.
> 
> If it doesn't work it may be either:
> 1. bug in my patch
> 2. your notebook doesn't follow ACPI specification (par. B.7)
> 
> If you are skilled user and you want to know more (and you have compiled ACPI modules with ACPI_DEBUG) you can set
> debug.acpi.level="ACPI_LV_ALL_EXCEPTIONS"
> then look on console messages what happens when you press the special keys.
> 
> 							Dan

I see nothing wrong with this patch and I think it can be committed, but I would like others to take a look.

--
Rui Paulo




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?8D820BAA-C1B9-408C-AAF3-EA69612D2C46>