fedora core 6 1.2949 + vserver 2.2.0
[linux-2.6.git] / drivers / acpi / utilities / uteval.c
index c3f5900..d6d7121 100644 (file)
@@ -5,7 +5,7 @@
  *****************************************************************************/
 
 /*
- * Copyright (C) 2000 - 2004, R. Byron Moore
+ * Copyright (C) 2000 - 2006, R. Byron Moore
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * POSSIBILITY OF SUCH DAMAGES.
  */
 
-
 #include <acpi/acpi.h>
 #include <acpi/acnamesp.h>
 #include <acpi/acinterp.h>
 
-
 #define _COMPONENT          ACPI_UTILITIES
-        ACPI_MODULE_NAME    ("uteval")
+ACPI_MODULE_NAME("uteval")
+
+/* Local prototypes */
+static void
+acpi_ut_copy_id_string(char *destination, char *source, acpi_size max_length);
 
+static acpi_status
+acpi_ut_translate_one_cid(union acpi_operand_object *obj_desc,
+                         struct acpi_compatible_id *one_cid);
+
+/*
+ * Strings supported by the _OSI predefined (internal) method.
+ */
+static const char *acpi_interfaces_supported[] = {
+       /* Operating System Vendor Strings */
+
+       "Linux",
+       "Windows 2000",
+       "Windows 2001",
+       "Windows 2001 SP0",
+       "Windows 2001 SP1",
+       "Windows 2001 SP2",
+       "Windows 2001 SP3",
+       "Windows 2001 SP4",
+       "Windows 2001.1",
+       "Windows 2001.1 SP1",   /* Added 03/2006 */
+       "Windows 2006",         /* Added 03/2006 */
+
+       /* Feature Group Strings */
+
+       "Extended Address Space Descriptor"
+           /*
+            * All "optional" feature group strings (features that are implemented
+            * by the host) should be implemented in the host version of
+            * acpi_os_validate_interface and should not be added here.
+            */
+};
 
 /*******************************************************************************
  *
  *
  * RETURN:      Status
  *
- * DESCRIPTION: Implementation of _OSI predefined control method
- *              Supported = _OSI (String)
+ * DESCRIPTION: Implementation of the _OSI predefined control method
  *
  ******************************************************************************/
 
