Re: Call for testers: New acpi_ibm driver

In message <[email protected]>, Markus Brueffer wrote:
>--nextPart1704232.STgpkmBTov
>Content-Type: text/plain;
>  charset="iso-8859-1"
>Content-Transfer-Encoding: quoted-printable
>Content-Disposition: inline
>
>Hi,
>
>I'm looking for testers for a mostly rewritten and enhanced version of the=
>=20
>acpi_ibm (ACPI Thinkpad Extras) driver.
>
>You can find it along with a first draft of the new manpage at:
>
>http://people.freebsd.org/~markus/ibm/
>
>If you are running -CURRENT, simply replace the former driver with the new
>one in sys/dev/acpi_support and rebuild it. (sorry, no patch for 5.x for
>now).

>Please mail me on failure and success and include the output of=20
># sysctl hw.acpi.ibm

hw.acpi.ibm.initialmask: 2060
hw.acpi.ibm.availmask: 2524
hw.acpi.ibm.events: 0
hw.acpi.ibm.eventmask: 2060
hw.acpi.ibm.hotkey: 308
hw.acpi.ibm.lcd_brightness: 7
hw.acpi.ibm.volume: 14
hw.acpi.ibm.mute: 0
hw.acpi.ibm.thinklight: 0
hw.acpi.ibm.bluetooth: 1
hw.acpi.ibm.wlan: 1
hw.acpi.ibm.fan_rpm: 0
hw.acpi.ibm.fan_status: 1
hw.acpi.ibm.thermal: 55 52 55 47 38 -1 27 -1

>In any case: Please send me your DSDT for further improvement of the driver=
>=20
>(even if you run FreeBSD 5):
># acpidump -t -d > name-system.asl

http://www.init-main.com/acpi_tpkey/TPX31JHJ.asl

>Changes:
>
> Relocation of the sysctl tree to hw.acpi.ibm and renaming of some sysctls

I don't think it a good thing. New device specific tree
should not be introduced. 

> Support for more hotkeys by the hotkey sysctl (see manpage)
> Improved support of ACPI events, disabled by default. You can enable it
> by setting the events sysctl to 1 (on my T41p there are now 24 different
>keypress events that get reported)
> write support for: volume, mute, brightness and thinklight

Is it ok to  touch EC? I know the two way of impliment it.
One is modifying RTC register, another is using SMI interface
via ACPI control method.

> led(4) interface for the thinklight (thanks simon)
> support for reading the fan status and speed
> support for reading up to 8 thermal sensors

Works great.