]> git.baikalelectronics.ru Git - kernel.git/commit
acpi,msi-laptop: Fall back to EC polling mode for MSI laptop specific EC commands
authorLennart Poettering <mzxreary@0pointer.de>
Fri, 4 May 2007 12:16:19 +0000 (14:16 +0200)
committerLen Brown <len.brown@intel.com>
Thu, 10 May 2007 07:52:22 +0000 (03:52 -0400)
commit420792619c71a3121d9a58820ad86057d9c3135c
tree32cab402e51882c7854eab20556f0da75525cdc7
parentf1f077c477cb0e27ba651823955eea4472ee46ee
acpi,msi-laptop: Fall back to EC polling mode for MSI laptop specific EC commands

The ACPI EC that is used in MSI laptops knows some non-standard
commands for changing the screen brighntess and a few other things,
which are used by the msi-laptop.c driver. Unfortunately for these
commands no GPE events for IBF and OBF are triggered. Since nowadays
the EC code uses the ec_intr=1 mode by default, this causes these
operations to timeout, although they don't fail. In result, all
operations that you can do with the msi-laptop.c driver take more or
less 1s to complete, which is awfully slow.

In one of the more recent kernels (2.6.20?) the EC subsystem has been
revamped. With that change the EC timeout has been increased. before
that increase the MSI EC accesses were slow -- but not *that* slow,
hence I took notice of this limitation of the MSI EC hardware only very
recently.

The standard EC operations on the MSI EC as defined in the ACPI spec
support GPE events properly.

The following patch adds a new argument "force_poll" to the
ec_transaction() function (and friends). If set to 1, the function
will poll for IBF/OBF even if ec_intr=1 is enabled. If set to 0 the
current behaviour is used. The msi-laptop driver is modified to make
use of this new flag, so that OBF/IBF is polled for the special MSI EC
transactions -- but only for them.

Signed-off-by: Lennart Poettering <mzxreary@0pointer.de>
Acked-by: Alexey Starikovskiy <aystarik@gmail.com>
Signed-off-by: Len Brown <len.brown@intel.com>
drivers/acpi/ec.c
drivers/misc/msi-laptop.c
include/linux/acpi.h