ACPI: Change the ordering of acpi_bus_check_add()
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 20 Dec 2012 23:36:44 +0000 (00:36 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 3 Jan 2013 12:09:38 +0000 (13:09 +0100)
If acpi_bus_check_add() is called for a handle already having an
existing struct acpi_device object attached, it is not necessary to
check the type and status of the device correspondig to it, so
change the ordering of acpi_bus_check_add() to avoid that.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Acked-by: Yinghai Lu <yinghai@kernel.org>
Acked-by: Toshi Kani <toshi.kani@hp.com>
drivers/acpi/scan.c

index 7082d11ce5d2bb9fb9d12908cdefd14fad7ed6a8..eb7ecb1f20324588721316875990b6f8aaf18e6b 100644 (file)
@@ -1582,6 +1582,10 @@ static acpi_status acpi_bus_check_add(acpi_handle handle, u32 lvl,
        acpi_status status;
        int result;
 
+       acpi_bus_get_device(handle, &device);
+       if (device)
+               goto out;
+
        result = acpi_bus_type_and_status(handle, &type, &sta);
        if (result)
                return AE_OK;
@@ -1598,18 +1602,14 @@ static acpi_status acpi_bus_check_add(acpi_handle handle, u32 lvl,
                return AE_CTRL_DEPTH;
        }
 
-       acpi_bus_get_device(handle, &device);
-       if (!device) {
-               acpi_add_single_object(&device, handle, type, sta,
-                                      ACPI_BUS_ADD_BASIC);
-               if (!device)
-                       return AE_CTRL_DEPTH;
+       acpi_add_single_object(&device, handle, type, sta, ACPI_BUS_ADD_BASIC);
+       if (!device)
+               return AE_CTRL_DEPTH;
 
-               device->add_type = context ?
-                                       ACPI_BUS_ADD_START : ACPI_BUS_ADD_MATCH;
-               acpi_hot_add_bind(device);
-       }
+       device->add_type = context ? ACPI_BUS_ADD_START : ACPI_BUS_ADD_MATCH;
+       acpi_hot_add_bind(device);
 
+ out:
        if (!*return_value)
                *return_value = device;