diff options
author | Chao Guan <chao.guan@intel.com> | 2013-06-07 20:58:14 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-06-15 18:55:05 -0400 |
commit | 1d1ea1b723d9f239f736b8cf284327cbbf9d15d1 (patch) | |
tree | a94241a1f42a8952415d68e623f58de6d43c6369 /drivers/acpi/acpica/exoparg1.c | |
parent | b6872ff9a4785a790f9647ee2076e7e616a3bb0e (diff) |
ACPICA: Standardize all switch() blocks
After many years, different formatting for switch() has crept in.
This change makes every switch block identical. Chao Guan.
ACPICA bugzilla 997.
References: https://bugs.acpica.org/show_bug.cgi?id=997
Signed-off-by: Chao Guan <chao.guan@intel.com>
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Lv Zheng <lv.zheng@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/acpi/acpica/exoparg1.c')
-rw-r--r-- | drivers/acpi/acpica/exoparg1.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/drivers/acpi/acpica/exoparg1.c b/drivers/acpi/acpica/exoparg1.c index b60c877f5906..814b4a3d656a 100644 --- a/drivers/acpi/acpica/exoparg1.c +++ b/drivers/acpi/acpica/exoparg1.c | |||
@@ -327,7 +327,6 @@ acpi_status acpi_ex_opcode_1A_1T_1R(struct acpi_walk_state *walk_state) | |||
327 | break; | 327 | break; |
328 | 328 | ||
329 | case AML_FROM_BCD_OP: /* from_bcd (BCDValue, Result) */ | 329 | case AML_FROM_BCD_OP: /* from_bcd (BCDValue, Result) */ |
330 | |||
331 | /* | 330 | /* |
332 | * The 64-bit ACPI integer can hold 16 4-bit BCD characters | 331 | * The 64-bit ACPI integer can hold 16 4-bit BCD characters |
333 | * (if table is 32-bit, integer can hold 8 BCD characters) | 332 | * (if table is 32-bit, integer can hold 8 BCD characters) |
@@ -407,7 +406,6 @@ acpi_status acpi_ex_opcode_1A_1T_1R(struct acpi_walk_state *walk_state) | |||
407 | break; | 406 | break; |
408 | 407 | ||
409 | case AML_COND_REF_OF_OP: /* cond_ref_of (source_object, Result) */ | 408 | case AML_COND_REF_OF_OP: /* cond_ref_of (source_object, Result) */ |
410 | |||
411 | /* | 409 | /* |
412 | * This op is a little strange because the internal return value is | 410 | * This op is a little strange because the internal return value is |
413 | * different than the return value stored in the result descriptor | 411 | * different than the return value stored in the result descriptor |
@@ -442,13 +440,14 @@ acpi_status acpi_ex_opcode_1A_1T_1R(struct acpi_walk_state *walk_state) | |||
442 | goto cleanup; | 440 | goto cleanup; |
443 | 441 | ||
444 | default: | 442 | default: |
443 | |||
445 | /* No other opcodes get here */ | 444 | /* No other opcodes get here */ |
445 | |||
446 | break; | 446 | break; |
447 | } | 447 | } |
448 | break; | 448 | break; |
449 | 449 | ||
450 | case AML_STORE_OP: /* Store (Source, Target) */ | 450 | case AML_STORE_OP: /* Store (Source, Target) */ |
451 | |||
452 | /* | 451 | /* |
453 | * A store operand is typically a number, string, buffer or lvalue | 452 | * A store operand is typically a number, string, buffer or lvalue |
454 | * Be careful about deleting the source object, | 453 | * Be careful about deleting the source object, |
@@ -615,7 +614,6 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state) | |||
615 | 614 | ||
616 | case AML_DECREMENT_OP: /* Decrement (Operand) */ | 615 | case AML_DECREMENT_OP: /* Decrement (Operand) */ |
617 | case AML_INCREMENT_OP: /* Increment (Operand) */ | 616 | case AML_INCREMENT_OP: /* Increment (Operand) */ |
618 | |||
619 | /* | 617 | /* |
620 | * Create a new integer. Can't just get the base integer and | 618 | * Create a new integer. Can't just get the base integer and |
621 | * increment it because it may be an Arg or Field. | 619 | * increment it because it may be an Arg or Field. |
@@ -682,7 +680,6 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state) | |||
682 | break; | 680 | break; |
683 | 681 | ||
684 | case AML_TYPE_OP: /* object_type (source_object) */ | 682 | case AML_TYPE_OP: /* object_type (source_object) */ |
685 | |||
686 | /* | 683 | /* |
687 | * Note: The operand is not resolved at this point because we want to | 684 | * Note: The operand is not resolved at this point because we want to |
688 | * get the associated object, not its value. For example, we don't | 685 | * get the associated object, not its value. For example, we don't |
@@ -709,7 +706,6 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state) | |||
709 | break; | 706 | break; |
710 | 707 | ||
711 | case AML_SIZE_OF_OP: /* size_of (source_object) */ | 708 | case AML_SIZE_OF_OP: /* size_of (source_object) */ |
712 | |||
713 | /* | 709 | /* |
714 | * Note: The operand is not resolved at this point because we want to | 710 | * Note: The operand is not resolved at this point because we want to |
715 | * get the associated object, not its value. | 711 | * get the associated object, not its value. |
@@ -735,10 +731,12 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state) | |||
735 | */ | 731 | */ |
736 | switch (type) { | 732 | switch (type) { |
737 | case ACPI_TYPE_INTEGER: | 733 | case ACPI_TYPE_INTEGER: |
734 | |||
738 | value = acpi_gbl_integer_byte_width; | 735 | value = acpi_gbl_integer_byte_width; |
739 | break; | 736 | break; |
740 | 737 | ||
741 | case ACPI_TYPE_STRING: | 738 | case ACPI_TYPE_STRING: |
739 | |||
742 | value = temp_desc->string.length; | 740 | value = temp_desc->string.length; |
743 | break; | 741 | break; |
744 | 742 | ||
@@ -759,6 +757,7 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state) | |||
759 | break; | 757 | break; |
760 | 758 | ||
761 | default: | 759 | default: |
760 | |||
762 | ACPI_ERROR((AE_INFO, | 761 | ACPI_ERROR((AE_INFO, |
763 | "Operand must be Buffer/Integer/String/Package - found type %s", | 762 | "Operand must be Buffer/Integer/String/Package - found type %s", |
764 | acpi_ut_get_type_name(type))); | 763 | acpi_ut_get_type_name(type))); |
@@ -860,9 +859,11 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state) | |||
860 | break; | 859 | break; |
861 | 860 | ||
862 | case ACPI_TYPE_STRING: | 861 | case ACPI_TYPE_STRING: |
862 | |||
863 | break; | 863 | break; |
864 | 864 | ||
865 | default: | 865 | default: |
866 | |||
866 | status = AE_AML_OPERAND_TYPE; | 867 | status = AE_AML_OPERAND_TYPE; |
867 | goto cleanup; | 868 | goto cleanup; |
868 | } | 869 | } |
@@ -923,7 +924,6 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state) | |||
923 | */ | 924 | */ |
924 | switch (operand[0]->reference.class) { | 925 | switch (operand[0]->reference.class) { |
925 | case ACPI_REFCLASS_INDEX: | 926 | case ACPI_REFCLASS_INDEX: |
926 | |||
927 | /* | 927 | /* |
928 | * The target type for the Index operator must be | 928 | * The target type for the Index operator must be |
929 | * either a Buffer or a Package | 929 | * either a Buffer or a Package |
@@ -956,7 +956,6 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state) | |||
956 | break; | 956 | break; |
957 | 957 | ||
958 | case ACPI_TYPE_PACKAGE: | 958 | case ACPI_TYPE_PACKAGE: |
959 | |||
960 | /* | 959 | /* |
961 | * Return the referenced element of the package. We must | 960 | * Return the referenced element of the package. We must |
962 | * add another reference to the referenced object, however. | 961 | * add another reference to the referenced object, however. |
@@ -999,6 +998,7 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state) | |||
999 | break; | 998 | break; |
1000 | 999 | ||
1001 | default: | 1000 | default: |
1001 | |||
1002 | ACPI_ERROR((AE_INFO, | 1002 | ACPI_ERROR((AE_INFO, |
1003 | "Unknown class in reference(%p) - 0x%2.2X", | 1003 | "Unknown class in reference(%p) - 0x%2.2X", |
1004 | operand[0], | 1004 | operand[0], |