Bug 14735 - ACPI sleep button and lid events not generated in 2.6.32 series
Summary: ACPI sleep button and lid events not generated in 2.6.32 series
Status: CLOSED DUPLICATE of bug 14667
Alias: None
Product: ACPI
Classification: Unclassified
Component: EC (show other bugs)
Hardware: All Linux
: P1 normal
Assignee: acpi_ec
URL:
Keywords:
Depends on:
Blocks: 14230
  Show dependency tree
 
Reported: 2009-12-04 19:28 UTC by James Ettle
Modified: 2010-02-07 10:58 UTC (History)
3 users (show)

See Also:
Kernel Version: 2.6.32
Subsystem:
Regression: Yes
Bisected commit-id:


Attachments
dmesg from a session with non-functioning sleep and lid (57.80 KB, text/plain)
2009-12-04 19:28 UTC, James Ettle
Details
acpidump output (127.59 KB, text/plain)
2009-12-04 19:28 UTC, James Ettle
Details
dmesg output from 2.6.31.6-162.fc12.x86_64 (working) over a suspend/resume cycle (58.30 KB, text/plain)
2009-12-07 07:54 UTC, James Ettle
Details
grep . /sys/firmware/acpi/interrupts/* from 2.6.31.6 after suspend/resume (2.13 KB, text/plain)
2009-12-07 07:55 UTC, James Ettle
Details
grep . /sys/firmware/acpi/interrupts/* from 2.6.31.6 before suspend/resume (2.13 KB, text/plain)
2009-12-07 07:55 UTC, James Ettle
Details
sysfs grep for 2.6.32 after s/r cycle (2.13 KB, text/plain)
2009-12-07 10:35 UTC, James Ettle
Details
sysfs grep for 2.6.32 before s/r cycle (2.13 KB, text/plain)
2009-12-07 10:36 UTC, James Ettle
Details

Description James Ettle 2009-12-04 19:28:04 UTC
Created attachment 24027 [details]
dmesg from a session with non-functioning sleep and lid

ACPI events are not received on my notebook when running kernel-2.6.32-1.fc13.x86_64. The 2.6.31 series kernels worked OK. Power button
events, temperature and power information seem to be working.

The notebook is a Clevo M720R with Intel Core 2 Duo T8100 processor. dmesg and
acpidump output attached below. (I seem to recall them not working in the past, before WMI was in Linux; I think that changed around 2.6.25/26?)
Comment 1 James Ettle 2009-12-04 19:28:26 UTC
Created attachment 24028 [details]
acpidump output
Comment 2 Zhang Rui 2009-12-07 06:30:16 UTC
please attach the dmesg output in a working kernel, say 2.6.31.
please run the output of "grep . /sys/firmware/acpi/interrupts/*" both before and after pressing the power button, both in 2.6.31 and 2.6.32.
Comment 3 James Ettle 2009-12-07 07:54:37 UTC
Created attachment 24065 [details]
dmesg output from 2.6.31.6-162.fc12.x86_64 (working) over a suspend/resume cycle
Comment 4 James Ettle 2009-12-07 07:55:13 UTC
Created attachment 24066 [details]
grep . /sys/firmware/acpi/interrupts/* from 2.6.31.6 after suspend/resume
Comment 5 James Ettle 2009-12-07 07:55:38 UTC
Created attachment 24067 [details]
grep . /sys/firmware/acpi/interrupts/* from 2.6.31.6 before suspend/resume
Comment 6 James Ettle 2009-12-07 10:35:50 UTC
Created attachment 24072 [details]
sysfs grep for 2.6.32 after s/r cycle
Comment 7 James Ettle 2009-12-07 10:36:09 UTC
Created attachment 24073 [details]
sysfs grep for 2.6.32 before s/r cycle
Comment 8 Zhang Rui 2009-12-08 08:38:26 UTC
could you please verify if reverting this patch helps.

commit 6a63b06f3c494cc87eade97f081300bda60acec7
Author: Alexey Starikovskiy <astarikovskiy@suse.de>
Date:   Fri Aug 28 23:29:44 2009 +0400

    ACPI: EC: use BURST mode only for MSI notebooks
    
    Signed-off-by: Alexey Starikovskiy <astarikovskiy@suse.de>
    Signed-off-by: Len Brown <len.brown@intel.com>

diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c
index 839b542..788db78 100644
--- a/drivers/acpi/ec.c
+++ b/drivers/acpi/ec.c
@@ -575,7 +575,8 @@ acpi_ec_space_handler(u32 function, acpi_physical_address address,
        if (bits != 8 && acpi_strict)
                return AE_BAD_PARAMETER;
 
-       acpi_ec_burst_enable(ec);
+       if (EC_FLAGS_MSI)
+               acpi_ec_burst_enable(ec);
 
        if (function == ACPI_READ) {
                result = acpi_ec_read(ec, address, &temp);
@@ -596,7 +597,8 @@ acpi_ec_space_handler(u32 function, acpi_physical_address address,
                }
        }
 
-       acpi_ec_burst_disable(ec);
+       if (EC_FLAGS_MSI)
+               acpi_ec_burst_disable(ec);
 
        switch (result) {
        case -EINVAL:
Comment 9 James Ettle 2009-12-17 20:05:03 UTC
(In reply to comment #8)
> could you please verify if reverting this patch helps.
> 
> commit 6a63b06f3c494cc87eade97f081300bda60acec7

Yes, reverting that patch does help.
Comment 10 Zhang Rui 2009-12-22 02:20:49 UTC

*** This bug has been marked as a duplicate of bug 14667 ***

Note You need to log in before you can comment on or make changes to this bug.