Fedora kernel-2.6.17-1.2142_FC4 patched with stable patch-2.6.17.4-vs2.0.2-rc26.diff
[linux-2.6.git] / drivers / acpi / executer / excreate.c
index d94c260..6805754 100644 (file)
@@ -5,7 +5,7 @@
  *****************************************************************************/
 
 /*
- * Copyright (C) 2000 - 2005, R. Byron Moore
+ * Copyright (C) 2000 - 2006, R. Byron Moore
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -41,7 +41,6 @@
  * POSSIBILITY OF SUCH DAMAGES.
  */
 
-
 #include <acpi/acpi.h>
 #include <acpi/acinterp.h>
 #include <acpi/amlcode.h>
 #include <acpi/acevents.h>
 #include <acpi/actables.h>
 
-
 #define _COMPONENT          ACPI_EXECUTER
-        ACPI_MODULE_NAME    ("excreate")
-
+ACPI_MODULE_NAME("excreate")
 
 #ifndef ACPI_NO_METHOD_EXECUTION
-/*****************************************************************************
+/*******************************************************************************
  *
  * FUNCTION:    acpi_ex_create_alias
  *
  *
  * DESCRIPTION: Create a new named alias
  *
- ****************************************************************************/
-
-acpi_status
-acpi_ex_create_alias (
-       struct acpi_walk_state          *walk_state)
+ ******************************************************************************/
+acpi_status acpi_ex_create_alias(struct acpi_walk_state *walk_state)
 {
-       struct acpi_namespace_node      *target_node;
-       struct acpi_namespace_node      *alias_node;
-       acpi_status                     status = AE_OK;
-
-
-       ACPI_FUNCTION_TRACE ("ex_create_alias");
+       struct acpi_namespace_node *target_node;
+       struct acpi_namespace_node *alias_node;
+       acpi_status status = AE_OK;
 
+       ACPI_FUNCTION_TRACE("ex_create_alias");
 
        /* Get the source/alias operands (both namespace nodes) */
 
-       alias_node = (struct acpi_namespace_node *) walk_state->operands[0];
-       target_node = (struct acpi_namespace_node *) walk_state->operands[1];
+       alias_node = (struct acpi_namespace_node *)walk_state->operands[0];
+       target_node = (struct acpi_namespace_node *)walk_state->operands[1];
 
        if ((target_node->type == ACPI_TYPE_LOCAL_ALIAS) ||
-               (target_node->type == ACPI_TYPE_LOCAL_METHOD_ALIAS)) {
+           (target_node->type == ACPI_TYPE_LOCAL_METHOD_ALIAS)) {
                /*
                 * Dereference an existing alias so that we don't create a chain
                 * of aliases.  With this code, we guarantee that an alias is
                 * always exactly one level of indirection away from the
                 * actual aliased name.
                 */
-               target_node = ACPI_CAST_PTR (struct acpi_namespace_node, target_node->object);
+               target_node =
+                   ACPI_CAST_PTR(struct acpi_namespace_node,
+                                 target_node->object);
        }
 
        /*
@@ -115,7 +109,8 @@ acpi_ex_create_alias (
                 * types, the object can change dynamically via a Store.
                 */
                alias_node->type = ACPI_TYPE_LOCAL_ALIAS;
-               alias_node->object = ACPI_CAST_PTR (union acpi_operand_object, target_node);
+               alias_node->object =
+                   ACPI_CAST_PTR(union acpi_operand_object, target_node);
                break;
 
        case ACPI_TYPE_METHOD:
@@ -126,7 +121,8 @@ acpi_ex_create_alias (
                 * types, the object can change dynamically via a Store.
                 */
                alias_node->type = ACPI_TYPE_LOCAL_METHOD_ALIAS;
-               alias_node->object = ACPI_CAST_PTR (union acpi_operand_object, target_node);
+               alias_node->object =
+                   ACPI_CAST_PTR(union acpi_operand_object, target_node);
                break;
 
        default:
@@ -139,19 +135,19 @@ acpi_ex_create_alias (
                 * additional reference to prevent deletion out from under either the
                 * target node or the alias Node
                 */
-               status = acpi_ns_attach_object (alias_node,
-                                acpi_ns_get_attached_object (target_node),
-                                target_node->type);
+               status = acpi_ns_attach_object(alias_node,
+                                              acpi_ns_get_attached_object
+                                              (target_node),
+                                              target_node->type);
                break;
        }
 
        /* Since both operands are Nodes, we don't need to delete them */
 
-       return_ACPI_STATUS (status);
+       return_ACPI_STATUS(status);
 }
 
-
-/*****************************************************************************
+/*******************************************************************************
  *
  * FUNCTION:    acpi_ex_create_event
  *
@@ -161,20 +157,16 @@ acpi_ex_create_alias (
  *
  * DESCRIPTION: Create a new event object
  *
- ****************************************************************************/
+ ******************************************************************************/
 
-acpi_status
-acpi_ex_create_event (
-       struct acpi_walk_state          *walk_state)
+acpi_status acpi_ex_create_event(struct acpi_walk_state *walk_state)
 {
-       acpi_status                     status;
-       union acpi_operand_object       *obj_desc;
-
-
-       ACPI_FUNCTION_TRACE ("ex_create_event");
+       acpi_status status;
+       union acpi_operand_object *obj_desc;
 
+       ACPI_FUNCTION_TRACE("ex_create_event");
 
-       obj_desc = acpi_ut_create_internal_object (ACPI_TYPE_EVENT);
+       obj_desc = acpi_ut_create_internal_object(ACPI_TYPE_EVENT);
        if (!obj_desc) {
                status = AE_NO_MEMORY;
                goto cleanup;
@@ -184,28 +176,28 @@ acpi_ex_create_event (
         * Create the actual OS semaphore, with zero initial units -- meaning
         * that the event is created in an unsignalled state
         */
-       status = acpi_os_create_semaphore (ACPI_NO_UNIT_LIMIT, 0,
-                          &obj_desc->event.semaphore);
-       if (ACPI_FAILURE (status)) {
+       status = acpi_os_create_semaphore(ACPI_NO_UNIT_LIMIT, 0,
+                                         &obj_desc->event.semaphore);
+       if (ACPI_FAILURE(status)) {
                goto cleanup;
        }
 
        /* Attach object to the Node */
 
-       status = acpi_ns_attach_object ((struct acpi_namespace_node *) walk_state->operands[0],
-                          obj_desc, ACPI_TYPE_EVENT);
+       status =
+           acpi_ns_attach_object((struct acpi_namespace_node *)walk_state->
+                                 operands[0], obj_desc, ACPI_TYPE_EVENT);
 
-cleanup:
+      cleanup:
        /*
         * Remove local reference to the object (on error, will cause deletion
         * of both object and semaphore if present.)
         */
-       acpi_ut_remove_reference (obj_desc);
-       return_ACPI_STATUS (status);
+       acpi_ut_remove_reference(obj_desc);
+       return_ACPI_STATUS(status);
 }
 
-
-/*****************************************************************************
+/*******************************************************************************
  *
  * FUNCTION:    acpi_ex_create_mutex
  *
@@ -217,22 +209,18 @@ cleanup:
  *
  *              Mutex (Name[0], sync_level[1])
  *
- ****************************************************************************/
+ ******************************************************************************/
 
-acpi_status
-acpi_ex_create_mutex (
-       struct acpi_walk_state          *walk_state)
+acpi_status acpi_ex_create_mutex(struct acpi_walk_state *walk_state)
 {
-       acpi_status                     status = AE_OK;
-       union acpi_operand_object       *obj_desc;
-
-
-       ACPI_FUNCTION_TRACE_PTR ("ex_create_mutex", ACPI_WALK_OPERANDS);
+       acpi_status status = AE_OK;
+       union acpi_operand_object *obj_desc;
 
+       ACPI_FUNCTION_TRACE_PTR("ex_create_mutex", ACPI_WALK_OPERANDS);
 
        /* Create the new mutex object */
 
-       obj_desc = acpi_ut_create_internal_object (ACPI_TYPE_MUTEX);
+       obj_desc = acpi_ut_create_internal_object(ACPI_TYPE_MUTEX);
        if (!obj_desc) {
                status = AE_NO_MEMORY;
                goto cleanup;
@@ -243,60 +231,56 @@ acpi_ex_create_mutex (
         * One unit max to make it a mutex, with one initial unit to allow
         * the mutex to be acquired.
         */
-       status = acpi_os_create_semaphore (1, 1, &obj_desc->mutex.semaphore);
-       if (ACPI_FAILURE (status)) {
+       status = acpi_os_create_semaphore(1, 1, &obj_desc->mutex.semaphore);
+       if (ACPI_FAILURE(status)) {
                goto cleanup;
        }
 
        /* Init object and attach to NS node */
 
-       obj_desc->mutex.sync_level = (u8) walk_state->operands[1]->integer.value;
-       obj_desc->mutex.node = (struct acpi_namespace_node *) walk_state->operands[0];
-
-       status = acpi_ns_attach_object (obj_desc->mutex.node,
-                        obj_desc, ACPI_TYPE_MUTEX);
+       obj_desc->mutex.sync_level =
+           (u8) walk_state->operands[1]->integer.value;
+       obj_desc->mutex.node =
+           (struct acpi_namespace_node *)walk_state->operands[0];
 
+       status = acpi_ns_attach_object(obj_desc->mutex.node,
+                                      obj_desc, ACPI_TYPE_MUTEX);
 
-cleanup:
+      cleanup:
        /*
         * Remove local reference to the object (on error, will cause deletion
         * of both object and semaphore if present.)
         */
-       acpi_ut_remove_reference (obj_desc);
-       return_ACPI_STATUS (status);
+       acpi_ut_remove_reference(obj_desc);
+       return_ACPI_STATUS(status);
 }
 
-
-/*****************************************************************************
+/*******************************************************************************
  *
  * FUNCTION:    acpi_ex_create_region
  *
  * PARAMETERS:  aml_start           - Pointer to the region declaration AML
  *              aml_length          - Max length of the declaration AML
- *              Operands            - List of operands for the opcode
+ *              region_space        - space_iD for the region
  *              walk_state          - Current state
  *
  * RETURN:      Status
  *
  * DESCRIPTION: Create a new operation region object
  *
- ****************************************************************************/
+ ******************************************************************************/
 
 acpi_status
-acpi_ex_create_region (
-       u8                              *aml_start,
-       u32                             aml_length,
-       u8                              region_space,
-       struct acpi_walk_state          *walk_state)
+acpi_ex_create_region(u8 * aml_start,
+                     u32 aml_length,
+                     u8 region_space, struct acpi_walk_state *walk_state)
 {
-       acpi_status                     status;
-       union acpi_operand_object       *obj_desc;
-       struct acpi_namespace_node      *node;
-       union acpi_operand_object       *region_obj2;
-
-
-       ACPI_FUNCTION_TRACE ("ex_create_region");
+       acpi_status status;
+       union acpi_operand_object *obj_desc;
+       struct acpi_namespace_node *node;
+       union acpi_operand_object *region_obj2;
 
+       ACPI_FUNCTION_TRACE("ex_create_region");
 
        /* Get the Namespace Node */
 
@@ -306,8 +290,8 @@ acpi_ex_create_region (
         * If the region object is already attached to this node,
         * just return
         */
-       if (acpi_ns_get_attached_object (node)) {
-               return_ACPI_STATUS (AE_OK);
+       if (acpi_ns_get_attached_object(node)) {
+               return_ACPI_STATUS(AE_OK);
        }
 
        /*
@@ -315,17 +299,18 @@ acpi_ex_create_region (
         * range
         */
        if ((region_space >= ACPI_NUM_PREDEFINED_REGIONS) &&
-               (region_space < ACPI_USER_REGION_BEGIN)) {
-               ACPI_REPORT_ERROR (("Invalid address_space type %X\n", region_space));
-               return_ACPI_STATUS (AE_AML_INVALID_SPACE_ID);
+           (region_space < ACPI_USER_REGION_BEGIN)) {
+               ACPI_ERROR((AE_INFO, "Invalid address_space type %X",
+                           region_space));
+               return_ACPI_STATUS(AE_AML_INVALID_SPACE_ID);
        }
 
-       ACPI_DEBUG_PRINT ((ACPI_DB_LOAD, "Region Type - %s (%X)\n",
-                         acpi_ut_get_region_name (region_space), region_space));
+       ACPI_DEBUG_PRINT((ACPI_DB_LOAD, "Region Type - %s (%X)\n",
+                         acpi_ut_get_region_name(region_space), region_space));
 
        /* Create the region descriptor */
 
-       obj_desc = acpi_ut_create_internal_object (ACPI_TYPE_REGION);
+       obj_desc = acpi_ut_create_internal_object(ACPI_TYPE_REGION);
        if (!obj_desc) {
                status = AE_NO_MEMORY;
                goto cleanup;
@@ -335,7 +320,7 @@ acpi_ex_create_region (
         * Remember location in AML stream of address & length
         * operands since they need to be evaluated at run time.
         */
-       region_obj2                 = obj_desc->common.next_object;
+       region_obj2 = obj_desc->common.next_object;
        region_obj2->extra.aml_start = aml_start;
        region_obj2->extra.aml_length = aml_length;
 
@@ -344,23 +329,21 @@ acpi_ex_create_region (
        obj_desc->region.space_id = region_space;
        obj_desc->region.address = 0;
        obj_desc->region.length = 0;
-       obj_desc->region.node   = node;
+       obj_desc->region.node = node;
 
        /* Install the new region object in the parent Node */
 
-       status = acpi_ns_attach_object (node, obj_desc, ACPI_TYPE_REGION);
-
+       status = acpi_ns_attach_object(node, obj_desc, ACPI_TYPE_REGION);
 
-cleanup:
+      cleanup:
 
        /* Remove local reference to the object */
 
-       acpi_ut_remove_reference (obj_desc);
-       return_ACPI_STATUS (status);
+       acpi_ut_remove_reference(obj_desc);
+       return_ACPI_STATUS(status);
 }
 
-
-/*****************************************************************************
+/*******************************************************************************
  *
  * FUNCTION:    acpi_ex_create_table_region
  *
@@ -370,22 +353,18 @@ cleanup:
  *
  * DESCRIPTION: Create a new data_table_region object
  *
- ****************************************************************************/
+ ******************************************************************************/
 
-acpi_status
-acpi_ex_create_table_region (
-       struct acpi_walk_state          *walk_state)
+acpi_status acpi_ex_create_table_region(struct acpi_walk_state *walk_state)
 {
-       acpi_status                     status;
-       union acpi_operand_object       **operand = &walk_state->operands[0];
-       union acpi_operand_object       *obj_desc;
-       struct acpi_namespace_node      *node;
-       struct acpi_table_header        *table;
-       union acpi_operand_object       *region_obj2;
-
-
-       ACPI_FUNCTION_TRACE ("ex_create_table_region");
+       acpi_status status;
+       union acpi_operand_object **operand = &walk_state->operands[0];
+       union acpi_operand_object *obj_desc;
+       struct acpi_namespace_node *node;
+       struct acpi_table_header *table;
+       union acpi_operand_object *region_obj2;
 
+       ACPI_FUNCTION_TRACE("ex_create_table_region");
 
        /* Get the Node from the object stack  */
 
@@ -395,67 +374,65 @@ acpi_ex_create_table_region (
         * If the region object is already attached to this node,
         * just return
         */
-       if (acpi_ns_get_attached_object (node)) {
-               return_ACPI_STATUS (AE_OK);
+       if (acpi_ns_get_attached_object(node)) {
+               return_ACPI_STATUS(AE_OK);
        }
 
        /* Find the ACPI table */
 
-       status = acpi_tb_find_table (operand[1]->string.pointer,
-                          operand[2]->string.pointer,
-                          operand[3]->string.pointer, &table);
-       if (ACPI_FAILURE (status)) {
-               return_ACPI_STATUS (status);
+       status = acpi_tb_find_table(operand[1]->string.pointer,
+                                   operand[2]->string.pointer,
+                                   operand[3]->string.pointer, &table);
+       if (ACPI_FAILURE(status)) {
+               return_ACPI_STATUS(status);
        }
 
        /* Create the region descriptor */
 
-       obj_desc = acpi_ut_create_internal_object (ACPI_TYPE_REGION);
+       obj_desc = acpi_ut_create_internal_object(ACPI_TYPE_REGION);
        if (!obj_desc) {
-               return_ACPI_STATUS (AE_NO_MEMORY);
+               return_ACPI_STATUS(AE_NO_MEMORY);
        }
 
-       region_obj2                     = obj_desc->common.next_object;
+       region_obj2 = obj_desc->common.next_object;
        region_obj2->extra.region_context = NULL;
 
        /* Init the region from the operands */
 
        obj_desc->region.space_id = REGION_DATA_TABLE;
-       obj_desc->region.address = (acpi_physical_address) ACPI_TO_INTEGER (table);
+       obj_desc->region.address =
+           (acpi_physical_address) ACPI_TO_INTEGER(table);
        obj_desc->region.length = table->length;
-       obj_desc->region.node   = node;
-       obj_desc->region.flags  = AOPOBJ_DATA_VALID;
+       obj_desc->region.node = node;
+       obj_desc->region.flags = AOPOBJ_DATA_VALID;
 
        /* Install the new region object in the parent Node */
 
-       status = acpi_ns_attach_object (node, obj_desc, ACPI_TYPE_REGION);
-       if (ACPI_FAILURE (status)) {
+       status = acpi_ns_attach_object(node, obj_desc, ACPI_TYPE_REGION);
+       if (ACPI_FAILURE(status)) {
                goto cleanup;
        }
 
-       status = acpi_ev_initialize_region (obj_desc, FALSE);
-       if (ACPI_FAILURE (status)) {
+       status = acpi_ev_initialize_region(obj_desc, FALSE);
+       if (ACPI_FAILURE(status)) {
                if (status == AE_NOT_EXIST) {
                        status = AE_OK;
-               }
-               else {
+               } else {
                        goto cleanup;
                }
        }
 
        obj_desc->region.flags |= AOPOBJ_SETUP_COMPLETE;
 
-
-cleanup:
+      cleanup:
 
        /* Remove local reference to the object */
 
-       acpi_ut_remove_reference (obj_desc);
-       return_ACPI_STATUS (status);
+       acpi_ut_remove_reference(obj_desc);
+       return_ACPI_STATUS(status);
 }
 
-
-/*****************************************************************************
+/*******************************************************************************
  *
  * FUNCTION:    acpi_ex_create_processor
  *
@@ -467,47 +444,42 @@ cleanup:
  *
  *              Processor (Name[0], cpu_iD[1], pblock_addr[2], pblock_length[3])
  *
- ****************************************************************************/
+ ******************************************************************************/
 
-acpi_status
-acpi_ex_create_processor (
-       struct acpi_walk_state          *walk_state)
+acpi_status acpi_ex_create_processor(struct acpi_walk_state *walk_state)
 {
-       union acpi_operand_object       **operand = &walk_state->operands[0];
-       union acpi_operand_object       *obj_desc;
-       acpi_status                     status;
-
-
-       ACPI_FUNCTION_TRACE_PTR ("ex_create_processor", walk_state);
+       union acpi_operand_object **operand = &walk_state->operands[0];
+       union acpi_operand_object *obj_desc;
+       acpi_status status;
 
+       ACPI_FUNCTION_TRACE_PTR("ex_create_processor", walk_state);
 
        /* Create the processor object */
 
-       obj_desc = acpi_ut_create_internal_object (ACPI_TYPE_PROCESSOR);
+       obj_desc = acpi_ut_create_internal_object(ACPI_TYPE_PROCESSOR);
        if (!obj_desc) {
-               return_ACPI_STATUS (AE_NO_MEMORY);
+               return_ACPI_STATUS(AE_NO_MEMORY);
        }
 
-       /*
-        * Initialize the processor object from the operands
-        */
-       obj_desc->processor.proc_id = (u8)          operand[1]->integer.value;
-       obj_desc->processor.address = (acpi_io_address) operand[2]->integer.value;
-       obj_desc->processor.length = (u8)           operand[3]->integer.value;
+       /* Initialize the processor object from the operands */
+
+       obj_desc->processor.proc_id = (u8) operand[1]->integer.value;
+       obj_desc->processor.address =
+           (acpi_io_address) operand[2]->integer.value;
+       obj_desc->processor.length = (u8) operand[3]->integer.value;
 
        /* Install the processor object in the parent Node */
 
-       status = acpi_ns_attach_object ((struct acpi_namespace_node *) operand[0],
-                         obj_desc, ACPI_TYPE_PROCESSOR);
+       status = acpi_ns_attach_object((struct acpi_namespace_node *)operand[0],
+                                      obj_desc, ACPI_TYPE_PROCESSOR);
 
        /* Remove local reference to the object */
 
-       acpi_ut_remove_reference (obj_desc);
-       return_ACPI_STATUS (status);
+       acpi_ut_remove_reference(obj_desc);
+       return_ACPI_STATUS(status);
 }
 
-
-/*****************************************************************************
+/*******************************************************************************
  *
  * FUNCTION:    acpi_ex_create_power_resource
  *
@@ -519,46 +491,42 @@ acpi_ex_create_processor (
  *
  *              power_resource (Name[0], system_level[1], resource_order[2])
  *
- ****************************************************************************/
+ ******************************************************************************/
 
-acpi_status
-acpi_ex_create_power_resource (
-       struct acpi_walk_state          *walk_state)
+acpi_status acpi_ex_create_power_resource(struct acpi_walk_state *walk_state)
 {
-       union acpi_operand_object       **operand = &walk_state->operands[0];
-       acpi_status                     status;
-       union acpi_operand_object       *obj_desc;
-
-
-       ACPI_FUNCTION_TRACE_PTR ("ex_create_power_resource", walk_state);
+       union acpi_operand_object **operand = &walk_state->operands[0];
+       acpi_status status;
+       union acpi_operand_object *obj_desc;
 
+       ACPI_FUNCTION_TRACE_PTR("ex_create_power_resource", walk_state);
 
        /* Create the power resource object */
 
-       obj_desc = acpi_ut_create_internal_object (ACPI_TYPE_POWER);
+       obj_desc = acpi_ut_create_internal_object(ACPI_TYPE_POWER);
        if (!obj_desc) {
-               return_ACPI_STATUS (AE_NO_MEMORY);
+               return_ACPI_STATUS(AE_NO_MEMORY);
        }
 
        /* Initialize the power object from the operands */
 
        obj_desc->power_resource.system_level = (u8) operand[1]->integer.value;
-       obj_desc->power_resource.resource_order = (u16) operand[2]->integer.value;
+       obj_desc->power_resource.resource_order =
+           (u16) operand[2]->integer.value;
 
        /* Install the  power resource object in the parent Node */
 
-       status = acpi_ns_attach_object ((struct acpi_namespace_node *) operand[0],
-                         obj_desc, ACPI_TYPE_POWER);
+       status = acpi_ns_attach_object((struct acpi_namespace_node *)operand[0],
+                                      obj_desc, ACPI_TYPE_POWER);
 
        /* Remove local reference to the object */
 
-       acpi_ut_remove_reference (obj_desc);
-       return_ACPI_STATUS (status);
+       acpi_ut_remove_reference(obj_desc);
+       return_ACPI_STATUS(status);
 }
-
 #endif
 
-/*****************************************************************************
+/*******************************************************************************
  *
  * FUNCTION:    acpi_ex_create_method
  *
@@ -570,28 +538,24 @@ acpi_ex_create_power_resource (
  *
  * DESCRIPTION: Create a new method object
  *
- ****************************************************************************/
+ ******************************************************************************/
 
 acpi_status
-acpi_ex_create_method (
-       u8                              *aml_start,
-       u32                             aml_length,
-       struct acpi_walk_state          *walk_state)
+acpi_ex_create_method(u8 * aml_start,
+                     u32 aml_length, struct acpi_walk_state *walk_state)
 {
-       union acpi_operand_object       **operand = &walk_state->operands[0];
-       union acpi_operand_object       *obj_desc;
-       acpi_status                     status;
-       u8                              method_flags;
-
-
-       ACPI_FUNCTION_TRACE_PTR ("ex_create_method", walk_state);
+       union acpi_operand_object **operand = &walk_state->operands[0];
+       union acpi_operand_object *obj_desc;
+       acpi_status status;
+       u8 method_flags;
 
+       ACPI_FUNCTION_TRACE_PTR("ex_create_method", walk_state);
 
        /* Create a new method object */
 
-       obj_desc = acpi_ut_create_internal_object (ACPI_TYPE_METHOD);
+       obj_desc = acpi_ut_create_internal_object(ACPI_TYPE_METHOD);
        if (!obj_desc) {
-          return_ACPI_STATUS (AE_NO_MEMORY);
+               return_ACPI_STATUS(AE_NO_MEMORY);
        }
 
        /* Save the method's AML pointer and length  */
@@ -605,8 +569,10 @@ acpi_ex_create_method (
         */
        method_flags = (u8) operand[1]->integer.value;
 
-       obj_desc->method.method_flags = (u8) (method_flags & ~AML_METHOD_ARG_COUNT);
-       obj_desc->method.param_count = (u8) (method_flags & AML_METHOD_ARG_COUNT);
+       obj_desc->method.method_flags =
+           (u8) (method_flags & ~AML_METHOD_ARG_COUNT);
+       obj_desc->method.param_count =
+           (u8) (method_flags & AML_METHOD_ARG_COUNT);
 
        /*
         * Get the concurrency count.  If required, a semaphore will be
@@ -615,32 +581,28 @@ acpi_ex_create_method (
        if (acpi_gbl_all_methods_serialized) {
                obj_desc->method.concurrency = 1;
                obj_desc->method.method_flags |= AML_METHOD_SERIALIZED;
-       }
-       else if (method_flags & AML_METHOD_SERIALIZED) {
+       } else if (method_flags & AML_METHOD_SERIALIZED) {
                /*
                 * ACPI 1.0: Concurrency = 1
                 * ACPI 2.0: Concurrency = (sync_level (in method declaration) + 1)
                 */
                obj_desc->method.concurrency = (u8)
-                                 (((method_flags & AML_METHOD_SYNCH_LEVEL) >> 4) + 1);
-       }
-       else {
+                   (((method_flags & AML_METHOD_SYNCH_LEVEL) >> 4) + 1);
+       } else {
                obj_desc->method.concurrency = ACPI_INFINITE_CONCURRENCY;
        }
 
        /* Attach the new object to the method Node */
 
-       status = acpi_ns_attach_object ((struct acpi_namespace_node *) operand[0],
-                         obj_desc, ACPI_TYPE_METHOD);
+       status = acpi_ns_attach_object((struct acpi_namespace_node *)operand[0],
+                                      obj_desc, ACPI_TYPE_METHOD);
 
        /* Remove local reference to the object */
 
-       acpi_ut_remove_reference (obj_desc);
+       acpi_ut_remove_reference(obj_desc);
 
        /* Remove a reference to the operand */
 
-       acpi_ut_remove_reference (operand[1]);
-       return_ACPI_STATUS (status);
+       acpi_ut_remove_reference(operand[1]);
+       return_ACPI_STATUS(status);
 }
-
-