diff options
| author |   Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-11-14 00:54:00 +0100 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-11-19 01:02:22 +0100 |
| commit | 5beaee4f195d0297473da9a45459050b561e10cf (patch) | |
| tree | 2d72e0f132bdae893097d60a8018c6678f79cd2a /drivers | |
| parent | ACPI / hotplug: Consolidate deferred execution of ACPI hotplug routines (diff) | |
| download | linux-dev-5beaee4f195d0297473da9a45459050b561e10cf.tar.xz linux-dev-5beaee4f195d0297473da9a45459050b561e10cf.zip | |
ACPI / hotplug: Fix acpi_bus_get_device() return value check
Since acpi_bus_get_device() returns a plain int and not acpi_status,
ACPI_FAILURE() should not be used for checking its return value. Fix
that.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Acked-by: Toshi Kani <toshi.kani@hp.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/acpi/scan.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c index 276cde70a514..4698060ba3f0 100644 --- a/drivers/acpi/scan.c +++ b/drivers/acpi/scan.c @@ -411,8 +411,7 @@ static void acpi_hotplug_notify_cb(acpi_handle handle, u32 type, void *data) break; case ACPI_NOTIFY_EJECT_REQUEST: acpi_handle_debug(handle, "ACPI_NOTIFY_EJECT_REQUEST event\n"); - status = acpi_bus_get_device(handle, &adev); - if (ACPI_FAILURE(status)) + if (acpi_bus_get_device(handle, &adev)) goto err_out; get_device(&adev->dev); |