diff options
author | Rafael J. Wysocki <rjw@sisk.pl> | 2010-07-07 18:44:42 -0400 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2010-07-12 14:33:35 -0400 |
commit | a0d468718b9049f7396d101075a129a2d683ad66 (patch) | |
tree | a5003982bda335cf8836e13054673076df6b72c5 | |
parent | 9ce10df8d83d0528e80cd319b35ac5f6812b4f62 (diff) |
ACPI / ACPICA: Simplify acpi_ev_initialize_gpe_block()
Simplify the main loop in acpi_ev_initialize_gpe_block() by
rearranging code and removing the "enabled" label that is not
necessary any more.
Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
Signed-off-by: Len Brown <len.brown@intel.com>
-rw-r--r-- | drivers/acpi/acpica/evgpeblk.c | 21 |
1 files changed, 8 insertions, 13 deletions
diff --git a/drivers/acpi/acpica/evgpeblk.c b/drivers/acpi/acpica/evgpeblk.c index 12ca1bc5f1fd..85445fb5844e 100644 --- a/drivers/acpi/acpica/evgpeblk.c +++ b/drivers/acpi/acpica/evgpeblk.c | |||
@@ -472,26 +472,21 @@ acpi_ev_initialize_gpe_block(struct acpi_namespace_node *gpe_device, | |||
472 | gpe_event_info = &gpe_block->event_info[gpe_index]; | 472 | gpe_event_info = &gpe_block->event_info[gpe_index]; |
473 | gpe_number = gpe_index + gpe_block->block_base_number; | 473 | gpe_number = gpe_index + gpe_block->block_base_number; |
474 | 474 | ||
475 | /* | ||
476 | * If the GPE has already been enabled for runtime | ||
477 | * signaling, make sure it remains enabled, but do not | ||
478 | * increment its reference counter. | ||
479 | */ | ||
480 | if (gpe_event_info->runtime_count) { | ||
481 | status = acpi_ev_enable_gpe(gpe_event_info); | ||
482 | goto enabled; | ||
483 | } | ||
484 | |||
485 | /* Ignore GPEs that have no corresponding _Lxx/_Exx method */ | 475 | /* Ignore GPEs that have no corresponding _Lxx/_Exx method */ |
486 | 476 | ||
487 | if (!(gpe_event_info->flags & ACPI_GPE_DISPATCH_METHOD)) { | 477 | if (!(gpe_event_info->flags & ACPI_GPE_DISPATCH_METHOD)) { |
488 | continue; | 478 | continue; |
489 | } | 479 | } |
490 | 480 | ||
491 | /* Enable this GPE */ | 481 | /* |
482 | * If the GPE has already been enabled for runtime | ||
483 | * signaling, make sure it remains enabled, but do not | ||
484 | * increment its reference counter. | ||
485 | */ | ||
486 | status = gpe_event_info->runtime_count ? | ||
487 | acpi_ev_enable_gpe(gpe_event_info) : | ||
488 | acpi_enable_gpe(gpe_device, gpe_number); | ||
492 | 489 | ||
493 | status = acpi_enable_gpe(gpe_device, gpe_number); | ||
494 | enabled: | ||
495 | if (ACPI_FAILURE(status)) { | 490 | if (ACPI_FAILURE(status)) { |
496 | ACPI_EXCEPTION((AE_INFO, status, | 491 | ACPI_EXCEPTION((AE_INFO, status, |
497 | "Could not enable GPE 0x%02X", | 492 | "Could not enable GPE 0x%02X", |