diff --git a/src/add-ons/kernel/drivers/power/acpi_button/acpi_button.c b/src/add-ons/kernel/drivers/power/acpi_button/acpi_button.c index 17414e90ea..95f6585031 100644 --- a/src/add-ons/kernel/drivers/power/acpi_button/acpi_button.c +++ b/src/add-ons/kernel/drivers/power/acpi_button/acpi_button.c @@ -61,19 +61,10 @@ acpi_button_read (void* cookie, off_t position, void *buf, size_t* num_bytes) *((uint8 *)(buf)) = acpi->read_acpi_reg((uint32)(cookie)); - /* acpi->write_acpi_reg((uint32)(cookie),0); */ + acpi->write_acpi_reg((uint32)(cookie),1); - /* The above simple clearing of the status register doesn't seem to work. - So we toggle the ENABLE bit instead. Hopefully this doesn't interfere - with the debounce logic and result in two events */ - - if ((uint32)(cookie) == ACPI_BITREG_POWER_BUTTON_STATUS) { - acpi->write_acpi_reg(ACPI_BITREG_POWER_BUTTON_ENABLE,1); - acpi->write_acpi_reg(ACPI_BITREG_POWER_BUTTON_ENABLE,0); - } else if ((uint32)(cookie) == ACPI_BITREG_SLEEP_BUTTON_STATUS) { - acpi->write_acpi_reg(ACPI_BITREG_SLEEP_BUTTON_ENABLE,1); - acpi->write_acpi_reg(ACPI_BITREG_SLEEP_BUTTON_ENABLE,0); - } + /* You need to write 1 to the status register to clear it to 0. + No, I don't understand Intel either. */ *num_bytes = 1; return B_OK;