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: >=20 > Most notebooks have special keys (mostly Fn+something) to change = brightness of LCD display. >=20 > 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. >=20 > I implemented such handling as part of acpi-video module. >=20 > It's verified to work on my HP Mini 5101 & FreeBSD 8.0. >=20 > No test on other notebook done as I have no other notebook. >=20 > 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. >=20 > If someone want to test it then the patch is here: > http://www.freebsd.cz/~dan/patch-acpi-video >=20 > 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. >=20 > I'm interested to know on which notebook and FreeBSD version is = verified to be (not)working. >=20 > 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) >=20 > 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=3D"ACPI_LV_ALL_EXCEPTIONS" > then look on console messages what happens when you press the special = keys. >=20 > 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>