ACPICA: Eliminate extra call to NsGetParentNode
authorAlex Chiang <achiang@hp.com>
Thu, 21 May 2009 02:59:15 +0000 (10:59 +0800)
committerLen Brown <len.brown@intel.com>
Wed, 27 May 2009 04:35:51 +0000 (00:35 -0400)
Was called twice in the same function with the same parameter.
Alex Chiang.

Signed-off-by: Alex Chiang <achiang@hp.com>
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Lin Ming <ming.m.lin@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
drivers/acpi/acpica/nsxfobj.c

index dd33d8b53d6d261c1960b11ef3c348c40fd6478a..4071bad4458ed7d229bc04e30cb02abcb8efe26b 100644 (file)
@@ -162,6 +162,7 @@ ACPI_EXPORT_SYMBOL(acpi_get_type)
 acpi_status acpi_get_parent(acpi_handle handle, acpi_handle * ret_handle)
 {
        struct acpi_namespace_node *node;
+       struct acpi_namespace_node *parent_node;
        acpi_status status;
 
        if (!ret_handle) {
@@ -189,12 +190,12 @@ acpi_status acpi_get_parent(acpi_handle handle, acpi_handle * ret_handle)
 
        /* Get the parent entry */
 
-       *ret_handle =
-           acpi_ns_convert_entry_to_handle(acpi_ns_get_parent_node(node));
+       parent_node = acpi_ns_get_parent_node(node);
+       *ret_handle = acpi_ns_convert_entry_to_handle(parent_node);
 
        /* Return exception if parent is null */
 
-       if (!acpi_ns_get_parent_node(node)) {
+       if (!parent_node) {
                status = AE_NULL_ENTRY;
        }