ACPICA: Standardize all switch() blocks
authorChao Guan <chao.guan@intel.com>
Sat, 8 Jun 2013 00:58:14 +0000 (00:58 +0000)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Sat, 15 Jun 2013 22:55:05 +0000 (00:55 +0200)
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>
71 files changed:
drivers/acpi/acpica/dscontrol.c
drivers/acpi/acpica/dsfield.c
drivers/acpi/acpica/dsinit.c
drivers/acpi/acpica/dsmthdat.c
drivers/acpi/acpica/dsobject.c
drivers/acpi/acpica/dsopcode.c
drivers/acpi/acpica/dsutils.c
drivers/acpi/acpica/dswexec.c
drivers/acpi/acpica/dswload.c
drivers/acpi/acpica/dswload2.c
drivers/acpi/acpica/evgpe.c
drivers/acpi/acpica/evgpeinit.c
drivers/acpi/acpica/evhandler.c
drivers/acpi/acpica/evmisc.c
drivers/acpi/acpica/evrgnini.c
drivers/acpi/acpica/evxfgpe.c
drivers/acpi/acpica/evxfregn.c
drivers/acpi/acpica/exconfig.c
drivers/acpi/acpica/exconvrt.c
drivers/acpi/acpica/excreate.c
drivers/acpi/acpica/exdebug.c
drivers/acpi/acpica/exdump.c
drivers/acpi/acpica/exfield.c
drivers/acpi/acpica/exfldio.c
drivers/acpi/acpica/exmisc.c
drivers/acpi/acpica/exoparg1.c
drivers/acpi/acpica/exoparg2.c
drivers/acpi/acpica/exoparg3.c
drivers/acpi/acpica/exoparg6.c
drivers/acpi/acpica/exprep.c
drivers/acpi/acpica/exregion.c
drivers/acpi/acpica/exresnte.c
drivers/acpi/acpica/exresolv.c
drivers/acpi/acpica/exresop.c
drivers/acpi/acpica/exstore.c
drivers/acpi/acpica/exstoren.c
drivers/acpi/acpica/hwacpi.c
drivers/acpi/acpica/hwgpe.c
drivers/acpi/acpica/hwregs.c
drivers/acpi/acpica/hwxface.c
drivers/acpi/acpica/hwxfsleep.c
drivers/acpi/acpica/nsaccess.c
drivers/acpi/acpica/nsconvert.c
drivers/acpi/acpica/nsdump.c
drivers/acpi/acpica/nsinit.c
drivers/acpi/acpica/nspredef.c
drivers/acpi/acpica/nsprepkg.c
drivers/acpi/acpica/nsrepair.c
drivers/acpi/acpica/nsrepair2.c
drivers/acpi/acpica/nsutils.c
drivers/acpi/acpica/nsxfeval.c
drivers/acpi/acpica/psargs.c
drivers/acpi/acpica/psloop.c
drivers/acpi/acpica/psobject.c
drivers/acpi/acpica/psparse.c
drivers/acpi/acpica/pstree.c
drivers/acpi/acpica/rscalc.c
drivers/acpi/acpica/rsdump.c
drivers/acpi/acpica/rsmisc.c
drivers/acpi/acpica/rsutils.c
drivers/acpi/acpica/rsxface.c
drivers/acpi/acpica/tbinstal.c
drivers/acpi/acpica/utcopy.c
drivers/acpi/acpica/utdelete.c
drivers/acpi/acpica/uteval.c
drivers/acpi/acpica/utexcep.c
drivers/acpi/acpica/utids.c
drivers/acpi/acpica/utmisc.c
drivers/acpi/acpica/utobject.c
drivers/acpi/acpica/utstring.c
drivers/acpi/acpica/uttrack.c

