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





------- Comment #3 from [EMAIL PROTECTED]  2008-05-27 11:55 -------
The problem still occurs with the latest kernel (v.2.6.26-rc4).

I have done some more digging, and the problem specifically seems to occur
after a call to any of get_lcd_level(), get_auto_brightness() and
set_auto_brightness() functions in msi-laptop.c.

These functions each contain the following (lines 97, 109 and 123 in current
revision of msi-laptop.c):
    result = ec_transaction(MSI_EC_COMMAND_LCD_LEVEL, &wdata, 1, &rdata, 1, 1); 

When using the current value of 1 for force_poll in these calls (at least for
get_lcd_level(), get_auto_brightness()), reading the corresponding values from
sysfs (/sys/devices/platform/msi-laptop-pf/) always returns 0, and subsequent
reads of the battery status from /proc/acpi/battery/ return garbage values.

Changing the force_poll argument in these calls to 0 (which effectively
restores the pre-00eb43a1... behaviour) in each of these instances gives the
correct behaviour (both sysfs info and battery info), however each call takes
approximately 1 second to execute in this case (which was the original reason
for the force_poll patch).

The other calls to ec_transaction() in msi-laptop.c all work correctly
regardless of the force_poll argument.

The battery status is read correctly until one of the functions referenced
above is called, usually by reading /sys/devices/platform/msi-laptop-pf/.  The
battery state continues to be read incorrectly even if the module is unloaded.

By default, the module init function msi_init() calls set_auto_brightness() on
load, causing the erroneous behaviour when the module is loaded.  If this is
disabled with auto_brightness=2 then the battery info continues to be reported
correctly until one of the offending functions is read via sysfs.


-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

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