diff options
author | Yijing Wang <wangyijing@huawei.com> | 2013-11-20 04:28:23 -0500 |
---|---|---|
committer | Guenter Roeck <linux@roeck-us.net> | 2013-11-20 11:31:01 -0500 |
commit | 3a4cbc190643d0c6f7ca36a84af8367947f558c7 (patch) | |
tree | c64260637872287617d9cfaf962b975730d4ec87 | |
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>
-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 8d40da314a8e..6a34f7f48eb9 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) | |||
602 | 602 | ||
603 | /* Create a symlink to domain objects */ | 603 | /* Create a symlink to domain objects */ |
604 | resource->domain_devices[i] = NULL; | 604 | resource->domain_devices[i] = NULL; |
605 | status = acpi_bus_get_device(element->reference.handle, | 605 | if (acpi_bus_get_device(element->reference.handle, |
606 | &resource->domain_devices[i]); | 606 | &resource->domain_devices[i])) |
607 | if (ACPI_FAILURE(status)) | ||
608 | continue; | 607 | continue; |
609 | 608 | ||
610 | obj = resource->domain_devices[i]; | 609 | obj = resource->domain_devices[i]; |