index 7ea0f162f11ca42e11d7247fc3263d4c1fb0b2e1..eb56b66444b5e3d671572b6290a1245473e7d165 100644 (file)
@@ -78,7 +78,6 @@ acpi_ds_exec_begin_control_op(struct acpi_walk_state *walk_state,
 
        switch (op->common.aml_opcode) {
        case AML_WHILE_OP:
-
                /*
                 * If this is an additional iteration of a while loop, continue.
                 * There is no need to allocate a new control state.
@@ -99,7 +98,6 @@ acpi_ds_exec_begin_control_op(struct acpi_walk_state *walk_state,
                /*lint -fallthrough */
 
        case AML_IF_OP:
-
                /*
                 * IF/WHILE: Create a new control state to manage these
                 * constructs. We need to manage these as a stack, in order
@@ -142,6 +140,7 @@ acpi_ds_exec_begin_control_op(struct acpi_walk_state *walk_state,
                break;
 
        default:
+
                break;
        }
 
@@ -344,6 +343,7 @@ acpi_ds_exec_end_control_op(struct acpi_walk_state * walk_state,
        case AML_NOOP_OP:
 
                /* Just do nothing! */
+
                break;
 
        case AML_BREAK_POINT_OP:
index feadeed1012dc2256c1596bb79e7ff6190e6cb41..d4bfe7b7f90a54a5b5cdafb8119f0d60698be171 100644 (file)
@@ -563,21 +563,25 @@ acpi_ds_init_field_objects(union acpi_parse_object *op,
         */
        switch (walk_state->opcode) {
        case AML_FIELD_OP:
+
                arg = acpi_ps_get_arg(op, 2);
                type = ACPI_TYPE_LOCAL_REGION_FIELD;
                break;
 
        case AML_BANK_FIELD_OP:
+
                arg = acpi_ps_get_arg(op, 4);
                type = ACPI_TYPE_LOCAL_BANK_FIELD;
                break;
 
        case AML_INDEX_FIELD_OP:
+
                arg = acpi_ps_get_arg(op, 3);
                type = ACPI_TYPE_LOCAL_INDEX_FIELD;
                break;
 
        default:
+
                return_ACPI_STATUS(AE_BAD_PARAMETER);
        }
 
index bc8e63f7784b96a7bf39c5476dc9cacd07688bb5..14424200d246c8538bcde5070072100528dcf1dd 100644 (file)
@@ -127,6 +127,7 @@ acpi_ds_init_one_object(acpi_handle obj_handle,
                break;
 
        default:
+
                break;
        }
 
index 3da80460ce38af069241fb59ee41d2aeb695d319..c4b0b365723758fe5181e26eaeaa7cb5314f18d3 100644 (file)
@@ -285,6 +285,7 @@ acpi_ds_method_data_get_node(u8 type,
                break;
 
        default:
+
                ACPI_ERROR((AE_INFO, "Type %u is invalid", type));
                return_ACPI_STATUS(AE_TYPE);
        }
@@ -428,7 +429,6 @@ acpi_ds_method_data_get_value(u8 type,
                                return_ACPI_STATUS(AE_AML_UNINITIALIZED_ARG);
 
                        case ACPI_REFCLASS_LOCAL:
-
                                /*
                                 * No error message for this case, will be trapped again later to
                                 * detect and ignore cases of Store(local_x,local_x)
index e20e9f84eee80653b6dfc932f140c6eaebf4a5ac..63f0d220ca3d99a2da601b1b4571972bf2f71397 100644 (file)
@@ -648,7 +648,6 @@ acpi_ds_init_object_from_op(struct acpi_walk_state *walk_state,
 
        switch (obj_desc->common.type) {
        case ACPI_TYPE_BUFFER:
-
                /*
                 * Defer evaluation of Buffer term_arg operand
                 */
@@ -660,7 +659,6 @@ acpi_ds_init_object_from_op(struct acpi_walk_state *walk_state,
                break;
 
        case ACPI_TYPE_PACKAGE:
-
                /*
                 * Defer evaluation of Package term_arg operand
                 */
@@ -741,6 +739,7 @@ acpi_ds_init_object_from_op(struct acpi_walk_state *walk_state,
                        break;
 
                default:
+
                        ACPI_ERROR((AE_INFO, "Unknown Integer type 0x%X",
                                    op_info->type));
                        status = AE_AML_OPERAND_TYPE;
index ee6367b8eaf7afcf6866ff75193e5ca59bfa94f4..1fc1ff114f269867bf1a281e584511c00595d960 100644 (file)
@@ -636,6 +636,7 @@ acpi_ds_eval_data_object_operands(struct acpi_walk_state *walk_state,
                break;
 
        default:
+
                return_ACPI_STATUS(AE_AML_BAD_OPCODE);
        }
 
index 99778997c35ab8f1fb5c6249c030c9e48fd35366..c666fc01498795afdb76f6afad8383f75389d1bb 100644 (file)
@@ -240,7 +240,6 @@ acpi_ds_is_result_used(union acpi_parse_object * op,
 
                case AML_IF_OP:
                case AML_WHILE_OP:
-
                        /*
                         * If we are executing the predicate AND this is the predicate op,
                         * we will use the return value
@@ -254,7 +253,9 @@ acpi_ds_is_result_used(union acpi_parse_object * op,
                        break;
 
                default:
+
                        /* Ignore other control opcodes */
+
                        break;
                }
 
@@ -263,7 +264,6 @@ acpi_ds_is_result_used(union acpi_parse_object * op,
                goto result_not_used;
 
        case AML_CLASS_CREATE:
-
                /*
                 * These opcodes allow term_arg(s) as operands and therefore
                 * the operands can be method calls. The result is used.
@@ -292,7 +292,6 @@ acpi_ds_is_result_used(union acpi_parse_object * op,
                goto result_not_used;
 
        default:
-
                /*
                 * In all other cases. the parent will actually use the return
                 * object, so keep it.
index e2199a9474703196326024af6bbfc0c5460e7201..151d924817e1b2041bfd60cad46aaaf1a5dc9e20 100644 (file)
@@ -327,6 +327,7 @@ acpi_ds_exec_begin_op(struct acpi_walk_state *walk_state,
                break;
 
        default:
+
                break;
        }
 
@@ -488,7 +489,6 @@ acpi_status acpi_ds_exec_end_op(struct acpi_walk_state *walk_state)
                        break;
 
                case AML_TYPE_METHOD_CALL:
-
                        /*
                         * If the method is referenced from within a package
                         * declaration, it is not a invocation of the method, just
@@ -582,7 +582,6 @@ acpi_status acpi_ds_exec_end_op(struct acpi_walk_state *walk_state)
 
                        switch (op->common.parent->common.aml_opcode) {
                        case AML_NAME_OP:
-
                                /*
                                 * Put the Node on the object stack (Contains the ACPI Name
                                 * of this object)
index 6e17c0e24e63ca4e758d1e2a2147419ed5e4b0ec..95e681a36f9c28d820b40f55d211426ae4b2de92 100644 (file)
@@ -74,6 +74,7 @@ acpi_ds_init_callbacks(struct acpi_walk_state *walk_state, u32 pass_number)
 
        switch (pass_number) {
        case 1:
+
                walk_state->parse_flags = ACPI_PARSE_LOAD_PASS1 |
                    ACPI_PARSE_DELETE_TREE;
                walk_state->descending_callback = acpi_ds_load1_begin_op;
@@ -81,6 +82,7 @@ acpi_ds_init_callbacks(struct acpi_walk_state *walk_state, u32 pass_number)
                break;
 
        case 2:
+
                walk_state->parse_flags = ACPI_PARSE_LOAD_PASS1 |
                    ACPI_PARSE_DELETE_TREE;
                walk_state->descending_callback = acpi_ds_load2_begin_op;
@@ -88,6 +90,7 @@ acpi_ds_init_callbacks(struct acpi_walk_state *walk_state, u32 pass_number)
                break;
 
        case 3:
+
 #ifndef ACPI_NO_METHOD_EXECUTION
                walk_state->parse_flags |= ACPI_PARSE_EXECUTE |
                    ACPI_PARSE_DELETE_TREE;
@@ -97,6 +100,7 @@ acpi_ds_init_callbacks(struct acpi_walk_state *walk_state, u32 pass_number)
                break;
 
        default:
+
                return (AE_BAD_PARAMETER);
        }
 
@@ -161,7 +165,6 @@ acpi_ds_load1_begin_op(struct acpi_walk_state * walk_state,
 
        switch (walk_state->opcode) {
        case AML_SCOPE_OP:
-
                /*
                 * The target name of the Scope() operator must exist at this point so
                 * that we can actually open the scope to enter new names underneath it.
@@ -210,7 +213,6 @@ acpi_ds_load1_begin_op(struct acpi_walk_state * walk_state,
                case ACPI_TYPE_INTEGER:
                case ACPI_TYPE_STRING:
                case ACPI_TYPE_BUFFER:
-
                        /*
                         * These types we will allow, but we will change the type.
                         * This enables some existing code of the form:
@@ -232,7 +234,6 @@ acpi_ds_load1_begin_op(struct acpi_walk_state * walk_state,
                        break;
 
                case ACPI_TYPE_METHOD:
-
                        /*
                         * Allow scope change to root during execution of module-level
                         * code. Root is typed METHOD during this time.
index 4407ff2377d5918b10fe08875434ed07b981d836..b1f8f4725c230b0d6b0bf9155f13028f42d62667 100644 (file)
@@ -509,6 +509,7 @@ acpi_status acpi_ds_load2_end_op(struct acpi_walk_state *walk_state)
                        break;
 
                default:
+
                        /* All NAMED_FIELD opcodes must be handled above */
                        break;
                }
@@ -548,6 +549,7 @@ acpi_status acpi_ds_load2_end_op(struct acpi_walk_state *walk_state)
                        break;
 
                default:
+
                        /* Unknown opcode */
 
                        status = AE_OK;
@@ -674,6 +676,7 @@ acpi_status acpi_ds_load2_end_op(struct acpi_walk_state *walk_state)
 #endif                         /* ACPI_NO_METHOD_EXECUTION */
 
                default:
+
                        /* All NAMED_COMPLEX opcodes must be handled above */
                        break;
                }
@@ -721,6 +724,7 @@ acpi_status acpi_ds_load2_end_op(struct acpi_walk_state *walk_state)
                break;
 
        default:
+
                break;
        }
 
index ae50d6cc535fde4ed09ea0b10233186b6c8dc5bb..c8a1f7d5931f518b004cc8311d31e5ad8e5cdaeb 100644 (file)
@@ -529,7 +529,6 @@ static void ACPI_SYSTEM_XFACE acpi_ev_asynch_execute_gpe_method(void *context)
 
        switch (local_gpe_event_info->flags & ACPI_GPE_DISPATCH_MASK) {
        case ACPI_GPE_DISPATCH_NOTIFY:
-
                /*
                 * Implicit notify.
                 * Dispatch a DEVICE_WAKE notify to the appropriate handler.
@@ -582,7 +581,8 @@ static void ACPI_SYSTEM_XFACE acpi_ev_asynch_execute_gpe_method(void *context)
                break;
 
        default:
-               return_VOID;    /* Should never happen */
+
+               return_VOID;    /* Should never happen */
        }
 
        /* Defer enabling of GPE until all notify handlers are done */
@@ -754,7 +754,6 @@ acpi_ev_gpe_dispatch(struct acpi_namespace_node *gpe_device,
 
        case ACPI_GPE_DISPATCH_METHOD:
        case ACPI_GPE_DISPATCH_NOTIFY:
-
                /*
                 * Execute the method associated with the GPE
                 * NOTE: Level-triggered GPEs are cleared after the method completes.
@@ -770,7 +769,6 @@ acpi_ev_gpe_dispatch(struct acpi_namespace_node *gpe_device,
                break;
 
        default:
-
                /*
                 * No handler or method to run!
                 * 03/2010: This case should no longer be possible. We will not allow
index 72b8f6b3f4cafeac10eee0bacfa2af5c1851f281..9037f17c9608f70a6bc2df68eca5b55043ad9723 100644 (file)
@@ -363,14 +363,17 @@ acpi_ev_match_gpe_method(acpi_handle obj_handle,
         */
        switch (name[1]) {
        case 'L':
+
                type = ACPI_GPE_LEVEL_TRIGGERED;
                break;
 
        case 'E':
+
                type = ACPI_GPE_EDGE_TRIGGERED;
                break;
 
        default:
+
                /* Unknown method type, just ignore it */
 
                ACPI_DEBUG_PRINT((ACPI_DB_LOAD,
index d4f83112c2e266ff2fcf5f8415d9714ffa071330..068af96134b89787a739cce281f481db82e46e19 100644 (file)
@@ -354,36 +354,43 @@ acpi_ev_install_space_handler(struct acpi_namespace_node * node,
 
                switch (space_id) {
                case ACPI_ADR_SPACE_SYSTEM_MEMORY:
+
                        handler = acpi_ex_system_memory_space_handler;
                        setup = acpi_ev_system_memory_region_setup;
                        break;
 
                case ACPI_ADR_SPACE_SYSTEM_IO:
+
                        handler = acpi_ex_system_io_space_handler;
                        setup = acpi_ev_io_space_region_setup;
                        break;
 
                case ACPI_ADR_SPACE_PCI_CONFIG:
+
                        handler = acpi_ex_pci_config_space_handler;
                        setup = acpi_ev_pci_config_region_setup;
                        break;
 
                case ACPI_ADR_SPACE_CMOS:
+
                        handler = acpi_ex_cmos_space_handler;
                        setup = acpi_ev_cmos_region_setup;
                        break;
 
                case ACPI_ADR_SPACE_PCI_BAR_TARGET:
+
                        handler = acpi_ex_pci_bar_space_handler;
                        setup = acpi_ev_pci_bar_region_setup;
                        break;
 
                case ACPI_ADR_SPACE_DATA_TABLE:
+
                        handler = acpi_ex_data_table_space_handler;
                        setup = NULL;
                        break;
 
                default:
+
                        status = AE_BAD_PARAMETER;
                        goto unlock_and_exit;
                }
index c986b2336b81d6b1b85e60ee4e2e9568e2750a33..db59fe3f63919220faa6818641a14082c1c91793 100644 (file)
@@ -78,6 +78,7 @@ u8 acpi_ev_is_notify_object(struct acpi_namespace_node *node)
                return (TRUE);
 
        default:
+
                return (FALSE);
        }
 }
index 3bb616794b3b46fe5aaaaa7b77335523b351b159..8354c4f7f10c55c604765a92020abd2d30547de9 100644 (file)
@@ -596,7 +596,9 @@ acpi_ev_initialize_region(union acpi_operand_object *region_obj,
                                break;
 
                        default:
+
                                /* Ignore other objects */
+
                                break;
                        }
 
index aff4cc2612115304f41ea11a6aa57a5038a6f101..7662f1a42ff6bb19513e0f9272f90c921f64ec90 100644 (file)
@@ -366,16 +366,19 @@ acpi_set_gpe_wake_mask(acpi_handle gpe_device, u32 gpe_number, u8 action)
 
        switch (action) {
        case ACPI_GPE_ENABLE:
+
                ACPI_SET_BIT(gpe_register_info->enable_for_wake,
                             (u8)register_bit);
                break;
 
        case ACPI_GPE_DISABLE:
+
                ACPI_CLEAR_BIT(gpe_register_info->enable_for_wake,
                               (u8)register_bit);
                break;
 
        default:
+
                ACPI_ERROR((AE_INFO, "%u, Invalid action", action));
                status = AE_BAD_PARAMETER;
                break;
index 96c9e5f355aed40f7e07fc598d60b61907c501a6..80cecf8385917c7ed32ca75eb9afc807dbb6b0b2 100644 (file)
@@ -139,6 +139,7 @@ acpi_install_address_space_handler(acpi_handle device,
                break;
 
        default:
+
                break;
        }
 
index a689aacae2bab65b2ce3d65788a9dd7a46aa504e..06d216c8d43ab18edc0fb1f66def077014b2aa3f 100644 (file)
@@ -480,6 +480,7 @@ acpi_ex_load_op(union acpi_operand_object *obj_desc,
                break;
 
        default:
+
                return_ACPI_STATUS(AE_AML_OPERAND_TYPE);
        }
 
index d2b9613bbf01d231e72f44d87381e83ab87f11e4..69e4a8cc9b71728f98b1aad402a0a44538033457 100644 (file)
@@ -99,6 +99,7 @@ acpi_ex_convert_to_integer(union acpi_operand_object *obj_desc,
                break;
 
        default:
+
                return_ACPI_STATUS(AE_TYPE);
        }
 
@@ -117,7 +118,6 @@ acpi_ex_convert_to_integer(union acpi_operand_object *obj_desc,
 
        switch (obj_desc->common.type) {
        case ACPI_TYPE_STRING:
-
                /*
                 * Convert string to an integer - for most cases, the string must be
                 * hexadecimal as per the ACPI specification. The only exception (as
@@ -161,6 +161,7 @@ acpi_ex_convert_to_integer(union acpi_operand_object *obj_desc,
        default:
 
                /* No other types can get here */
+
                break;
        }
 
@@ -213,7 +214,6 @@ acpi_ex_convert_to_buffer(union acpi_operand_object *obj_desc,
                return_ACPI_STATUS(AE_OK);
 
        case ACPI_TYPE_INTEGER:
-
                /*
                 * Create a new Buffer object.
                 * Need enough space for one integer
@@ -233,7 +233,6 @@ acpi_ex_convert_to_buffer(union acpi_operand_object *obj_desc,
                break;
 
        case ACPI_TYPE_STRING:
-
                /*
                 * Create a new Buffer object
                 * Size will be the string length
@@ -258,6 +257,7 @@ acpi_ex_convert_to_buffer(union acpi_operand_object *obj_desc,
                break;
 
        default:
+
                return_ACPI_STATUS(AE_TYPE);
        }
 
@@ -304,15 +304,18 @@ acpi_ex_convert_to_ascii(u64 integer, u16 base, u8 *string, u8 data_width)
 
                switch (data_width) {
                case 1:
+
                        decimal_length = ACPI_MAX8_DECIMAL_DIGITS;
                        break;
 
                case 4:
+
                        decimal_length = ACPI_MAX32_DECIMAL_DIGITS;
                        break;
 
                case 8:
                default:
+
                        decimal_length = ACPI_MAX64_DECIMAL_DIGITS;
                        break;
                }
@@ -546,6 +549,7 @@ acpi_ex_convert_to_string(union acpi_operand_object * obj_desc,
                break;
 
        default:
+
                return_ACPI_STATUS(AE_TYPE);
        }
 
@@ -599,6 +603,7 @@ acpi_ex_convert_to_target_type(acpi_object_type destination_type,
                        break;
 
                default:
+
                        /* No conversion allowed for these types */
 
                        if (destination_type != source_desc->common.type) {
@@ -649,6 +654,7 @@ acpi_ex_convert_to_target_type(acpi_object_type destination_type,
                        break;
 
                default:
+
                        ACPI_ERROR((AE_INFO,
                                    "Bad destination type during conversion: 0x%X",
                                    destination_type));
@@ -664,6 +670,7 @@ acpi_ex_convert_to_target_type(acpi_object_type destination_type,
                break;
 
        default:
+
                ACPI_ERROR((AE_INFO,
                            "Unknown Target type ID 0x%X AmlOpcode 0x%X DestType %s",
                            GET_CURRENT_ARG_TYPE(walk_state->op_info->
index 26a13f67977ea11248cc54ec9f034ecf1a584e45..269e81d86ef41c59e7eaa3d61bb7c63ba23457bd 100644 (file)
@@ -103,7 +103,6 @@ acpi_status acpi_ex_create_alias(struct acpi_walk_state *walk_state)
        case ACPI_TYPE_BUFFER:
        case ACPI_TYPE_PACKAGE:
        case ACPI_TYPE_BUFFER_FIELD:
-
                /*
                 * These types open a new scope, so we need the NS node in order to access
                 * any children.
@@ -113,7 +112,6 @@ acpi_status acpi_ex_create_alias(struct acpi_walk_state *walk_state)
        case ACPI_TYPE_PROCESSOR:
        case ACPI_TYPE_THERMAL:
        case ACPI_TYPE_LOCAL_SCOPE:
-
                /*
                 * The new alias has the type ALIAS and points to the original
                 * NS node, not the object itself.
@@ -124,7 +122,6 @@ acpi_status acpi_ex_create_alias(struct acpi_walk_state *walk_state)
                break;
 
        case ACPI_TYPE_METHOD:
-
                /*
                 * Control method aliases need to be differentiated
                 */
index 7eb853cd279f776f5b384543dba38fdedfafff38..81c72a4ecd823eaca8542135aea5396d7d6d5626 100644 (file)
@@ -193,6 +193,7 @@ acpi_ex_do_debug_object(union acpi_operand_object *source_desc,
                        return_VOID;
 
                default:
+
                        break;
                }
 
@@ -226,6 +227,7 @@ acpi_ex_do_debug_object(union acpi_operand_object *source_desc,
                                        break;
 
                                default:
+
                                        acpi_ex_do_debug_object((source_desc->
                                                                 reference.
                                                                 node)->object,
index e5a3c249f7fad23c544a856504d13d33134c104e..c740f24e3101069af922a5e5842a77fb60d34710 100644 (file)
@@ -357,6 +357,7 @@ acpi_ex_dump_object(union acpi_operand_object *obj_desc,
 
                switch (info->opcode) {
                case ACPI_EXD_INIT:
+
                        break;
 
                case ACPI_EXD_TYPE:
@@ -718,6 +719,7 @@ void acpi_ex_dump_operand(union acpi_operand_object *obj_desc, u32 depth)
                break;
 
        default:
+
                /* Unknown Type */
 
                acpi_os_printf("Unknown Type %X\n", obj_desc->common.type);
index 7d4bae71e8c62da6399bc6af7e0f3fd132efa574..c2a65aaf29af006f3be629aab6b9f6b698650b40 100644 (file)
@@ -331,21 +331,25 @@ acpi_ex_write_data_to_field(union acpi_operand_object *source_desc,
 
        switch (source_desc->common.type) {
        case ACPI_TYPE_INTEGER:
+
                buffer = &source_desc->integer.value;
                length = sizeof(source_desc->integer.value);
                break;
 
        case ACPI_TYPE_BUFFER:
+
                buffer = source_desc->buffer.pointer;
                length = source_desc->buffer.length;
                break;
 
        case ACPI_TYPE_STRING:
+
                buffer = source_desc->string.pointer;
                length = source_desc->string.length;
                break;
 
        default:
+
                return_ACPI_STATUS(AE_AML_OPERAND_TYPE);
        }
 
index c84ee956fa4c58f50c8849cc660a151d46febe39..7e0afe72487ea73ee5504d73891b294b83aceb91 100644 (file)
@@ -446,7 +446,6 @@ acpi_ex_field_datum_io(union acpi_operand_object *obj_desc,
                break;
 
        case ACPI_TYPE_LOCAL_BANK_FIELD:
-
                /*
                 * Ensure that the bank_value is not beyond the capacity of
                 * the register
@@ -488,7 +487,6 @@ acpi_ex_field_datum_io(union acpi_operand_object *obj_desc,
                break;
 
        case ACPI_TYPE_LOCAL_INDEX_FIELD:
-
                /*
                 * Ensure that the index_value is not beyond the capacity of
                 * the register
index 72a2a13b6d36de72eb97cf7d38aacb0cada1587b..00bf298775741763054eabd2d3030dcb6494e150 100644 (file)
@@ -105,7 +105,6 @@ acpi_ex_get_object_reference(union acpi_operand_object *obj_desc,
                break;
 
        case ACPI_DESC_TYPE_NAMED:
-
                /*
                 * A named reference that has already been resolved to a Node
                 */
@@ -261,20 +260,24 @@ acpi_ex_do_concatenate(union acpi_operand_object *operand0,
         */
        switch (operand0->common.type) {
        case ACPI_TYPE_INTEGER:
+
                status =
                    acpi_ex_convert_to_integer(operand1, &local_operand1, 16);
                break;
 
        case ACPI_TYPE_STRING:
+
                status = acpi_ex_convert_to_string(operand1, &local_operand1,
                                                   ACPI_IMPLICIT_CONVERT_HEX);
                break;
 
        case ACPI_TYPE_BUFFER:
+
                status = acpi_ex_convert_to_buffer(operand1, &local_operand1);
                break;
 
        default:
+
                ACPI_ERROR((AE_INFO, "Invalid object type: 0x%X",
                            operand0->common.type));
                status = AE_AML_INTERNAL;
@@ -519,6 +522,7 @@ acpi_ex_do_logical_numeric_op(u16 opcode,
                break;
 
        default:
+
                status = AE_AML_INTERNAL;
                break;
        }
@@ -580,20 +584,24 @@ acpi_ex_do_logical_op(u16 opcode,
         */
        switch (operand0->common.type) {
        case ACPI_TYPE_INTEGER:
+
                status =
                    acpi_ex_convert_to_integer(operand1, &local_operand1, 16);
                break;
 
        case ACPI_TYPE_STRING:
+
                status = acpi_ex_convert_to_string(operand1, &local_operand1,
                                                   ACPI_IMPLICIT_CONVERT_HEX);
                break;
 
        case ACPI_TYPE_BUFFER:
+
                status = acpi_ex_convert_to_buffer(operand1, &local_operand1);
                break;
 
        default:
+
                status = AE_AML_INTERNAL;
                break;
        }
@@ -636,6 +644,7 @@ acpi_ex_do_logical_op(u16 opcode,
                        break;
 
                default:
+
                        status = AE_AML_INTERNAL;
                        break;
                }
@@ -703,6 +712,7 @@ acpi_ex_do_logical_op(u16 opcode,
                        break;
 
                default:
+
                        status = AE_AML_INTERNAL;
                        break;
                }
index b60c877f5906d9d8b7d00d9a022d3105f4aa2c2a..814b4a3d656ad1dc3d9acf438258735569017656 100644 (file)
@@ -327,7 +327,6 @@ acpi_status acpi_ex_opcode_1A_1T_1R(struct acpi_walk_state *walk_state)
                        break;
 
                case AML_FROM_BCD_OP:   /* from_bcd (BCDValue, Result) */
-
                        /*
                         * The 64-bit ACPI integer can hold 16 4-bit BCD characters
                         * (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)
                        break;
 
                case AML_COND_REF_OF_OP:        /* cond_ref_of (source_object, Result) */
-
                        /*
                         * This op is a little strange because the internal return value is
                         * 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)
                        goto cleanup;
 
                default:
+
                        /* No other opcodes get here */
+
                        break;
                }
                break;
 
        case AML_STORE_OP:      /* Store (Source, Target) */
-
                /*
                 * A store operand is typically a number, string, buffer or lvalue
                 * 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)
 
        case AML_DECREMENT_OP:  /* Decrement (Operand)  */
        case AML_INCREMENT_OP:  /* Increment (Operand)  */
-
                /*
                 * Create a new integer. Can't just get the base integer and
                 * 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)
                break;
 
        case AML_TYPE_OP:       /* object_type (source_object) */
-
                /*
                 * Note: The operand is not resolved at this point because we want to
                 * 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)
                break;
 
        case AML_SIZE_OF_OP:    /* size_of (source_object) */
-
                /*
                 * Note: The operand is not resolved at this point because we want to
                 * 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)
                 */
                switch (type) {
                case ACPI_TYPE_INTEGER:
+
                        value = acpi_gbl_integer_byte_width;
                        break;
 
                case ACPI_TYPE_STRING:
+
                        value = temp_desc->string.length;
                        break;
 
@@ -759,6 +757,7 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state)
                        break;
 
                default:
+
                        ACPI_ERROR((AE_INFO,
                                    "Operand must be Buffer/Integer/String/Package - found type %s",
                                    acpi_ut_get_type_name(type)));
@@ -860,9 +859,11 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state)
                                break;
 
                        case ACPI_TYPE_STRING:
+
                                break;
 
                        default:
+
                                status = AE_AML_OPERAND_TYPE;
                                goto cleanup;
                        }
@@ -923,7 +924,6 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state)
                         */
                        switch (operand[0]->reference.class) {
                        case ACPI_REFCLASS_INDEX:
-
                                /*
                                 * The target type for the Index operator must be
                                 * either a Buffer or a Package
@@ -956,7 +956,6 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state)
                                        break;
 
                                case ACPI_TYPE_PACKAGE:
-
                                        /*
                                         * Return the referenced element of the package. We must
                                         * 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)
                                break;
 
                        default:
+
                                ACPI_ERROR((AE_INFO,
                                            "Unknown class in reference(%p) - 0x%2.2X",
                                            operand[0],
index b0838a4ea53e6fc70cb9b57a8b3fc2fab1b2bce7..d5088f7030c753fb9e36b78b25d894e2c48cf74c 100644 (file)
@@ -304,7 +304,6 @@ acpi_status acpi_ex_opcode_2A_1T_1R(struct acpi_walk_state *walk_state)
                break;
 
        case AML_TO_STRING_OP:  /* to_string (Buffer, Length, Result) (ACPI 2.0) */
-
                /*
                 * Input object is guaranteed to be a buffer at this point (it may have
                 * been converted.)  Copy the raw buffer data to a new object of
index 2d7491f3126ef906ee6e589494f7936f20885166..37656f12f204bcd3b12ae6c1d8384eb392990e1a 100644 (file)
@@ -155,7 +155,6 @@ acpi_status acpi_ex_opcode_3A_1T_1R(struct acpi_walk_state *walk_state)
 
        switch (walk_state->opcode) {
        case AML_MID_OP:        /* Mid (Source[0], Index[1], Length[2], Result[3]) */
-
                /*
                 * Create the return object. The Source operand is guaranteed to be
                 * either a String or a Buffer, so just use its type.
index b76b97002dffd1701741b5fe18031f3104580dfb..879b6cd8319cdc09023e3852da1efe1b51f84282 100644 (file)
@@ -119,7 +119,6 @@ acpi_ex_do_match(u32 match_op,
                break;
 
        case MATCH_MEQ:
-
                /*
                 * True if equal: (P[i] == M)
                 * Change to:     (M == P[i])
@@ -133,7 +132,6 @@ acpi_ex_do_match(u32 match_op,
                break;
 
        case MATCH_MLE:
-
                /*
                 * True if less than or equal: (P[i] <= M) (P[i] not_greater than M)
                 * Change to:                  (M >= P[i]) (M not_less than P[i])
@@ -148,7 +146,6 @@ acpi_ex_do_match(u32 match_op,
                break;
 
        case MATCH_MLT:
-
                /*
                 * True if less than: (P[i] < M)
                 * Change to:         (M > P[i])
@@ -162,7 +159,6 @@ acpi_ex_do_match(u32 match_op,
                break;
 
        case MATCH_MGE:
-
                /*
                 * True if greater than or equal: (P[i] >= M) (P[i] not_less than M)
                 * Change to:                     (M <= P[i]) (M not_greater than P[i])
@@ -177,7 +173,6 @@ acpi_ex_do_match(u32 match_op,
                break;
 
        case MATCH_MGT:
-
                /*
                 * True if greater than: (P[i] > M)
                 * Change to:            (M < P[i])
index 6b728aef2dcab54804d11509e23660b06e11d21e..5a588611ab484f12e33966fcbdf4ce2a03ffdc4e 100644 (file)
@@ -253,26 +253,31 @@ acpi_ex_decode_field_access(union acpi_operand_object *obj_desc,
 
        case AML_FIELD_ACCESS_BYTE:
        case AML_FIELD_ACCESS_BUFFER:   /* ACPI 2.0 (SMBus Buffer) */
+
                byte_alignment = 1;
                bit_length = 8;
                break;
 
        case AML_FIELD_ACCESS_WORD:
+
                byte_alignment = 2;
                bit_length = 16;
                break;
 
        case AML_FIELD_ACCESS_DWORD:
+
                byte_alignment = 4;
                bit_length = 32;
                break;
 
        case AML_FIELD_ACCESS_QWORD:    /* ACPI 2.0 */
+
                byte_alignment = 8;
                bit_length = 64;
                break;
 
        default:
+
                /* Invalid field access type */
 
                ACPI_ERROR((AE_INFO, "Unknown field access type 0x%X", access));
@@ -598,7 +603,9 @@ acpi_status acpi_ex_prep_field_value(struct acpi_create_field_info *info)
                break;
 
        default:
+
                /* No other types should get here */
+
                break;
        }
 
index 182abaf045e165588606bd4a52de7c98413717c1..303429bb4d5d6631878878a872b3f0c46f82f390 100644 (file)
@@ -88,22 +88,27 @@ acpi_ex_system_memory_space_handler(u32 function,
 
        switch (bit_width) {
        case 8:
+
                length = 1;
                break;
 
        case 16:
+
                length = 2;
                break;
 
        case 32:
+
                length = 4;
                break;
 
        case 64:
+
                length = 8;
                break;
 
        default:
+
                ACPI_ERROR((AE_INFO, "Invalid SystemMemory width %u",
                            bit_width));
                return_ACPI_STATUS(AE_AML_OPERAND_VALUE);
@@ -214,23 +219,29 @@ acpi_ex_system_memory_space_handler(u32 function,
                *value = 0;
                switch (bit_width) {
                case 8:
-                       *value = (u64) ACPI_GET8(logical_addr_ptr);
+
+                       *value = (u64)ACPI_GET8(logical_addr_ptr);
                        break;
 
                case 16:
-                       *value = (u64) ACPI_GET16(logical_addr_ptr);
+
+                       *value = (u64)ACPI_GET16(logical_addr_ptr);
                        break;
 
                case 32:
-                       *value = (u64) ACPI_GET32(logical_addr_ptr);
+
+                       *value = (u64)ACPI_GET32(logical_addr_ptr);
                        break;
 
                case 64:
-                       *value = (u64) ACPI_GET64(logical_addr_ptr);
+
+                       *value = (u64)ACPI_GET64(logical_addr_ptr);
                        break;
 
                default:
+
                        /* bit_width was already validated */
+
                        break;
                }
                break;
@@ -239,28 +250,35 @@ acpi_ex_system_memory_space_handler(u32 function,
 
                switch (bit_width) {
                case 8:
+
                        ACPI_SET8(logical_addr_ptr, *value);
                        break;
 
                case 16:
+
                        ACPI_SET16(logical_addr_ptr, *value);
                        break;
 
                case 32:
+
                        ACPI_SET32(logical_addr_ptr, *value);
                        break;
 
                case 64:
+
                        ACPI_SET64(logical_addr_ptr, *value);
                        break;
 
                default:
+
                        /* bit_width was already validated */
+
                        break;
                }
                break;
 
        default:
+
                status = AE_BAD_PARAMETER;
                break;
        }
@@ -320,6 +338,7 @@ acpi_ex_system_io_space_handler(u32 function,
                break;
 
        default:
+
                status = AE_BAD_PARAMETER;
                break;
        }
index 8565b6bd12bb41a886aa8ad94d9355dd74d3a942..acd34f5993131d573aa6bcdf6046e7dd34afad12 100644 (file)
@@ -248,6 +248,7 @@ acpi_ex_resolve_node_to_value(struct acpi_namespace_node **object_ptr,
                        break;
 
                default:
+
                        /* No named references are allowed here */
 
                        ACPI_ERROR((AE_INFO,
index e4f9dfbb2a13f737e37e65bba8d374a7e673a987..ac04278ad28f00294d6fc7a3034795bcbb1c0bfb 100644 (file)
@@ -156,7 +156,6 @@ acpi_ex_resolve_object_to_value(union acpi_operand_object **stack_ptr,
                switch (ref_type) {
                case ACPI_REFCLASS_LOCAL:
                case ACPI_REFCLASS_ARG:
-
                        /*
                         * Get the local from the method's state info
                         * Note: this increments the local's object reference count
@@ -309,6 +308,7 @@ acpi_ex_resolve_object_to_value(union acpi_operand_object **stack_ptr,
                break;
 
        default:
+
                break;
        }
 
@@ -348,10 +348,12 @@ acpi_ex_resolve_multiple(struct acpi_walk_state *walk_state,
 
        switch (ACPI_GET_DESCRIPTOR_TYPE(obj_desc)) {
        case ACPI_DESC_TYPE_OPERAND:
+
                type = obj_desc->common.type;
                break;
 
        case ACPI_DESC_TYPE_NAMED:
+
                type = ((struct acpi_namespace_node *)obj_desc)->type;
                obj_desc =
                    acpi_ns_get_attached_object((struct acpi_namespace_node *)
@@ -538,7 +540,9 @@ acpi_ex_resolve_multiple(struct acpi_walk_state *walk_state,
                break;
 
        default:
+
                /* No change to Type required */
+
                break;
        }
 
index 9fb9f5e9a4daefe2f3a477eeb222d6ba1e560719..00e5af7129c1234338e4c5f0aeddcde5941ce94d 100644 (file)
@@ -307,7 +307,6 @@ acpi_ex_resolve_operands(u16 opcode,
                case ARGI_TARGETREF:    /* Allows implicit conversion rules before store */
                case ARGI_FIXED_TARGET: /* No implicit conversion before store to target */
                case ARGI_SIMPLE_TARGET:        /* Name, Local, or arg - no implicit conversion  */
-
                        /*
                         * Need an operand of type ACPI_TYPE_LOCAL_REFERENCE
                         * A Namespace Node is OK as-is
@@ -326,7 +325,6 @@ acpi_ex_resolve_operands(u16 opcode,
                        goto next_operand;
 
                case ARGI_DATAREFOBJ:   /* Store operator only */
-
                        /*
                         * We don't want to resolve index_op reference objects during
                         * a store because this would be an implicit de_ref_of operation.
@@ -343,7 +341,9 @@ acpi_ex_resolve_operands(u16 opcode,
                        break;
 
                default:
+
                        /* All cases covered above */
+
                        break;
                }
 
@@ -433,7 +433,6 @@ acpi_ex_resolve_operands(u16 opcode,
                        goto next_operand;
 
                case ARGI_BUFFER:
-
                        /*
                         * Need an operand of type ACPI_TYPE_BUFFER,
                         * But we can implicitly convert from a STRING or INTEGER
@@ -459,7 +458,6 @@ acpi_ex_resolve_operands(u16 opcode,
                        goto next_operand;
 
                case ARGI_STRING:
-
                        /*
                         * Need an operand of type ACPI_TYPE_STRING,
                         * But we can implicitly convert from a BUFFER or INTEGER
@@ -562,6 +560,7 @@ acpi_ex_resolve_operands(u16 opcode,
                                break;
 
                        default:
+
                                ACPI_ERROR((AE_INFO,
                                            "Needed [Buffer/String/Package/Reference], found [%s] %p",
                                            acpi_ut_get_object_type_name
@@ -584,6 +583,7 @@ acpi_ex_resolve_operands(u16 opcode,
                                break;
 
                        default:
+
                                ACPI_ERROR((AE_INFO,
                                            "Needed [Buffer/String/Package], found [%s] %p",
                                            acpi_ut_get_object_type_name
@@ -605,6 +605,7 @@ acpi_ex_resolve_operands(u16 opcode,
                                break;
 
                        default:
+
                                ACPI_ERROR((AE_INFO,
                                            "Needed [Region/Buffer], found [%s] %p",
                                            acpi_ut_get_object_type_name
index 93c6049c2d754d3e621729599510344e2d06d7dc..2bdba6f7d7620c43a8f6dafcbe83d5a14679f8b4 100644 (file)
@@ -114,6 +114,7 @@ acpi_ex_store(union acpi_operand_object *source_desc,
 
        switch (dest_desc->common.type) {
        case ACPI_TYPE_LOCAL_REFERENCE:
+
                break;
 
        case ACPI_TYPE_INTEGER:
@@ -178,7 +179,6 @@ acpi_ex_store(union acpi_operand_object *source_desc,
                break;
 
        case ACPI_REFCLASS_DEBUG:
-
                /*
                 * Storing to the Debug object causes the value stored to be
                 * displayed and otherwise has no effect -- see ACPI Specification
@@ -291,7 +291,6 @@ acpi_ex_store_object_to_index(union acpi_operand_object *source_desc,
                break;
 
        case ACPI_TYPE_BUFFER_FIELD:
-
                /*
                 * Store into a Buffer or String (not actually a real buffer_field)
                 * at a location defined by an Index.
@@ -447,7 +446,6 @@ acpi_ex_store_object_to_node(union acpi_operand_object *source_desc,
        case ACPI_TYPE_INTEGER:
        case ACPI_TYPE_STRING:
        case ACPI_TYPE_BUFFER:
-
                /*
                 * These target types are all of type Integer/String/Buffer, and
                 * therefore support implicit conversion before the store.
index 1cefe777068eef26b2d66851418c7992ede3244a..20d809d90c5b8f113e7b0f3c56fcafa58893dda2 100644 (file)
@@ -85,11 +85,9 @@ acpi_ex_resolve_object(union acpi_operand_object **source_desc_ptr,
                 * These cases all require only Integers or values that
                 * can be converted to Integers (Strings or Buffers)
                 */
-
        case ACPI_TYPE_INTEGER:
        case ACPI_TYPE_STRING:
        case ACPI_TYPE_BUFFER:
-
                /*
                 * Stores into a Field/Region or into a Integer/Buffer/String
                 * are all essentially the same. This case handles the
@@ -133,7 +131,6 @@ acpi_ex_resolve_object(union acpi_operand_object **source_desc_ptr,
 
        case ACPI_TYPE_LOCAL_ALIAS:
        case ACPI_TYPE_LOCAL_METHOD_ALIAS:
-
                /*
                 * All aliases should have been resolved earlier, during the
                 * operand resolution phase.
@@ -144,7 +141,6 @@ acpi_ex_resolve_object(union acpi_operand_object **source_desc_ptr,
 
        case ACPI_TYPE_PACKAGE:
        default:
-
                /*
                 * All other types than Alias and the various Fields come here,
                 * including the untyped case - ACPI_TYPE_ANY.
index 579c3a53ac87ede953e7610dab23f1270c6bafcc..3d36df828f520b1c196783453905c3f05f34543c 100644 (file)
@@ -108,7 +108,6 @@ acpi_status acpi_hw_set_mode(u32 mode)
                break;
 
        case ACPI_SYS_MODE_LEGACY:
-
                /*
                 * BIOS should clear all fixed status bits and restore fixed event
                 * enable bits to default
@@ -120,6 +119,7 @@ acpi_status acpi_hw_set_mode(u32 mode)
                break;
 
        default:
+
                return_ACPI_STATUS(AE_BAD_PARAMETER);
        }
 
index 20d02e93c99021e10038849b5290d77e360b55fc..96540506058fcc94351e2980feab2adf990d9ad6 100644 (file)
@@ -127,14 +127,17 @@ acpi_hw_low_set_gpe(struct acpi_gpe_event_info *gpe_event_info, u32 action)
                /*lint -fallthrough */
 
        case ACPI_GPE_ENABLE:
+
                ACPI_SET_BIT(enable_mask, register_bit);
                break;
 
        case ACPI_GPE_DISABLE:
+
                ACPI_CLEAR_BIT(enable_mask, register_bit);
                break;
 
        default:
+
                ACPI_ERROR((AE_INFO, "Invalid GPE Action, %u", action));
                return (AE_BAD_PARAMETER);
        }
index 083d6551f0e2b2db97020d6e0ecc4e9d7ed85211..8d2e866be15f84a301183cf5690205785ec4f2ea 100644 (file)
@@ -419,6 +419,7 @@ acpi_status acpi_hw_register_read(u32 register_id, u32 *return_value)
                break;
 
        default:
+
                ACPI_ERROR((AE_INFO, "Unknown Register ID: 0x%X", register_id));
                status = AE_BAD_PARAMETER;
                break;
@@ -491,7 +492,6 @@ acpi_status acpi_hw_register_write(u32 register_id, u32 value)
                break;
 
        case ACPI_REGISTER_PM1_CONTROL: /* PM1 A/B: 16-bit access each */
-
                /*
                 * Perform a read first to preserve certain bits (per ACPI spec)
                 * Note: This includes SCI_EN, we never want to change this bit
@@ -520,7 +520,6 @@ acpi_status acpi_hw_register_write(u32 register_id, u32 value)
                break;
 
        case ACPI_REGISTER_PM2_CONTROL: /* 8-bit access */
-
                /*
                 * For control registers, all reserved bits must be preserved,
                 * as per the ACPI spec.
@@ -555,6 +554,7 @@ acpi_status acpi_hw_register_write(u32 register_id, u32 value)
                break;
 
        default:
+
                ACPI_ERROR((AE_INFO, "Unknown Register ID: 0x%X", register_id));
                status = AE_BAD_PARAMETER;
                break;
index f4224e6f4e7a98617d33d49a81b48cfb16cb3d63..5ee7a814cd9207db34d42b8945867e415ab733bb 100644 (file)
@@ -528,10 +528,12 @@ acpi_get_sleep_type_data(u8 sleep_state, u8 *sleep_type_a, u8 *sleep_type_b)
        elements = info->return_object->package.elements;
        switch (info->return_object->package.count) {
        case 0:
+
                status = AE_AML_PACKAGE_LIMIT;
                break;
 
        case 1:
+
                if (elements[0]->common.type != ACPI_TYPE_INTEGER) {
                        status = AE_AML_OPERAND_TYPE;
                        break;
@@ -545,6 +547,7 @@ acpi_get_sleep_type_data(u8 sleep_state, u8 *sleep_type_a, u8 *sleep_type_b)
 
        case 2:
        default:
+
                if ((elements[0]->common.type != ACPI_TYPE_INTEGER) ||
                    (elements[1]->common.type != ACPI_TYPE_INTEGER)) {
                        status = AE_AML_OPERAND_TYPE;
index 35eebdac0f9d1e02976d382376005abbe4a870a5..111849c2117f1cf6255fe56b627aac7367beaeae 100644 (file)
@@ -314,20 +314,24 @@ acpi_status acpi_enter_sleep_state_prep(u8 sleep_state)
 
        switch (sleep_state) {
        case ACPI_STATE_S0:
+
                sst_value = ACPI_SST_WORKING;
                break;
 
        case ACPI_STATE_S1:
        case ACPI_STATE_S2:
        case ACPI_STATE_S3:
+
                sst_value = ACPI_SST_SLEEPING;
                break;
 
        case ACPI_STATE_S4:
+
                sst_value = ACPI_SST_SLEEP_CONTEXT;
                break;
 
        default:
+
                sst_value = ACPI_SST_INDICATOR_OFF;     /* Default is off */
                break;
        }
index 8769cf83b044ff704db113a4ef7ffc8a5f78fbc0..c5316e5bd4abf2efce5aad7d4968708d3bccdb3d 100644 (file)
@@ -151,6 +151,7 @@ acpi_status acpi_ns_root_initialize(void)
                         */
                        switch (init_val->type) {
                        case ACPI_TYPE_METHOD:
+
                                obj_desc->method.param_count =
                                    (u8) ACPI_TO_INTEGER(val);
                                obj_desc->common.flags |= AOPOBJ_DATA_VALID;
index 8f79a9d2d50ec659f157e0c3ccd011d81c0658df..acd2964c26906f995b159bd382a42ce000cfa2c3 100644 (file)
@@ -103,6 +103,7 @@ acpi_ns_convert_to_integer(union acpi_operand_object *original_object,
                break;
 
        default:
+
                return (AE_AML_OPERAND_TYPE);
        }
 
@@ -191,6 +192,7 @@ acpi_ns_convert_to_string(union acpi_operand_object *original_object,
                break;
 
        default:
+
                return (AE_AML_OPERAND_TYPE);
        }
 
@@ -294,6 +296,7 @@ acpi_ns_convert_to_buffer(union acpi_operand_object *original_object,
                break;
 
        default:
+
                return (AE_AML_OPERAND_TYPE);
        }
 
index ce6e9732620523ba36e1a474d76dddfcf03005d9..7418c77fde8c70db88d0d0af9c0c6a8fcad6cad5 100644 (file)
@@ -244,10 +244,12 @@ acpi_ns_dump_one_object(acpi_handle obj_handle,
                        case ACPI_TYPE_BUFFER:
                        case ACPI_TYPE_STRING:
                        case ACPI_TYPE_METHOD:
+
                                acpi_os_printf("<No attached object>");
                                break;
 
                        default:
+
                                break;
                        }
 
@@ -433,6 +435,7 @@ acpi_ns_dump_one_object(acpi_handle obj_handle,
                        break;
 
                default:
+
                        break;
                }
                break;
@@ -567,32 +570,39 @@ acpi_ns_dump_one_object(acpi_handle obj_handle,
                        goto cleanup;
 
                case ACPI_TYPE_BUFFER_FIELD:
+
                        obj_desc =
                            (union acpi_operand_object *)obj_desc->buffer_field.
                            buffer_obj;
                        break;
 
                case ACPI_TYPE_PACKAGE:
+
                        obj_desc = (void *)obj_desc->package.elements;
                        break;
 
                case ACPI_TYPE_METHOD:
+
                        obj_desc = (void *)obj_desc->method.aml_start;
                        break;
 
                case ACPI_TYPE_LOCAL_REGION_FIELD:
+
                        obj_desc = (void *)obj_desc->field.region_obj;
                        break;
 
                case ACPI_TYPE_LOCAL_BANK_FIELD:
+
                        obj_desc = (void *)obj_desc->bank_field.region_obj;
                        break;
 
                case ACPI_TYPE_LOCAL_INDEX_FIELD:
+
                        obj_desc = (void *)obj_desc->index_field.index_obj;
                        break;
 
                default:
+
                        goto cleanup;
                }
 
index 23e426f7fd1a59807b3886049d2527d4945790fc..dd2ceae3f7177aff169e56391292794c0651a74c 100644 (file)
@@ -266,28 +266,34 @@ acpi_ns_init_one_object(acpi_handle obj_handle,
 
        switch (type) {
        case ACPI_TYPE_REGION:
+
                info->op_region_count++;
                break;
 
        case ACPI_TYPE_BUFFER_FIELD:
+
                info->field_count++;
                break;
 
        case ACPI_TYPE_LOCAL_BANK_FIELD:
+
                info->field_count++;
                break;
 
        case ACPI_TYPE_BUFFER:
+
                info->buffer_count++;
                break;
 
        case ACPI_TYPE_PACKAGE:
+
                info->package_count++;
                break;
 
        default:
 
                /* No init required, just exit now */
+
                return (AE_OK);
        }
 
@@ -337,7 +343,9 @@ acpi_ns_init_one_object(acpi_handle obj_handle,
                break;
 
        default:
+
                /* No other types can get here */
+
                break;
        }
 
@@ -416,6 +424,7 @@ acpi_ns_find_ini_methods(acpi_handle obj_handle,
                break;
 
        default:
+
                break;
        }
 
index 5c83a8b1c157c43a62c1b54adf9139a97748f2ab..600268d33aa875ee64fc20713424e699edb61d08 100644 (file)
@@ -344,26 +344,32 @@ static u32 acpi_ns_get_bitmapped_type(union acpi_operand_object *return_object)
 
        switch (return_object->common.type) {
        case ACPI_TYPE_INTEGER:
+
                return_btype = ACPI_RTYPE_INTEGER;
                break;
 
        case ACPI_TYPE_BUFFER:
+
                return_btype = ACPI_RTYPE_BUFFER;
                break;
 
        case ACPI_TYPE_STRING:
+
                return_btype = ACPI_RTYPE_STRING;
                break;
 
        case ACPI_TYPE_PACKAGE:
+
                return_btype = ACPI_RTYPE_PACKAGE;
                break;
 
        case ACPI_TYPE_LOCAL_REFERENCE:
+
                return_btype = ACPI_RTYPE_REFERENCE;
                break;
 
        default:
+
                /* Not one of the supported objects, must be incorrect */
 
                return_btype = ACPI_RTYPE_ANY;
index ab2a3f477e1edba90d78cdce054a1591892b6dcd..6d55cef7916c4ba4715a9108697073822edc48ec 100644 (file)
@@ -136,7 +136,6 @@ acpi_ns_check_package(struct acpi_evaluate_info *info,
         */
        switch (package->ret_info.type) {
        case ACPI_PTYPE1_FIXED:
-
                /*
                 * The package count is fixed and there are no sub-packages
                 *
@@ -169,7 +168,6 @@ acpi_ns_check_package(struct acpi_evaluate_info *info,
                break;
 
        case ACPI_PTYPE1_VAR:
-
                /*
                 * The package count is variable, there are no sub-packages, and all
                 * elements must be of the same type
@@ -186,7 +184,6 @@ acpi_ns_check_package(struct acpi_evaluate_info *info,
                break;
 
        case ACPI_PTYPE1_OPTION:
-
                /*
                 * The package count is variable, there are no sub-packages. There are
                 * a fixed number of required elements, and a variable number of
@@ -284,7 +281,6 @@ acpi_ns_check_package(struct acpi_evaluate_info *info,
        case ACPI_PTYPE2_MIN:
        case ACPI_PTYPE2_COUNT:
        case ACPI_PTYPE2_FIX_VAR:
-
                /*
                 * These types all return a single Package that consists of a
                 * variable number of sub-Packages.
@@ -500,7 +496,6 @@ acpi_ns_check_package_list(struct acpi_evaluate_info *info,
                        break;
 
                case ACPI_PTYPE2_COUNT:
-
                        /*
                         * First element is the (Integer) count of elements, including
                         * the count field (the ACPI name is num_elements)
index 89caef04691e0f11359879a9a5aeec66bfd7f09f..f8e71ea60319b21b6c014c88c69efc0b5a2dc751 100644 (file)
@@ -480,6 +480,7 @@ acpi_ns_remove_null_elements(struct acpi_evaluate_info *info,
        case ACPI_PTYPE2_MIN:
        case ACPI_PTYPE2_REV_FIXED:
        case ACPI_PTYPE2_FIX_VAR:
+
                break;
 
        default:
index 39acc4e6b3878a1f98eba734fe23cee7a9648b0b..daac8daeaa9f98ae157747fe1555e04536348372 100644 (file)
@@ -320,6 +320,7 @@ acpi_ns_repair_FDE(struct acpi_evaluate_info *info,
                break;
 
        default:
+
                return (AE_AML_OPERAND_TYPE);
        }
 
index 2808586fad30c61a7bb6c5e014cde4b5a5dcff15..08c0b5beec888c65ea0d175f2795bb1be0f95227 100644 (file)
@@ -419,10 +419,12 @@ acpi_ns_externalize_name(u32 internal_name_length,
 
        switch (internal_name[0]) {
        case AML_ROOT_PREFIX:
+
                prefix_length = 1;
                break;
 
        case AML_PARENT_PREFIX:
+
                for (i = 0; i < internal_name_length; i++) {
                        if (ACPI_IS_PARENT_PREFIX(internal_name[i])) {
                                prefix_length = i + 1;
@@ -438,6 +440,7 @@ acpi_ns_externalize_name(u32 internal_name_length,
                break;
 
        default:
+
                break;
        }
 
index 82bec40d4508e8162eb94a1ea0ccec61f45d2919..f553cfdb71ddd6ec350b63077f6eda49f30daac9 100644 (file)
@@ -511,6 +511,7 @@ static void acpi_ns_resolve_references(struct acpi_evaluate_info *info)
                break;
 
        default:
+
                return;
        }
 
index 9f25a3d4e992c0dc890325f8ed0029942c3bac5b..91a5a69db80c316552e13f9704eb9435f9b94a30 100644 (file)
@@ -629,24 +629,28 @@ static union acpi_parse_object *acpi_ps_get_next_field(struct acpi_parse_state
 
                                switch (opcode) {
                                case AML_BYTE_OP:       /* AML_BYTEDATA_ARG */
+
                                        buffer_length =
                                            ACPI_GET8(parser_state->aml);
                                        parser_state->aml += 1;
                                        break;
 
                                case AML_WORD_OP:       /* AML_WORDDATA_ARG */
+
                                        buffer_length =
                                            ACPI_GET16(parser_state->aml);
                                        parser_state->aml += 2;
                                        break;
 
                                case AML_DWORD_OP:      /* AML_DWORDATA_ARG */
+
                                        buffer_length =
                                            ACPI_GET32(parser_state->aml);
                                        parser_state->aml += 4;
                                        break;
 
                                default:
+
                                        buffer_length = 0;
                                        break;
                                }
index 63c4554474817b591fdc0d71771dde8f2cf970ed..065b44ae538f28b7595fe9af1870b5c824052237 100644 (file)
@@ -164,7 +164,6 @@ acpi_ps_get_arguments(struct acpi_walk_state *walk_state,
                        case AML_IF_OP:
                        case AML_ELSE_OP:
                        case AML_WHILE_OP:
-
                                /*
                                 * Currently supported module-level opcodes are:
                                 * IF/ELSE/WHILE. These appear to be the most common,
@@ -289,6 +288,7 @@ acpi_ps_get_arguments(struct acpi_walk_state *walk_state,
                default:
 
                        /* No action for all other opcodes */
+
                        break;
                }
 
index 12c4028002b145e0fef8b9ed60ab9f00ccd100dc..95dc608a66a85b3b01e5a461b58a39f4a02b20f7 100644 (file)
@@ -402,6 +402,7 @@ acpi_ps_complete_op(struct acpi_walk_state *walk_state,
 
        switch (status) {
        case AE_OK:
+
                break;
 
        case AE_CTRL_TRANSFER:
index abc4c48b2eddd7c18acb854a5eeb960d73080393..86198a9139b52a653d24d6758ba63ed0d04df800 100644 (file)
@@ -176,10 +176,10 @@ acpi_ps_complete_this_op(struct acpi_walk_state * walk_state,
 
                switch (parent_info->class) {
                case AML_CLASS_CONTROL:
+
                        break;
 
                case AML_CLASS_CREATE:
-
                        /*
                         * These opcodes contain term_arg operands. The current
                         * op must be replaced by a placeholder return op
@@ -192,7 +192,6 @@ acpi_ps_complete_this_op(struct acpi_walk_state * walk_state,
                        break;
 
                case AML_CLASS_NAMED_OBJECT:
-
                        /*
                         * These opcodes contain term_arg operands. The current
                         * op must be replaced by a placeholder return op
index c1934bf04f0af56f16db267dcdc5d3a914b749ab..877dc0de8df3e19da7057a8fd99042eb2d6f80b6 100644 (file)
@@ -308,7 +308,9 @@ union acpi_parse_object *acpi_ps_get_child(union acpi_parse_object *op)
                break;
 
        default:
+
                /* All others have no children */
+
                break;
        }
 
index 72077fa1eea5ecd7e5c489e3b65632c0be639ef2..608ebb56b6715609868f95c795a5bfecdba33f04 100644 (file)
@@ -352,6 +352,7 @@ acpi_rs_get_aml_length(struct acpi_resource * resource, acpi_size * size_needed)
                        break;
 
                default:
+
                        break;
                }
 
@@ -539,6 +540,7 @@ acpi_rs_get_list_length(u8 * aml_buffer,
                        break;
 
                default:
+
                        break;
                }
 
index b5fc0db2e87bb4a78ddbaa707ee91a886b4e7215..8a2d4986b0aa576695ed2f479703cfa8dc4fd139 100644 (file)
@@ -120,17 +120,20 @@ acpi_rs_dump_descriptor(void *resource, struct acpi_rsdump_info *table)
                        /* Strings */
 
                case ACPI_RSD_LITERAL:
+
                        acpi_rs_out_string(name,
                                           ACPI_CAST_PTR(char, table->pointer));
                        break;
 
                case ACPI_RSD_STRING:
+
                        acpi_rs_out_string(name, ACPI_CAST_PTR(char, target));
                        break;
 
                        /* Data items, 8/16/32/64 bit */
 
                case ACPI_RSD_UINT8:
+
                        if (table->pointer) {
                                acpi_rs_out_string(name, ACPI_CAST_PTR(char,
                                                                       table->
@@ -142,20 +145,24 @@ acpi_rs_dump_descriptor(void *resource, struct acpi_rsdump_info *table)
                        break;
 
                case ACPI_RSD_UINT16:
+
                        acpi_rs_out_integer16(name, ACPI_GET16(target));
                        break;
 
                case ACPI_RSD_UINT32:
+
                        acpi_rs_out_integer32(name, ACPI_GET32(target));
                        break;
 
                case ACPI_RSD_UINT64:
+
                        acpi_rs_out_integer64(name, ACPI_GET64(target));
                        break;
 
                        /* Flags: 1-bit and 2-bit flags supported */
 
                case ACPI_RSD_1BITFLAG:
+
                        acpi_rs_out_string(name, ACPI_CAST_PTR(char,
                                                               table->
                                                               pointer[*target &
@@ -163,6 +170,7 @@ acpi_rs_dump_descriptor(void *resource, struct acpi_rsdump_info *table)
                        break;
 
                case ACPI_RSD_2BITFLAG:
+
                        acpi_rs_out_string(name, ACPI_CAST_PTR(char,
                                                               table->
                                                               pointer[*target &
@@ -170,6 +178,7 @@ acpi_rs_dump_descriptor(void *resource, struct acpi_rsdump_info *table)
                        break;
 
                case ACPI_RSD_3BITFLAG:
+
                        acpi_rs_out_string(name, ACPI_CAST_PTR(char,
                                                               table->
                                                               pointer[*target &
@@ -258,6 +267,7 @@ acpi_rs_dump_descriptor(void *resource, struct acpi_rsdump_info *table)
                        break;
 
                default:
+
                        acpi_os_printf("**** Invalid table opcode [%X] ****\n",
                                       table->opcode);
                        return;
index d5bf05a96096ad785897966876929b02dba60c02..80d12994e0d078f10b8842d6c5ee3d8a4c965c35 100644 (file)
@@ -194,7 +194,6 @@ acpi_rs_convert_aml_to_resource(struct acpi_resource *resource,
                        break;
 
                case ACPI_RSC_COUNT_GPIO_RES:
-
                        /*
                         * Vendor data is optional (length/offset may both be zero)
                         * Examine vendor data length field first
@@ -410,12 +409,14 @@ acpi_rs_convert_aml_to_resource(struct acpi_resource *resource,
                         */
                        switch (info->resource_offset) {
                        case ACPI_RSC_COMPARE_AML_LENGTH:
+
                                if (aml_resource_length != info->value) {
                                        goto exit;
                                }
                                break;
 
                        case ACPI_RSC_COMPARE_VALUE:
+
                                if (ACPI_GET8(source) != info->value) {
                                        goto exit;
                                }
index 59bdf37164ba8c4f5e56d950be2ec9de190d734b..480b6b40c5eaa3527ce58b4f76f2b8a23b16db97 100644 (file)
@@ -147,6 +147,7 @@ acpi_rs_move_data(void *destination, void *source, u16 item_count, u8 move_type)
                case ACPI_RSC_MOVE_GPIO_RES:
                case ACPI_RSC_MOVE_SERIAL_VEN:
                case ACPI_RSC_MOVE_SERIAL_RES:
+
                        ACPI_MEMCPY(destination, source, item_count);
                        return;
 
@@ -157,21 +158,25 @@ acpi_rs_move_data(void *destination, void *source, u16 item_count, u8 move_type)
                         */
                case ACPI_RSC_MOVE16:
                case ACPI_RSC_MOVE_GPIO_PIN:
+
                        ACPI_MOVE_16_TO_16(&ACPI_CAST_PTR(u16, destination)[i],
                                           &ACPI_CAST_PTR(u16, source)[i]);
                        break;
 
                case ACPI_RSC_MOVE32:
+
                        ACPI_MOVE_32_TO_32(&ACPI_CAST_PTR(u32, destination)[i],
                                           &ACPI_CAST_PTR(u32, source)[i]);
                        break;
 
                case ACPI_RSC_MOVE64:
+
                        ACPI_MOVE_64_TO_64(&ACPI_CAST_PTR(u64, destination)[i],
                                           &ACPI_CAST_PTR(u64, source)[i]);
                        break;
 
                default:
+
                        return;
                }
        }
index c0e5d2d3ce6757380928c9e50d1f2de0f158160a..94e3517554f9d52d121dd4f31469ddb4a9ec6567 100644 (file)
@@ -402,6 +402,7 @@ acpi_resource_to_address64(struct acpi_resource *resource,
                break;
 
        default:
+
                return (AE_BAD_PARAMETER);
        }
 
index e57cd38004e33c3adb3b7850bc25b4f23f577430..a04e54d984d98e5846277f9d4cdbabd734e7dfb2 100644 (file)
@@ -471,15 +471,19 @@ void acpi_tb_delete_table(struct acpi_table_desc *table_desc)
        }
        switch (table_desc->flags & ACPI_TABLE_ORIGIN_MASK) {
        case ACPI_TABLE_ORIGIN_MAPPED:
+
                acpi_os_unmap_memory(table_desc->pointer, table_desc->length);
                break;
+
        case ACPI_TABLE_ORIGIN_ALLOCATED:
+
                ACPI_FREE(table_desc->pointer);
                break;
 
                /* Not mapped or allocated, there is nothing we can do */
 
        default:
+
                return;
        }
 
index e4c9291fc0a3f530bfaea472895f6217a23f8b99..1731c27c36a682e5d47beba328096ff9c633043a 100644 (file)
@@ -178,7 +178,6 @@ acpi_ut_copy_isimple_to_esimple(union acpi_operand_object *internal_object,
 
                switch (internal_object->reference.class) {
                case ACPI_REFCLASS_NAME:
-
                        /*
                         * For namepath, return the object handle ("reference")
                         * We are referring to the namespace node
@@ -264,7 +263,6 @@ acpi_ut_copy_ielement_to_eelement(u8 object_type,
 
        switch (object_type) {
        case ACPI_COPY_TYPE_SIMPLE:
-
                /*
                 * This is a simple or null object
                 */
@@ -278,7 +276,6 @@ acpi_ut_copy_ielement_to_eelement(u8 object_type,
                break;
 
        case ACPI_COPY_TYPE_PACKAGE:
-
                /*
                 * Build the package object
                 */
@@ -304,6 +301,7 @@ acpi_ut_copy_ielement_to_eelement(u8 object_type,
                break;
 
        default:
+
                return (AE_BAD_PARAMETER);
        }
 
@@ -481,6 +479,7 @@ acpi_ut_copy_esimple_to_isimple(union acpi_object *external_object,
                return_ACPI_STATUS(AE_OK);
 
        default:
+
                /* All other types are not supported */
 
                ACPI_ERROR((AE_INFO,
@@ -544,7 +543,9 @@ acpi_ut_copy_esimple_to_isimple(union acpi_object *external_object,
                break;
 
        default:
+
                /* Other types can't get here */
+
                break;
        }
 
@@ -800,7 +801,9 @@ acpi_ut_copy_simple_object(union acpi_operand_object *source_desc,
                break;
 
        default:
+
                /* Nothing to do for other simple objects */
+
                break;
        }
 
@@ -868,7 +871,6 @@ acpi_ut_copy_ielement_to_ielement(u8 object_type,
                break;
 
        case ACPI_COPY_TYPE_PACKAGE:
-
                /*
                 * This object is a package - go down another nesting level
                 * Create and build the package object
@@ -891,6 +893,7 @@ acpi_ut_copy_ielement_to_ielement(u8 object_type,
                break;
 
        default:
+
                return (AE_BAD_PARAMETER);
        }
 
index 29b930250b6fe01a7822a229dc0f45ed80290a52..d6b33f29d32754a50eef108ce1287b59886b013b 100644 (file)
@@ -303,6 +303,7 @@ static void acpi_ut_delete_internal_obj(union acpi_operand_object *object)
                break;
 
        default:
+
                break;
        }
 
@@ -508,7 +509,6 @@ acpi_ut_update_object_reference(union acpi_operand_object *object, u16 action)
                case ACPI_TYPE_PROCESSOR:
                case ACPI_TYPE_POWER:
                case ACPI_TYPE_THERMAL:
-
                        /*
                         * Update the notify objects for these types (if present)
                         * Two lists, system and device notify handlers.
@@ -623,6 +623,7 @@ acpi_ut_update_object_reference(union acpi_operand_object *object, u16 action)
 
                case ACPI_TYPE_REGION:
                default:
+
                        break;  /* No subobjects for all other types */
                }
 
index 9d6802911c01e12fd4e7b95d246e46369bdb06f1..ee83adb97b1e73827a6a7740de03edb3680e2a28 100644 (file)
@@ -123,22 +123,27 @@ acpi_ut_evaluate_object(struct acpi_namespace_node *prefix_node,
 
        switch ((info->return_object)->common.type) {
        case ACPI_TYPE_INTEGER:
+
                return_btype = ACPI_BTYPE_INTEGER;
                break;
 
        case ACPI_TYPE_BUFFER:
+
                return_btype = ACPI_BTYPE_BUFFER;
                break;
 
        case ACPI_TYPE_STRING:
+
                return_btype = ACPI_BTYPE_STRING;
                break;
 
        case ACPI_TYPE_PACKAGE:
+
                return_btype = ACPI_BTYPE_PACKAGE;
                break;
 
        default:
+
                return_btype = 0;
                break;
        }
index b543a144941a295a81c7aa9f237b56ac07e1a748..ff6d9e8aa8423ef04ed68ec18023d61ee570493b 100644 (file)
@@ -146,6 +146,7 @@ const struct acpi_exception_info *acpi_ut_validate_exception(acpi_status status)
                break;
 
        default:
+
                break;
        }
 
index 43a170a74a6139ae3aa10ae1120b27f773103085..fa69071db4181dc100e3242053756944590679f8 100644 (file)
@@ -341,14 +341,17 @@ acpi_ut_execute_CID(struct acpi_namespace_node *device_node,
 
                switch (cid_objects[i]->common.type) {
                case ACPI_TYPE_INTEGER:
+
                        string_area_size += ACPI_EISAID_STRING_SIZE;
                        break;
 
                case ACPI_TYPE_STRING:
+
                        string_area_size += cid_objects[i]->string.length + 1;
                        break;
 
                default:
+
                        status = AE_TYPE;
                        goto cleanup;
                }
index 785fdd07ae5673db012d27b1519dce75e85c3827..02f9101b65e47c29dc974020a8701322c9dc3824 100644 (file)
@@ -382,10 +382,12 @@ acpi_ut_display_init_pathname(u8 type,
 
        switch (type) {
        case ACPI_TYPE_METHOD:
+
                acpi_os_printf("Executing  ");
                break;
 
        default:
+
                acpi_os_printf("Initializing ");
                break;
        }
index 1099f5c069f8287fb354f148c9f8d681fcc94e44..aa61f66ee861ed1de71fee8bb72d0e412627382c 100644 (file)
@@ -129,6 +129,7 @@ union acpi_operand_object *acpi_ut_create_internal_object_dbg(const char
                break;
 
        default:
+
                /* All others have no secondary object */
                break;
        }
@@ -353,6 +354,7 @@ u8 acpi_ut_valid_internal_object(void *object)
                return (TRUE);
 
        default:
+
                ACPI_DEBUG_PRINT((ACPI_DB_EXEC,
                                  "%p is not not an ACPI operand obj [%s]\n",
                                  object, acpi_ut_get_descriptor_name(object)));
@@ -509,7 +511,6 @@ acpi_ut_get_simple_object_size(union acpi_operand_object *internal_object,
 
                switch (internal_object->reference.class) {
                case ACPI_REFCLASS_NAME:
-
                        /*
                         * Get the actual length of the full pathname to this object.
                         * The reference will be converted to the pathname to the object
@@ -525,7 +526,6 @@ acpi_ut_get_simple_object_size(union acpi_operand_object *internal_object,
                        break;
 
                default:
-
                        /*
                         * No other reference opcodes are supported.
                         * Notably, Locals and Args are not supported, but this may be
@@ -585,7 +585,6 @@ acpi_ut_get_element_length(u8 object_type,
 
        switch (object_type) {
        case ACPI_COPY_TYPE_SIMPLE:
-
                /*
                 * Simple object - just get the size (Null object/entry is handled
                 * here also) and sum it into the running package length
index b3e36a81aa4d1360b9280d2edd6d9d5f9e7330dd..31a06a17e29dfb01acf47e89dda71c593aa211c1 100644 (file)
@@ -186,10 +186,13 @@ acpi_status acpi_ut_strtoul64(char *string, u32 base, u64 *ret_integer)
        switch (base) {
        case ACPI_ANY_BASE:
        case 16:
+
                break;
 
        default:
+
                /* Invalid Base */
+
                return_ACPI_STATUS(AE_BAD_PARAMETER);
        }
 
@@ -355,36 +358,44 @@ void acpi_ut_print_string(char *string, u8 max_length)
 
                switch (string[i]) {
                case 0x07:
+
                        acpi_os_printf("\\a");  /* BELL */
                        break;
 
                case 0x08:
+
                        acpi_os_printf("\\b");  /* BACKSPACE */
                        break;
 
                case 0x0C:
+
                        acpi_os_printf("\\f");  /* FORMFEED */
                        break;
 
                case 0x0A:
+
                        acpi_os_printf("\\n");  /* LINEFEED */
                        break;
 
                case 0x0D:
+
                        acpi_os_printf("\\r");  /* CARRIAGE RETURN */
                        break;
 
                case 0x09:
+
                        acpi_os_printf("\\t");  /* HORIZONTAL TAB */
                        break;
 
                case 0x0B:
+
                        acpi_os_printf("\\v");  /* VERTICAL TAB */
                        break;
 
                case '\'':      /* Single Quote */
                case '\"':      /* Double Quote */
                case '\\':      /* Backslash */
+
                        acpi_os_printf("\\%c", (int)string[i]);
                        break;
 
index 62774c7b76a81402633d417277be05c5953239d0..160f13f4aab51a196200184c60cc625a5de4acc6 100644 (file)
@@ -603,6 +603,7 @@ void acpi_ut_dump_allocations(u32 component, const char *module)
                                        switch (ACPI_GET_DESCRIPTOR_TYPE
                                                (descriptor)) {
                                        case ACPI_DESC_TYPE_OPERAND:
+
                                                if (element->size ==
                                                    sizeof(union
                                                           acpi_operand_object))
@@ -613,6 +614,7 @@ void acpi_ut_dump_allocations(u32 component, const char *module)
                                                break;
 
                                        case ACPI_DESC_TYPE_PARSER:
+
                                                if (element->size ==
                                                    sizeof(union
                                                           acpi_parse_object)) {
@@ -622,6 +624,7 @@ void acpi_ut_dump_allocations(u32 component, const char *module)
                                                break;
 
                                        case ACPI_DESC_TYPE_NAMED:
+
                                                if (element->size ==
                                                    sizeof(struct
                                                           acpi_namespace_node))
@@ -632,6 +635,7 @@ void acpi_ut_dump_allocations(u32 component, const char *module)
                                                break;
 
                                        default:
+
                                                break;
                                        }
 
@@ -639,6 +643,7 @@ void acpi_ut_dump_allocations(u32 component, const char *module)
 
                                        switch (descriptor_type) {
                                        case ACPI_DESC_TYPE_OPERAND:
+
                                                acpi_os_printf
                                                    ("%12.12s RefCount 0x%04X\n",
                                                     acpi_ut_get_type_name
@@ -649,6 +654,7 @@ void acpi_ut_dump_allocations(u32 component, const char *module)
                                                break;
 
                                        case ACPI_DESC_TYPE_PARSER:
+
                                                acpi_os_printf
                                                    ("AmlOpcode 0x%04hX\n",
                                                     descriptor->op.asl.
@@ -656,6 +662,7 @@ void acpi_ut_dump_allocations(u32 component, const char *module)
                                                break;
 
                                        case ACPI_DESC_TYPE_NAMED:
+
                                                acpi_os_printf("%4.4s\n",
                                                               acpi_ut_get_node_name
                                                               (&descriptor->
@@ -663,6 +670,7 @@ void acpi_ut_dump_allocations(u32 component, const char *module)
                                                break;
 
                                        default:
+
                                                acpi_os_printf("\n");
                                                break;
                                        }