On Tue, 2008-11-11 at 16:09 +0800, Alexey Starikovskiy wrote: > Referencies: http://bugzilla.kernel.org/show_bug.cgi?id=12004 > > Signed-off-by: Alexey Starikovskiy <astarikovskiy@xxxxxxx> > --- > > drivers/acpi/ec.c | 11 ++++++++--- > 1 files changed, 8 insertions(+), 3 deletions(-) > > > diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c > index 628b076..b1dcc08 100644 > --- a/drivers/acpi/ec.c > +++ b/drivers/acpi/ec.c > @@ -576,11 +576,15 @@ static u32 acpi_ec_gpe_handler(void *data) > pr_debug(PREFIX "~~~> interrupt\n"); > status = acpi_ec_read_status(ec); > > - gpe_transaction(ec, status); > - if (ec_transaction_done(ec) && (status & ACPI_EC_FLAG_IBF) == 0) > - wake_up(&ec->wait); > + if (test_bit(EC_FLAGS_GPE_MODE, &ec->flags)) { > + gpe_transaction(ec, status); > + if (ec_transaction_done(ec) && > + (status & ACPI_EC_FLAG_IBF) == 0) > + wake_up(&ec->wait); > + } > It seems that the EC won't be switched to EC GPE mode in the EC GPE handler again after the following is added. >#if 0 >#endif Is it appropriate? If what I said is incorrect, please ignore it. Thanks. > ec_check_sci(ec, status); > +#if 0 > if (!test_bit(EC_FLAGS_GPE_MODE, &ec->flags) && > !test_bit(EC_FLAGS_NO_GPE, &ec->flags)) { > /* this is non-query, must be confirmation */ > @@ -589,6 +593,7 @@ static u32 acpi_ec_gpe_handler(void *data) > " switching to interrupt mode\n"); > set_bit(EC_FLAGS_GPE_MODE, &ec->flags); > } > +#endif > return ACPI_INTERRUPT_HANDLED; > } > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
