1 files changed, 16 insertions, 9 deletions

diff --git a/drivers/acpi/dispatcher/dsobject.c b/drivers/acpi/dispatcher/dsobject.c
index 8b21f0f9e517..72190abb1d59 100644
--- a/drivers/acpi/dispatcher/dsobject.c
+++ b/drivers/acpi/dispatcher/dsobject.c
@@ -81,7 +81,7 @@ acpi_ds_build_internal_object(struct acpi_walk_state *walk_state,
         union acpi_operand_object *obj_desc;
         acpi_status status;
 
-        ACPI_FUNCTION_TRACE("ds_build_internal_object");
+        ACPI_FUNCTION_TRACE(ds_build_internal_object);
 
         *obj_desc_ptr = NULL;
         if (op->common.aml_opcode == AML_INT_NAMEPATH_OP) {
@@ -103,6 +103,7 @@ acpi_ds_build_internal_object(struct acpi_walk_state *walk_state,
                                                                          common.
                                                                          node)));
                         if (ACPI_FAILURE(status)) {
+
                                 /* Check if we are resolving a named reference within a package */
 
                                 if ((status == AE_NOT_FOUND)
@@ -186,7 +187,7 @@ acpi_ds_build_internal_buffer_obj(struct acpi_walk_state *walk_state,
         union acpi_parse_object *byte_list;
         u32 byte_list_length = 0;
 
-        ACPI_FUNCTION_TRACE("ds_build_internal_buffer_obj");
+        ACPI_FUNCTION_TRACE(ds_build_internal_buffer_obj);
 
         /*
          * If we are evaluating a Named buffer object "Name (xxxx, Buffer)".
@@ -195,6 +196,7 @@ acpi_ds_build_internal_buffer_obj(struct acpi_walk_state *walk_state,
          */
         obj_desc = *obj_desc_ptr;
         if (!obj_desc) {
+
                 /* Create a new buffer object */
 
                 obj_desc = acpi_ut_create_internal_object(ACPI_TYPE_BUFFER);
@@ -243,7 +245,7 @@ acpi_ds_build_internal_buffer_obj(struct acpi_walk_state *walk_state,
                                   "Buffer defined with zero length in AML, creating\n"));
         } else {
                 obj_desc->buffer.pointer =
-                    ACPI_MEM_CALLOCATE(obj_desc->buffer.length);
+                    ACPI_ALLOCATE_ZEROED(obj_desc->buffer.length);
                 if (!obj_desc->buffer.pointer) {
                         acpi_ut_delete_object_desc(obj_desc);
                         return_ACPI_STATUS(AE_NO_MEMORY);
@@ -291,7 +293,7 @@ acpi_ds_build_internal_package_obj(struct acpi_walk_state *walk_state,
         acpi_status status = AE_OK;
         acpi_native_uint i;
 
-        ACPI_FUNCTION_TRACE("ds_build_internal_package_obj");
+        ACPI_FUNCTION_TRACE(ds_build_internal_package_obj);
 
         /* Find the parent of a possibly nested package */
 
@@ -339,9 +341,10 @@ acpi_ds_build_internal_package_obj(struct acpi_walk_state *walk_state,
          * individual objects). Add an extra pointer slot so
          * that the list is always null terminated.
          */
-        obj_desc->package.elements = ACPI_MEM_CALLOCATE(((acpi_size) obj_desc->
-                                                         package.count +
-                                                         1) * sizeof(void *));
+        obj_desc->package.elements = ACPI_ALLOCATE_ZEROED(((acpi_size)
+                                                           obj_desc->package.
+                                                           count +
+                                                           1) * sizeof(void *));
 
         if (!obj_desc->package.elements) {
                 acpi_ut_delete_object_desc(obj_desc);
@@ -355,6 +358,7 @@ acpi_ds_build_internal_package_obj(struct acpi_walk_state *walk_state,
         arg = arg->common.next;
         for (i = 0; arg; i++) {
                 if (arg->common.aml_opcode == AML_INT_RETURN_VALUE_OP) {
+
                         /* Object (package or buffer) is already built */
 
                         obj_desc->package.elements[i] =
@@ -396,7 +400,7 @@ acpi_ds_create_node(struct acpi_walk_state *walk_state,
         acpi_status status;
         union acpi_operand_object *obj_desc;
 
-        ACPI_FUNCTION_TRACE_PTR("ds_create_node", op);
+        ACPI_FUNCTION_TRACE_PTR(ds_create_node, op);
 
         /*
          * Because of the execution pass through the non-control-method
@@ -408,6 +412,7 @@ acpi_ds_create_node(struct acpi_walk_state *walk_state,
         }
 
         if (!op->common.value.arg) {
+
                 /* No arguments, there is nothing to do */
 
                 return_ACPI_STATUS(AE_OK);
@@ -464,11 +469,12 @@ acpi_ds_init_object_from_op(struct acpi_walk_state *walk_state,
         union acpi_operand_object *obj_desc;
         acpi_status status = AE_OK;
 
-        ACPI_FUNCTION_TRACE("ds_init_object_from_op");
+        ACPI_FUNCTION_TRACE(ds_init_object_from_op);
 
         obj_desc = *ret_obj_desc;
         op_info = acpi_ps_get_opcode_info(opcode);
         if (op_info->class == AML_CLASS_UNKNOWN) {
+
                 /* Unknown opcode */
 
                 return_ACPI_STATUS(AE_TYPE);
@@ -626,6 +632,7 @@ acpi_ds_init_object_from_op(struct acpi_walk_state *walk_state,
                 default:        /* Other literals, etc.. */
 
                         if (op->common.aml_opcode == AML_INT_NAMEPATH_OP) {
+
                                 /* Node was saved in Op */
 
                                 obj_desc->reference.node = op->common.node;