diff options
| author | Yijing Wang <wangyijing@huawei.com> | 2013-11-20 17:28:23 +0800 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2013-11-20 08:31:01 -0800 |
| commit | 3a4cbc190643d0c6f7ca36a84af8367947f558c7 (patch) | |
| tree | c64260637872287617d9cfaf962b975730d4ec87 /drivers | |
| parent | cc76dee17fc996cf23c3cebe543a2d1ea36aa89e (diff) | |
hwmon: (acpi_power_meter) Fix acpi_bus_get_device() return value check
Since acpi_bus_get_device() returns plain int and not acpi_status,
ACPI_FAILURE() should not be used for checking its return value. Fix
that.
Signed-off-by: Yijing Wang <wangyijing@huawei.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/hwmon/acpi_power_meter.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/hwmon/acpi_power_meter.c b/drivers/hwmon/acpi_power_meter.c index 8d40da314a8..6a34f7f48eb 100644 --- a/drivers/hwmon/acpi_power_meter.c +++ b/drivers/hwmon/acpi_power_meter.c @@ -602,9 +602,8 @@ static int read_domain_devices(struct acpi_power_meter_resource *resource) /* Create a symlink to domain objects */ resource->domain_devices[i] = NULL; - status = acpi_bus_get_device(element->reference.handle, - &resource->domain_devices[i]); - if (ACPI_FAILURE(status)) + if (acpi_bus_get_device(element->reference.handle, + &resource->domain_devices[i])) continue; obj = resource->domain_devices[i]; |