diff options
author | Lin Ming <ming.m.lin@intel.com> | 2008-08-07 23:57:11 -0400 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2008-10-22 17:56:40 -0400 |
commit | 08237974af22a97da59869979ef1a515524d5cc3 (patch) | |
tree | 1d0ac70ff4fa3bcb6852020fe24c554ae79e2cf5 /drivers/misc/acer-wmi.c | |
parent | 36eec5e29cf31011a7fbe121badfed7efb2a4ebb (diff) |
ACPI: replace AE_BAD_ADDRESS exception code with AE_ERROR
The AE_BAD_ADDRESS exception code is now unused in ACPICA.
For linux, it's only used at wmi.c and acer-wmi.c.
I checked both wmi.c and acer-wmi.c, the AE_BAD_ADDRESS exception code
has no special meaning. The parent functions just call AE_SUCCESS() or
AE_FAILURE() to check the return status.
So it's safe to replace AE_BAD_ADDRESS with AE_ERROR.
Signed-off-by Lin Ming <ming.m.lin@intel.com>
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/misc/acer-wmi.c')
-rw-r--r-- | drivers/misc/acer-wmi.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/misc/acer-wmi.c b/drivers/misc/acer-wmi.c index d8b0d326e452..a67aef5e284c 100644 --- a/drivers/misc/acer-wmi.c +++ b/drivers/misc/acer-wmi.c | |||
@@ -476,7 +476,7 @@ struct wmi_interface *iface) | |||
476 | } | 476 | } |
477 | break; | 477 | break; |
478 | default: | 478 | default: |
479 | return AE_BAD_ADDRESS; | 479 | return AE_ERROR; |
480 | } | 480 | } |
481 | return AE_OK; | 481 | return AE_OK; |
482 | } | 482 | } |
@@ -514,7 +514,7 @@ static acpi_status AMW0_set_u32(u32 value, u32 cap, struct wmi_interface *iface) | |||
514 | break; | 514 | break; |
515 | } | 515 | } |
516 | default: | 516 | default: |
517 | return AE_BAD_ADDRESS; | 517 | return AE_ERROR; |
518 | } | 518 | } |
519 | 519 | ||
520 | /* Actually do the set */ | 520 | /* Actually do the set */ |
@@ -689,7 +689,7 @@ struct wmi_interface *iface) | |||
689 | return 0; | 689 | return 0; |
690 | } | 690 | } |
691 | default: | 691 | default: |
692 | return AE_BAD_ADDRESS; | 692 | return AE_ERROR; |
693 | } | 693 | } |
694 | status = WMI_execute_u32(method_id, 0, &result); | 694 | status = WMI_execute_u32(method_id, 0, &result); |
695 | 695 | ||
@@ -735,7 +735,7 @@ static acpi_status WMID_set_u32(u32 value, u32 cap, struct wmi_interface *iface) | |||
735 | } | 735 | } |
736 | break; | 736 | break; |
737 | default: | 737 | default: |
738 | return AE_BAD_ADDRESS; | 738 | return AE_ERROR; |
739 | } | 739 | } |
740 | return WMI_execute_u32(method_id, (u32)value, NULL); | 740 | return WMI_execute_u32(method_id, (u32)value, NULL); |
741 | } | 741 | } |
@@ -785,7 +785,7 @@ static struct wmi_interface wmid_interface = { | |||
785 | 785 | ||
786 | static acpi_status get_u32(u32 *value, u32 cap) | 786 | static acpi_status get_u32(u32 *value, u32 cap) |
787 | { | 787 | { |
788 | acpi_status status = AE_BAD_ADDRESS; | 788 | acpi_status status = AE_ERROR; |
789 | 789 | ||
790 | switch (interface->type) { | 790 | switch (interface->type) { |
791 | case ACER_AMW0: | 791 | case ACER_AMW0: |