-acpi_status
-acpi_ut_osi_implementation (
-       struct acpi_walk_state          *walk_state)
+acpi_status acpi_ut_osi_implementation(struct acpi_walk_state *walk_state)
 {
-       union acpi_operand_object       *string_desc;
-       union acpi_operand_object       *return_desc;
-       acpi_native_uint                i;
-
-
-       ACPI_FUNCTION_TRACE ("ut_osi_implementation");
+       acpi_status status;
+       union acpi_operand_object *string_desc;
+       union acpi_operand_object *return_desc;
+       acpi_native_uint i;
 
+       ACPI_FUNCTION_TRACE(ut_osi_implementation);
 
        /* Validate the string input argument */
 
        string_desc = walk_state->arguments[0].object;
        if (!string_desc || (string_desc->common.type != ACPI_TYPE_STRING)) {
-               return_ACPI_STATUS (AE_TYPE);
+               return_ACPI_STATUS(AE_TYPE);
        }
 
-       /* Create a return object (Default value = 0) */
+       /* Create a return object */
 
-       return_desc = acpi_ut_create_internal_object (ACPI_TYPE_INTEGER);
+       return_desc = acpi_ut_create_internal_object(ACPI_TYPE_INTEGER);
        if (!return_desc) {
-               return_ACPI_STATUS (AE_NO_MEMORY);
+               return_ACPI_STATUS(AE_NO_MEMORY);
        }
 
-       /* Compare input string to table of supported strings */
+       /* Default return value is SUPPORTED */
+
+       return_desc->integer.value = ACPI_UINT32_MAX;
+       walk_state->return_desc = return_desc;
+
+       /* Compare input string to static table of supported interfaces */
 
-       for (i = 0; i < ACPI_NUM_OSI_STRINGS; i++) {
-               if (!ACPI_STRCMP (string_desc->string.pointer,
-                                  (char *) acpi_gbl_valid_osi_strings[i])) {
-                       /* This string is supported */
+       for (i = 0; i < ACPI_ARRAY_LENGTH(acpi_interfaces_supported); i++) {
+               if (!ACPI_STRCMP
+                   (string_desc->string.pointer,
+                    acpi_interfaces_supported[i])) {
 
-                       return_desc->integer.value = 0xFFFFFFFF;
-                       break;
+                       /* The interface is supported */
+
+                       return_ACPI_STATUS(AE_CTRL_TERMINATE);
                }
        }
 
-       walk_state->return_desc = return_desc;
-       return_ACPI_STATUS (AE_CTRL_TERMINATE);
-}
+       /*
+        * Did not match the string in the static table, call the host OSL to
+        * check for a match with one of the optional strings (such as
+        * "Module Device", "3.0 Thermal Model", etc.)
+        */
+       status = acpi_os_validate_interface(string_desc->string.pointer);
+       if (ACPI_SUCCESS(status)) {
+
+               /* The interface is supported */
 
+               return_ACPI_STATUS(AE_CTRL_TERMINATE);
+       }
+
+       /* The interface is not supported */
+
+       return_desc->integer.value = 0;
+       return_ACPI_STATUS(AE_CTRL_TERMINATE);
+}
 
 /*******************************************************************************
  *
@@ -127,52 +176,61 @@ acpi_ut_osi_implementation (
  ******************************************************************************/
 
 acpi_status
-acpi_ut_evaluate_object (
-       struct acpi_namespace_node      *prefix_node,
-       char                            *path,
-       u32                             expected_return_btypes,
-       union acpi_operand_object       **return_desc)
+acpi_ut_evaluate_object(struct acpi_namespace_node *prefix_node,
+                       char *path,
+                       u32 expected_return_btypes,
+                       union acpi_operand_object **return_desc)
 {
-       union acpi_operand_object       *obj_desc;
-       acpi_status                     status;
-       u32                             return_btype;
+       struct acpi_evaluate_info *info;
+       acpi_status status;
+       u32 return_btype;
 
+       ACPI_FUNCTION_TRACE(ut_evaluate_object);
 
-       ACPI_FUNCTION_TRACE ("ut_evaluate_object");
+       /* Allocate the evaluation information block */
 
+       info = ACPI_ALLOCATE_ZEROED(sizeof(struct acpi_evaluate_info));
+       if (!info) {
+               return_ACPI_STATUS(AE_NO_MEMORY);
+       }
+
+       info->prefix_node = prefix_node;
+       info->pathname = path;
+       info->parameter_type = ACPI_PARAM_ARGS;
 
        /* Evaluate the object/method */
 
-       status = acpi_ns_evaluate_relative (prefix_node, path, NULL, &obj_desc);
-       if (ACPI_FAILURE (status)) {
+       status = acpi_ns_evaluate(info);
+       if (ACPI_FAILURE(status)) {
                if (status == AE_NOT_FOUND) {
-                       ACPI_DEBUG_PRINT ((ACPI_DB_EXEC, "[%4.4s.%s] was not found\n",
-                               acpi_ut_get_node_name (prefix_node), path));
-               }
-               else {
-                       ACPI_REPORT_METHOD_ERROR ("Method execution failed",
-                               prefix_node, path, status);
+                       ACPI_DEBUG_PRINT((ACPI_DB_EXEC,
+                                         "[%4.4s.%s] was not found\n",
+                                         acpi_ut_get_node_name(prefix_node),
+                                         path));
+               } else {
+                       ACPI_ERROR_METHOD("Method execution failed",
+                                         prefix_node, path, status);
                }
 
-               return_ACPI_STATUS (status);
+               goto cleanup;
        }
 
        /* Did we get a return object? */
 
-       if (!obj_desc) {
+       if (!info->return_object) {
                if (expected_return_btypes) {
-                       ACPI_REPORT_METHOD_ERROR ("No object was returned from",
-                               prefix_node, path, AE_NOT_EXIST);
+                       ACPI_ERROR_METHOD("No object was returned from",
+                                         prefix_node, path, AE_NOT_EXIST);
 
-                       return_ACPI_STATUS (AE_NOT_EXIST);
+                       status = AE_NOT_EXIST;
                }
 
-               return_ACPI_STATUS (AE_OK);
+               goto cleanup;
        }
 
        /* Map the return object type to the bitmapped type */
 
-       switch (ACPI_GET_OBJECT_TYPE (obj_desc)) {
+       switch (ACPI_GET_OBJECT_TYPE(info->return_object)) {
        case ACPI_TYPE_INTEGER:
                return_btype = ACPI_BTYPE_INTEGER;
                break;
@@ -194,36 +252,51 @@ acpi_ut_evaluate_object (
                break;
        }
 
+       if ((acpi_gbl_enable_interpreter_slack) && (!expected_return_btypes)) {
+               /*
+                * We received a return object, but one was not expected.  This can
+                * happen frequently if the "implicit return" feature is enabled.
+                * Just delete the return object and return AE_OK.
+                */
+               acpi_ut_remove_reference(info->return_object);
+               goto cleanup;
+       }
+
        /* Is the return object one of the expected types? */
 
        if (!(expected_return_btypes & return_btype)) {
-               ACPI_REPORT_METHOD_ERROR ("Return object type is incorrect",
-                       prefix_node, path, AE_TYPE);
+               ACPI_ERROR_METHOD("Return object type is incorrect",
+                                 prefix_node, path, AE_TYPE);
 
-               ACPI_DEBUG_PRINT ((ACPI_DB_ERROR,
-                       "Type returned from %s was incorrect: %X\n",
-                       path, ACPI_GET_OBJECT_TYPE (obj_desc)));
+               ACPI_ERROR((AE_INFO,
+                           "Type returned from %s was incorrect: %s, expected Btypes: %X",
+                           path,
+                           acpi_ut_get_object_type_name(info->return_object),
+                           expected_return_btypes));
 
                /* On error exit, we must delete the return object */
 
-               acpi_ut_remove_reference (obj_desc);
-               return_ACPI_STATUS (AE_TYPE);
+               acpi_ut_remove_reference(info->return_object);
+               status = AE_TYPE;
+               goto cleanup;
        }
 
        /* Object type is OK, return it */
 
-       *return_desc = obj_desc;
-       return_ACPI_STATUS (AE_OK);
-}
+       *return_desc = info->return_object;
 
+      cleanup:
+       ACPI_FREE(info);
+       return_ACPI_STATUS(status);
+}
 
 /*******************************************************************************
  *
  * FUNCTION:    acpi_ut_evaluate_numeric_object
  *
- * PARAMETERS:  *object_name        - Object name to be evaluated
+ * PARAMETERS:  object_name         - Object name to be evaluated
  *              device_node         - Node for the device
- *              *Address            - Where the value is returned
+ *              Address             - Where the value is returned
  *
  * RETURN:      Status
  *
@@ -235,22 +308,19 @@ acpi_ut_evaluate_object (
  ******************************************************************************/
 
 acpi_status
-acpi_ut_evaluate_numeric_object (
-       char                            *object_name,
-       struct acpi_namespace_node      *device_node,
-       acpi_integer                    *address)
+acpi_ut_evaluate_numeric_object(char *object_name,
+                               struct acpi_namespace_node *device_node,
+                               acpi_integer * address)
 {
-       union acpi_operand_object       *obj_desc;
-       acpi_status                     status;
+       union acpi_operand_object *obj_desc;
+       acpi_status status;
 
+       ACPI_FUNCTION_TRACE(ut_evaluate_numeric_object);
 
-       ACPI_FUNCTION_TRACE ("ut_evaluate_numeric_object");
-
-
-       status = acpi_ut_evaluate_object (device_node, object_name,
-                        ACPI_BTYPE_INTEGER, &obj_desc);
-       if (ACPI_FAILURE (status)) {
-               return_ACPI_STATUS (status);
+       status = acpi_ut_evaluate_object(device_node, object_name,
+                                        ACPI_BTYPE_INTEGER, &obj_desc);
+       if (ACPI_FAILURE(status)) {
+               return_ACPI_STATUS(status);
        }
 
        /* Get the returned Integer */
@@ -259,11 +329,10 @@ acpi_ut_evaluate_numeric_object (
 
        /* On exit, we must delete the return object */
 
-       acpi_ut_remove_reference (obj_desc);
-       return_ACPI_STATUS (status);
+       acpi_ut_remove_reference(obj_desc);
+       return_ACPI_STATUS(status);
 }
 
-
 /*******************************************************************************
  *
  * FUNCTION:    acpi_ut_copy_id_string
@@ -281,13 +350,9 @@ acpi_ut_evaluate_numeric_object (
  ******************************************************************************/
 
 static void
-acpi_ut_copy_id_string (
-       char                            *destination,
-       char                            *source,
-       acpi_size                       max_length)
+acpi_ut_copy_id_string(char *destination, char *source, acpi_size max_length)
 {
 
-
        /*
         * Workaround for ID strings that have a leading asterisk. This construct
         * is not allowed by the ACPI specification  (ID strings must be
@@ -300,16 +365,15 @@ acpi_ut_copy_id_string (
 
        /* Do the actual copy */
 
-       ACPI_STRNCPY (destination, source, max_length);
+       ACPI_STRNCPY(destination, source, max_length);
 }
 
-
 /*******************************************************************************
  *
  * FUNCTION:    acpi_ut_execute_HID
  *
  * PARAMETERS:  device_node         - Node for the device
- *              *Hid                - Where the HID is returned
+ *              Hid                 - Where the HID is returned
  *
  * RETURN:      Status
  *
@@ -321,42 +385,40 @@ acpi_ut_copy_id_string (
  ******************************************************************************/
 
 acpi_status
-acpi_ut_execute_HID (
-       struct acpi_namespace_node      *device_node,
-       struct acpi_device_id           *hid)
+acpi_ut_execute_HID(struct acpi_namespace_node *device_node,
+                   struct acpi_device_id *hid)
 {
-       union acpi_operand_object       *obj_desc;
-       acpi_status                     status;
-
+       union acpi_operand_object *obj_desc;
+       acpi_status status;
 
-       ACPI_FUNCTION_TRACE ("ut_execute_HID");
+       ACPI_FUNCTION_TRACE(ut_execute_HID);
 
-
-       status = acpi_ut_evaluate_object (device_node, METHOD_NAME__HID,
-                        ACPI_BTYPE_INTEGER | ACPI_BTYPE_STRING, &obj_desc);
-       if (ACPI_FAILURE (status)) {
-               return_ACPI_STATUS (status);
+       status = acpi_ut_evaluate_object(device_node, METHOD_NAME__HID,
+                                        ACPI_BTYPE_INTEGER | ACPI_BTYPE_STRING,
+                                        &obj_desc);
+       if (ACPI_FAILURE(status)) {
+               return_ACPI_STATUS(status);
        }
 
-       if (ACPI_GET_OBJECT_TYPE (obj_desc) == ACPI_TYPE_INTEGER) {
+       if (ACPI_GET_OBJECT_TYPE(obj_desc) == ACPI_TYPE_INTEGER) {
+
                /* Convert the Numeric HID to string */
 
-               acpi_ex_eisa_id_to_string ((u32) obj_desc->integer.value, hid->value);
-       }
-       else {
+               acpi_ex_eisa_id_to_string((u32) obj_desc->integer.value,
+                                         hid->value);
+       else {
                /* Copy the String HID from the returned object */
 
-               acpi_ut_copy_id_string (hid->value, obj_desc->string.pointer,
-                               sizeof (hid->value));
+               acpi_ut_copy_id_string(hid->value, obj_desc->string.pointer,
+                                      sizeof(hid->value));
        }
 
        /* On exit, we must delete the return object */
 
-       acpi_ut_remove_reference (obj_desc);
-       return_ACPI_STATUS (status);
+       acpi_ut_remove_reference(obj_desc);
+       return_ACPI_STATUS(status);
 }
 
-
 /*******************************************************************************
  *
  * FUNCTION:    acpi_ut_translate_one_cid
@@ -375,18 +437,17 @@ acpi_ut_execute_HID (
  ******************************************************************************/
 
 static acpi_status
-acpi_ut_translate_one_cid (
-       union acpi_operand_object       *obj_desc,
-       struct acpi_compatible_id       *one_cid)
+acpi_ut_translate_one_cid(union acpi_operand_object *obj_desc,
+                         struct acpi_compatible_id *one_cid)
 {
 
-
-       switch (ACPI_GET_OBJECT_TYPE (obj_desc)) {
+       switch (ACPI_GET_OBJECT_TYPE(obj_desc)) {
        case ACPI_TYPE_INTEGER:
 
                /* Convert the Numeric CID to string */
 
-               acpi_ex_eisa_id_to_string ((u32) obj_desc->integer.value, one_cid->value);
+               acpi_ex_eisa_id_to_string((u32) obj_desc->integer.value,
+                                         one_cid->value);
                return (AE_OK);
 
        case ACPI_TYPE_STRING:
@@ -397,8 +458,8 @@ acpi_ut_translate_one_cid (
 
                /* Copy the String CID from the returned object */
 
-               acpi_ut_copy_id_string (one_cid->value, obj_desc->string.pointer,
-                               ACPI_MAX_CID_LENGTH);
+               acpi_ut_copy_id_string(one_cid->value, obj_desc->string.pointer,
+                                      ACPI_MAX_CID_LENGTH);
                return (AE_OK);
 
        default:
@@ -407,13 +468,12 @@ acpi_ut_translate_one_cid (
        }
 }
 
-
 /*******************************************************************************
  *
  * FUNCTION:    acpi_ut_execute_CID
  *
  * PARAMETERS:  device_node         - Node for the device
- *              *Cid                - Where the CID is returned
+ *              return_cid_list     - Where the CID list is returned
  *
  * RETURN:      Status
  *
@@ -425,45 +485,42 @@ acpi_ut_translate_one_cid (
  ******************************************************************************/
 
 acpi_status
-acpi_ut_execute_CID (
-       struct acpi_namespace_node      *device_node,
-       struct acpi_compatible_id_list **return_cid_list)
+acpi_ut_execute_CID(struct acpi_namespace_node * device_node,
+                   struct acpi_compatible_id_list ** return_cid_list)
 {
-       union acpi_operand_object       *obj_desc;
-       acpi_status                     status;
-       u32                             count;
-       u32                             size;
+       union acpi_operand_object *obj_desc;
+       acpi_status status;
+       u32 count;
+       u32 size;
        struct acpi_compatible_id_list *cid_list;
-       acpi_native_uint                i;
-
-
-       ACPI_FUNCTION_TRACE ("ut_execute_CID");
+       acpi_native_uint i;
 
+       ACPI_FUNCTION_TRACE(ut_execute_CID);
 
        /* Evaluate the _CID method for this device */
 
-       status = acpi_ut_evaluate_object (device_node, METHOD_NAME__CID,
-                        ACPI_BTYPE_INTEGER | ACPI_BTYPE_STRING | ACPI_BTYPE_PACKAGE,
-                        &obj_desc);
-       if (ACPI_FAILURE (status)) {
-               return_ACPI_STATUS (status);
+       status = acpi_ut_evaluate_object(device_node, METHOD_NAME__CID,
+                                        ACPI_BTYPE_INTEGER | ACPI_BTYPE_STRING
+                                        | ACPI_BTYPE_PACKAGE, &obj_desc);
+       if (ACPI_FAILURE(status)) {
+               return_ACPI_STATUS(status);
        }
 
        /* Get the number of _CIDs returned */
 
        count = 1;
-       if (ACPI_GET_OBJECT_TYPE (obj_desc) == ACPI_TYPE_PACKAGE) {
+       if (ACPI_GET_OBJECT_TYPE(obj_desc) == ACPI_TYPE_PACKAGE) {
                count = obj_desc->package.count;
        }
 
        /* Allocate a worst-case buffer for the _CIDs */
 
-       size = (((count - 1) * sizeof (struct acpi_compatible_id)) +
-                          sizeof (struct acpi_compatible_id_list));
+       size = (((count - 1) * sizeof(struct acpi_compatible_id)) +
+               sizeof(struct acpi_compatible_id_list));
 
-       cid_list = ACPI_MEM_CALLOCATE ((acpi_size) size);
+       cid_list = ACPI_ALLOCATE_ZEROED((acpi_size) size);
        if (!cid_list) {
-               return_ACPI_STATUS (AE_NO_MEMORY);
+               return_ACPI_STATUS(AE_NO_MEMORY);
        }
 
        /* Init CID list */
@@ -472,53 +529,53 @@ acpi_ut_execute_CID (
        cid_list->size = size;
 
        /*
-        *  A _CID can return either a single compatible ID or a package of compatible
-        *  IDs.  Each compatible ID can be one of the following:
-        *  -- Number (32 bit compressed EISA ID) or
-        *  -- String (PCI ID format, e.g. "PCI\VEN_vvvv&DEV_dddd&SUBSYS_ssssssss").
+        *  A _CID can return either a single compatible ID or a package of
+        *  compatible IDs.  Each compatible ID can be one of the following:
+        *  1) Integer (32 bit compressed EISA ID) or
+        *  2) String (PCI ID format, e.g. "PCI\VEN_vvvv&DEV_dddd&SUBSYS_ssssssss")
         */
 
        /* The _CID object can be either a single CID or a package (list) of CIDs */
 
-       if (ACPI_GET_OBJECT_TYPE (obj_desc) == ACPI_TYPE_PACKAGE) {
+       if (ACPI_GET_OBJECT_TYPE(obj_desc) == ACPI_TYPE_PACKAGE) {
+
                /* Translate each package element */
 
                for (i = 0; i < count; i++) {
-                       status = acpi_ut_translate_one_cid (obj_desc->package.elements[i],
-                                         &cid_list->id[i]);
-                       if (ACPI_FAILURE (status)) {
+                       status =
+                           acpi_ut_translate_one_cid(obj_desc->package.
+                                                     elements[i],
+                                                     &cid_list->id[i]);
+                       if (ACPI_FAILURE(status)) {
                                break;
                        }
                }
-       }
-       else {
+       } else {
                /* Only one CID, translate to a string */
 
-               status = acpi_ut_translate_one_cid (obj_desc, cid_list->id);
+               status = acpi_ut_translate_one_cid(obj_desc, cid_list->id);
        }
 
        /* Cleanup on error */
 
-       if (ACPI_FAILURE (status)) {
-               ACPI_MEM_FREE (cid_list);
-       }
-       else {
+       if (ACPI_FAILURE(status)) {
+               ACPI_FREE(cid_list);
+       } else {
                *return_cid_list = cid_list;
        }
 
        /* On exit, we must delete the _CID return object */
 
-       acpi_ut_remove_reference (obj_desc);
-       return_ACPI_STATUS (status);
+       acpi_ut_remove_reference(obj_desc);
+       return_ACPI_STATUS(status);
 }
 
-
 /*******************************************************************************
  *
  * FUNCTION:    acpi_ut_execute_UID
  *
  * PARAMETERS:  device_node         - Node for the device
- *              *Uid                - Where the UID is returned
+ *              Uid                 - Where the UID is returned
  *
  * RETURN:      Status
  *
@@ -530,48 +587,46 @@ acpi_ut_execute_CID (
  ******************************************************************************/
 
 acpi_status
-acpi_ut_execute_UID (
-       struct acpi_namespace_node      *device_node,
-       struct acpi_device_id           *uid)
+acpi_ut_execute_UID(struct acpi_namespace_node *device_node,
+                   struct acpi_device_id *uid)
 {
-       union acpi_operand_object       *obj_desc;
-       acpi_status                     status;
-
+       union acpi_operand_object *obj_desc;
+       acpi_status status;
 
-       ACPI_FUNCTION_TRACE ("ut_execute_UID");
+       ACPI_FUNCTION_TRACE(ut_execute_UID);
 
-
-       status = acpi_ut_evaluate_object (device_node, METHOD_NAME__UID,
-                        ACPI_BTYPE_INTEGER | ACPI_BTYPE_STRING, &obj_desc);
-       if (ACPI_FAILURE (status)) {
-               return_ACPI_STATUS (status);
+       status = acpi_ut_evaluate_object(device_node, METHOD_NAME__UID,
+                                        ACPI_BTYPE_INTEGER | ACPI_BTYPE_STRING,
+                                        &obj_desc);
+       if (ACPI_FAILURE(status)) {
+               return_ACPI_STATUS(status);
        }
 
-       if (ACPI_GET_OBJECT_TYPE (obj_desc) == ACPI_TYPE_INTEGER) {
+       if (ACPI_GET_OBJECT_TYPE(obj_desc) == ACPI_TYPE_INTEGER) {
+
                /* Convert the Numeric UID to string */
 
-               acpi_ex_unsigned_integer_to_string (obj_desc->integer.value, uid->value);
-       }
-       else {
+               acpi_ex_unsigned_integer_to_string(obj_desc->integer.value,
+                                                  uid->value);
+       else {
                /* Copy the String UID from the returned object */
 
-               acpi_ut_copy_id_string (uid->value, obj_desc->string.pointer,
-                               sizeof (uid->value));
+               acpi_ut_copy_id_string(uid->value, obj_desc->string.pointer,
+                                      sizeof(uid->value));
        }
 
        /* On exit, we must delete the return object */
 
-       acpi_ut_remove_reference (obj_desc);
-       return_ACPI_STATUS (status);
+       acpi_ut_remove_reference(obj_desc);
+       return_ACPI_STATUS(status);
 }
 
-
 /*******************************************************************************
  *
  * FUNCTION:    acpi_ut_execute_STA
  *
  * PARAMETERS:  device_node         - Node for the device
- *              *Flags              - Where the status flags are returned
+ *              Flags               - Where the status flags are returned
  *
  * RETURN:      Status
  *
@@ -583,30 +638,26 @@ acpi_ut_execute_UID (
  ******************************************************************************/
 
 acpi_status
-acpi_ut_execute_STA (
-       struct acpi_namespace_node      *device_node,
-       u32                             *flags)
+acpi_ut_execute_STA(struct acpi_namespace_node *device_node, u32 * flags)
 {
-       union acpi_operand_object       *obj_desc;
-       acpi_status                     status;
+       union acpi_operand_object *obj_desc;
+       acpi_status status;
 
+       ACPI_FUNCTION_TRACE(ut_execute_STA);
 
-       ACPI_FUNCTION_TRACE ("ut_execute_STA");
-
-
-       status = acpi_ut_evaluate_object (device_node, METHOD_NAME__STA,
-                        ACPI_BTYPE_INTEGER, &obj_desc);
-       if (ACPI_FAILURE (status)) {
+       status = acpi_ut_evaluate_object(device_node, METHOD_NAME__STA,
+                                        ACPI_BTYPE_INTEGER, &obj_desc);
+       if (ACPI_FAILURE(status)) {
                if (AE_NOT_FOUND == status) {
-                       ACPI_DEBUG_PRINT ((ACPI_DB_EXEC,
-                               "_STA on %4.4s was not found, assuming device is present\n",
-                               acpi_ut_get_node_name (device_node)));
+                       ACPI_DEBUG_PRINT((ACPI_DB_EXEC,
+                                         "_STA on %4.4s was not found, assuming device is present\n",
+                                         acpi_ut_get_node_name(device_node)));
 
-                       *flags = 0x0F;
+                       *flags = ACPI_UINT32_MAX;
                        status = AE_OK;
                }
 
-               return_ACPI_STATUS (status);
+               return_ACPI_STATUS(status);
        }
 
        /* Extract the status flags */
@@ -615,17 +666,16 @@ acpi_ut_execute_STA (
 
        /* On exit, we must delete the return object */
 
-       acpi_ut_remove_reference (obj_desc);
-       return_ACPI_STATUS (status);
+       acpi_ut_remove_reference(obj_desc);
+       return_ACPI_STATUS(status);
 }
 
-
 /*******************************************************************************
  *
  * FUNCTION:    acpi_ut_execute_Sxds
  *
  * PARAMETERS:  device_node         - Node for the device
- *              *Flags              - Where the status flags are returned
+ *              Flags               - Where the status flags are returned
  *
  * RETURN:      Status
  *
@@ -637,44 +687,45 @@ acpi_ut_execute_STA (
  ******************************************************************************/
 
 acpi_status
-acpi_ut_execute_sxds (
-       struct acpi_namespace_node      *device_node,
-       u8                              *highest)
+acpi_ut_execute_sxds(struct acpi_namespace_node *device_node, u8 * highest)
 {
-       union acpi_operand_object       *obj_desc;
-       acpi_status                     status;
-       u32                             i;
-
-
-       ACPI_FUNCTION_TRACE ("ut_execute_Sxds");
+       union acpi_operand_object *obj_desc;
+       acpi_status status;
+       u32 i;
 
+       ACPI_FUNCTION_TRACE(ut_execute_sxds);
 
        for (i = 0; i < 4; i++) {
                highest[i] = 0xFF;
-               status = acpi_ut_evaluate_object (device_node,
-                                (char *) acpi_gbl_highest_dstate_names[i],
-                                ACPI_BTYPE_INTEGER, &obj_desc);
-               if (ACPI_FAILURE (status)) {
+               status = acpi_ut_evaluate_object(device_node,
+                                                ACPI_CAST_PTR(char,
+                                                              acpi_gbl_highest_dstate_names
+                                                              [i]),
+                                                ACPI_BTYPE_INTEGER, &obj_desc);
+               if (ACPI_FAILURE(status)) {
                        if (status != AE_NOT_FOUND) {
-                               ACPI_DEBUG_PRINT ((ACPI_DB_EXEC,
-                                       "%s on Device %4.4s, %s\n",
-                                       (char *) acpi_gbl_highest_dstate_names[i],
-                                       acpi_ut_get_node_name (device_node),
-                                       acpi_format_exception (status)));
-
-                               return_ACPI_STATUS (status);
+                               ACPI_DEBUG_PRINT((ACPI_DB_EXEC,
+                                                 "%s on Device %4.4s, %s\n",
+                                                 ACPI_CAST_PTR(char,
+                                                               acpi_gbl_highest_dstate_names
+                                                               [i]),
+                                                 acpi_ut_get_node_name
+                                                 (device_node),
+                                                 acpi_format_exception
+                                                 (status)));
+
+                               return_ACPI_STATUS(status);
                        }
-               }
-               else {
+               } else {
                        /* Extract the Dstate value */
 
                        highest[i] = (u8) obj_desc->integer.value;
 
                        /* Delete the return object */
 
-                       acpi_ut_remove_reference (obj_desc);
+                       acpi_ut_remove_reference(obj_desc);
                }
        }
 
-       return_ACPI_STATUS (AE_OK);
+       return_ACPI_STATUS(AE_OK);
 }