8 files changed, 31 insertions, 44 deletions

diff --git a/drivers/acpi/parser/psargs.c b/drivers/acpi/parser/psargs.c
index 1c5934fe1853..bf88e076c3e9 100644
--- a/drivers/acpi/parser/psargs.c
+++ b/drivers/acpi/parser/psargs.c
@@ -79,7 +79,7 @@ acpi_ps_get_next_package_length(struct acpi_parse_state *parser_state)
         acpi_native_uint byte_count;
         u8 byte_zero_mask = 0x3F;       /* Default [0:5] */
 
-        ACPI_FUNCTION_TRACE("ps_get_next_package_length");
+        ACPI_FUNCTION_TRACE(ps_get_next_package_length);
 
         /*
          * Byte 0 bits [6:7] contain the number of additional bytes
@@ -128,7 +128,7 @@ u8 *acpi_ps_get_next_package_end(struct acpi_parse_state *parser_state)
         u8 *start = parser_state->aml;
         u32 package_length;
 
-        ACPI_FUNCTION_TRACE("ps_get_next_package_end");
+        ACPI_FUNCTION_TRACE(ps_get_next_package_end);
 
         /* Function below updates parser_state->Aml */
 
@@ -157,7 +157,7 @@ char *acpi_ps_get_next_namestring(struct acpi_parse_state *parser_state)
         u8 *start = parser_state->aml;
         u8 *end = parser_state->aml;
 
-        ACPI_FUNCTION_TRACE("ps_get_next_namestring");
+        ACPI_FUNCTION_TRACE(ps_get_next_namestring);
 
         /* Point past any namestring prefix characters (backslash or carat) */
 
@@ -237,7 +237,7 @@ acpi_ps_get_next_namepath(struct acpi_walk_state *walk_state,
         struct acpi_namespace_node *node;
         union acpi_generic_state scope_info;
 
-        ACPI_FUNCTION_TRACE("ps_get_next_namepath");
+        ACPI_FUNCTION_TRACE(ps_get_next_namepath);
 
         path = acpi_ps_get_next_namestring(parser_state);
         acpi_ps_init_op(arg, AML_INT_NAMEPATH_OP);
@@ -391,7 +391,7 @@ acpi_ps_get_next_simple_arg(struct acpi_parse_state *parser_state,
         u16 opcode;
         u8 *aml = parser_state->aml;
 
-        ACPI_FUNCTION_TRACE_U32("ps_get_next_simple_arg", arg_type);
+        ACPI_FUNCTION_TRACE_U32(ps_get_next_simple_arg, arg_type);
 
         switch (arg_type) {
         case ARGP_BYTEDATA:
@@ -456,7 +456,7 @@ acpi_ps_get_next_simple_arg(struct acpi_parse_state *parser_state,
 
         default:
 
-                ACPI_ERROR((AE_INFO, "Invalid arg_type %X", arg_type));
+                ACPI_ERROR((AE_INFO, "Invalid ArgType %X", arg_type));
                 return_VOID;
         }
 
@@ -487,7 +487,7 @@ static union acpi_parse_object *acpi_ps_get_next_field(struct acpi_parse_state
         u16 opcode;
         u32 name;
 
-        ACPI_FUNCTION_TRACE("ps_get_next_field");
+        ACPI_FUNCTION_TRACE(ps_get_next_field);
 
         /* Determine field type */
 
@@ -593,7 +593,7 @@ acpi_ps_get_next_arg(struct acpi_walk_state *walk_state,
         u32 subop;
         acpi_status status = AE_OK;
 
-        ACPI_FUNCTION_TRACE_PTR("ps_get_next_arg", parser_state);
+        ACPI_FUNCTION_TRACE_PTR(ps_get_next_arg, parser_state);
 
         switch (arg_type) {
         case ARGP_BYTEDATA:
@@ -718,7 +718,7 @@ acpi_ps_get_next_arg(struct acpi_walk_state *walk_state,
 
         default:
 
-                ACPI_ERROR((AE_INFO, "Invalid arg_type: %X", arg_type));
+                ACPI_ERROR((AE_INFO, "Invalid ArgType: %X", arg_type));
                 status = AE_AML_OPERAND_TYPE;
                 break;
         }
diff --git a/drivers/acpi/parser/psloop.c b/drivers/acpi/parser/psloop.c
index 3a29ce680370..e1541db3753a 100644
--- a/drivers/acpi/parser/psloop.c
+++ b/drivers/acpi/parser/psloop.c
@@ -83,7 +83,7 @@ acpi_status acpi_ps_parse_loop(struct acpi_walk_state *walk_state)
         struct acpi_parse_state *parser_state;
         u8 *aml_op_start = NULL;
 
-        ACPI_FUNCTION_TRACE_PTR("ps_parse_loop", walk_state);
+        ACPI_FUNCTION_TRACE_PTR(ps_parse_loop, walk_state);
 
         if (walk_state->descending_callback == NULL) {
                 return_ACPI_STATUS(AE_BAD_PARAMETER);
@@ -129,7 +129,7 @@ acpi_status acpi_ps_parse_loop(struct acpi_walk_state *walk_state)
 
                                         }
                                         ACPI_EXCEPTION((AE_INFO, status,
-                                                        "get_predicate Failed"));
+                                                        "GetPredicate Failed"));
                                         return_ACPI_STATUS(status);
                                 }
 
@@ -375,7 +375,7 @@ acpi_status acpi_ps_parse_loop(struct acpi_walk_state *walk_state)
 
                         if (walk_state->op_info) {
                                 ACPI_DEBUG_PRINT((ACPI_DB_PARSE,
-                                                  "Opcode %4.4X [%s] Op %p Aml %p aml_offset %5.5X\n",
+                                                  "Opcode %4.4X [%s] Op %p Aml %p AmlOffset %5.5X\n",
                                                   (u32) op->common.aml_opcode,
                                                   walk_state->op_info->name, op,
                                                   parser_state->aml,
diff --git a/drivers/acpi/parser/psopcode.c b/drivers/acpi/parser/psopcode.c
index 9eb8d0663886..4bd25e32769f 100644
--- a/drivers/acpi/parser/psopcode.c
+++ b/drivers/acpi/parser/psopcode.c
@@ -725,7 +725,7 @@ static const u8 acpi_gbl_long_op_index[NUM_EXTENDED_OPCODE] = {
 
 const struct acpi_opcode_info *acpi_ps_get_opcode_info(u16 opcode)
 {
-        ACPI_FUNCTION_NAME("ps_get_opcode_info");
+        ACPI_FUNCTION_NAME(ps_get_opcode_info);
 
         /*
          * Detect normal 8-bit opcode or extended 16-bit opcode
@@ -781,7 +781,7 @@ char *acpi_ps_get_opcode_name(u16 opcode)
         return (op->name);
 
 #else
-        return ("AE_NOT_CONFIGURED");
+        return ("OpcodeName unavailable");
 
 #endif
 }
diff --git a/drivers/acpi/parser/psparse.c b/drivers/acpi/parser/psparse.c
index 83d8916bc1af..796f6b719929 100644
--- a/drivers/acpi/parser/psparse.c
+++ b/drivers/acpi/parser/psparse.c
@@ -138,7 +138,7 @@ acpi_ps_complete_this_op(struct acpi_walk_state * walk_state,
         const struct acpi_opcode_info *parent_info;
         union acpi_parse_object *replacement_op = NULL;
 
-        ACPI_FUNCTION_TRACE_PTR("ps_complete_this_op", op);
+        ACPI_FUNCTION_TRACE_PTR(ps_complete_this_op, op);
 
         /* Check for null Op, can happen if AML code is corrupt */
 
@@ -333,7 +333,7 @@ acpi_ps_next_parse_state(struct acpi_walk_state *walk_state,
         struct acpi_parse_state *parser_state = &walk_state->parser_state;
         acpi_status status = AE_CTRL_PENDING;
 
-        ACPI_FUNCTION_TRACE_PTR("ps_next_parse_state", op);
+        ACPI_FUNCTION_TRACE_PTR(ps_next_parse_state, op);
 
         switch (callback_status) {
         case AE_CTRL_TERMINATE:
@@ -453,10 +453,10 @@ acpi_status acpi_ps_parse_aml(struct acpi_walk_state *walk_state)
         struct acpi_thread_state *prev_walk_list = acpi_gbl_current_walk_list;
         struct acpi_walk_state *previous_walk_state;
 
-        ACPI_FUNCTION_TRACE("ps_parse_aml");
+        ACPI_FUNCTION_TRACE(ps_parse_aml);
 
         ACPI_DEBUG_PRINT((ACPI_DB_PARSE,
-                          "Entered with walk_state=%p Aml=%p size=%X\n",
+                          "Entered with WalkState=%p Aml=%p size=%X\n",
                           walk_state, walk_state->parser_state.aml,
                           walk_state->parser_state.aml_size));
 
@@ -555,21 +555,9 @@ acpi_status acpi_ps_parse_aml(struct acpi_walk_state *walk_state)
                  */
                 if (((walk_state->parse_flags & ACPI_PARSE_MODE_MASK) ==
                      ACPI_PARSE_EXECUTE) || (ACPI_FAILURE(status))) {
-                        if (walk_state->method_desc) {
-
-                                /* Decrement the thread count on the method parse tree */
-
-                                if (walk_state->method_desc->method.
-                                    thread_count) {
-                                        walk_state->method_desc->method.
-                                            thread_count--;
-                                } else {
-                                        ACPI_ERROR((AE_INFO,
-                                                    "Invalid zero thread count in method"));
-                                }
-                        }
-
-                        acpi_ds_terminate_control_method(walk_state);
+                        acpi_ds_terminate_control_method(walk_state->
+                                                         method_desc,
+                                                         walk_state);
                 }
 
                 /* Delete this walk state and all linked control states */
@@ -578,7 +566,7 @@ acpi_status acpi_ps_parse_aml(struct acpi_walk_state *walk_state)
                 previous_walk_state = walk_state;
 
                 ACPI_DEBUG_PRINT((ACPI_DB_PARSE,
-                                  "return_value=%p, implicit_value=%p State=%p\n",
+                                  "ReturnValue=%p, ImplicitValue=%p State=%p\n",
                                   walk_state->return_desc,
                                   walk_state->implicit_return_obj, walk_state));
 
diff --git a/drivers/acpi/parser/psscope.c b/drivers/acpi/parser/psscope.c
index 424ab1c20da5..a3e0314de24d 100644
--- a/drivers/acpi/parser/psscope.c
+++ b/drivers/acpi/parser/psscope.c
@@ -106,7 +106,7 @@ acpi_ps_init_scope(struct acpi_parse_state * parser_state,
 {
         union acpi_generic_state *scope;
 
-        ACPI_FUNCTION_TRACE_PTR("ps_init_scope", root_op);
+        ACPI_FUNCTION_TRACE_PTR(ps_init_scope, root_op);
 
         scope = acpi_ut_create_generic_state();
         if (!scope) {
@@ -147,7 +147,7 @@ acpi_ps_push_scope(struct acpi_parse_state *parser_state,
 {
         union acpi_generic_state *scope;
 
-        ACPI_FUNCTION_TRACE_PTR("ps_push_scope", op);
+        ACPI_FUNCTION_TRACE_PTR(ps_push_scope, op);
 
         scope = acpi_ut_create_generic_state();
         if (!scope) {
@@ -200,7 +200,7 @@ acpi_ps_pop_scope(struct acpi_parse_state *parser_state,
 {
         union acpi_generic_state *scope = parser_state->scope;
 
-        ACPI_FUNCTION_TRACE("ps_pop_scope");
+        ACPI_FUNCTION_TRACE(ps_pop_scope);
 
         /* Only pop the scope if there is in fact a next scope */
 
@@ -247,7 +247,7 @@ void acpi_ps_cleanup_scope(struct acpi_parse_state *parser_state)
 {
         union acpi_generic_state *scope;
 
-        ACPI_FUNCTION_TRACE_PTR("ps_cleanup_scope", parser_state);
+        ACPI_FUNCTION_TRACE_PTR(ps_cleanup_scope, parser_state);
 
         if (!parser_state) {
                 return_VOID;
diff --git a/drivers/acpi/parser/psutils.c b/drivers/acpi/parser/psutils.c
index 43e3190583e3..182474ae8ce9 100644
--- a/drivers/acpi/parser/psutils.c
+++ b/drivers/acpi/parser/psutils.c
@@ -172,7 +172,7 @@ union acpi_parse_object *acpi_ps_alloc_op(u16 opcode)
 
 void acpi_ps_free_op(union acpi_parse_object *op)
 {
-        ACPI_FUNCTION_NAME("ps_free_op");
+        ACPI_FUNCTION_NAME(ps_free_op);
 
         if (op->common.aml_opcode == AML_INT_RETURN_VALUE_OP) {
                 ACPI_DEBUG_PRINT((ACPI_DB_ALLOCATIONS, "Free retval op: %p\n",
diff --git a/drivers/acpi/parser/pswalk.c b/drivers/acpi/parser/pswalk.c
index 3d08b8c6db41..a84a547a0f1b 100644
--- a/drivers/acpi/parser/pswalk.c
+++ b/drivers/acpi/parser/pswalk.c
@@ -64,7 +64,7 @@ void acpi_ps_delete_parse_tree(union acpi_parse_object *subtree_root)
         union acpi_parse_object *next = NULL;
         union acpi_parse_object *parent = NULL;
 
-        ACPI_FUNCTION_TRACE_PTR("ps_delete_parse_tree", subtree_root);
+        ACPI_FUNCTION_TRACE_PTR(ps_delete_parse_tree, subtree_root);
 
         /* Visit all nodes in the subtree */
 
diff --git a/drivers/acpi/parser/psxface.c b/drivers/acpi/parser/psxface.c
index e09a820ee294..6b1411f1087d 100644
--- a/drivers/acpi/parser/psxface.c
+++ b/drivers/acpi/parser/psxface.c
@@ -216,7 +216,7 @@ acpi_status acpi_ps_execute_method(struct acpi_parameter_info *info)
 {
         acpi_status status;
 
-        ACPI_FUNCTION_TRACE("ps_execute_method");
+        ACPI_FUNCTION_TRACE(ps_execute_method);
 
         /* Validate the Info and method Node */
 
@@ -286,8 +286,7 @@ acpi_status acpi_ps_execute_method(struct acpi_parameter_info *info)
          * a control exception code
          */
         if (info->return_object) {
-                ACPI_DEBUG_PRINT((ACPI_DB_PARSE,
-                                  "Method returned obj_desc=%p\n",
+                ACPI_DEBUG_PRINT((ACPI_DB_PARSE, "Method returned ObjDesc=%p\n",
                                   info->return_object));
                 ACPI_DUMP_STACK_ENTRY(info->return_object);
 
@@ -350,7 +349,7 @@ static acpi_status acpi_ps_execute_pass(struct acpi_parameter_info *info)
         union acpi_parse_object *op;
         struct acpi_walk_state *walk_state;
 
-        ACPI_FUNCTION_TRACE("ps_execute_pass");
+        ACPI_FUNCTION_TRACE(ps_execute_pass);
 
         /* Create and init a Root Node */