diff options
author | jkim <jkim@FreeBSD.org> | 2013-01-28 21:10:35 +0000 |
---|---|---|
committer | jkim <jkim@FreeBSD.org> | 2013-01-28 21:10:35 +0000 |
commit | 33e791eff7de52b00254c418cb1a2c85b8f16af2 (patch) | |
tree | 32dbee86536dcfc6d35bc89768f50f8c00e0f592 /sys/dev/acpica | |
parent | 93443619fc1ff66a407b86a70546fc5ec2281367 (diff) | |
download | FreeBSD-src-33e791eff7de52b00254c418cb1a2c85b8f16af2.zip FreeBSD-src-33e791eff7de52b00254c418cb1a2c85b8f16af2.tar.gz |
Fix two misusages of return_*() macros for ACPICA.
Noticed by: avg
Diffstat (limited to 'sys/dev/acpica')
-rw-r--r-- | sys/dev/acpica/Osd/OsdSchedule.c | 4 | ||||
-rw-r--r-- | sys/dev/acpica/acpi_pcib.c | 4 |
2 files changed, 6 insertions, 2 deletions
diff --git a/sys/dev/acpica/Osd/OsdSchedule.c b/sys/dev/acpica/Osd/OsdSchedule.c index 641d696..4afcec5 100644 --- a/sys/dev/acpica/Osd/OsdSchedule.c +++ b/sys/dev/acpica/Osd/OsdSchedule.c @@ -181,6 +181,7 @@ ACPI_STATUS AcpiOsExecute(ACPI_EXECUTE_TYPE Type, ACPI_OSD_EXEC_CALLBACK Function, void *Context) { + ACPI_STATUS status; int pri; ACPI_FUNCTION_TRACE((char *)(uintptr_t)__func__); @@ -211,7 +212,8 @@ AcpiOsExecute(ACPI_EXECUTE_TYPE Type, ACPI_OSD_EXEC_CALLBACK Function, return_ACPI_STATUS(AE_BAD_PARAMETER); } - return_ACPI_STATUS(acpi_task_enqueue(pri, Function, Context)); + status = acpi_task_enqueue(pri, Function, Context); + return_ACPI_STATUS(status); } void diff --git a/sys/dev/acpica/acpi_pcib.c b/sys/dev/acpica/acpi_pcib.c index eba4a62..1a42d74 100644 --- a/sys/dev/acpica/acpi_pcib.c +++ b/sys/dev/acpica/acpi_pcib.c @@ -130,6 +130,7 @@ int acpi_pcib_attach(device_t dev, ACPI_BUFFER *prt, int busno) { ACPI_STATUS status; + int error; ACPI_FUNCTION_TRACE((char *)(uintptr_t)__func__); @@ -168,7 +169,8 @@ acpi_pcib_attach(device_t dev, ACPI_BUFFER *prt, int busno) */ prt_walk_table(prt, prt_attach_devices, dev); - return_VALUE(bus_generic_attach(dev)); + error = bus_generic_attach(dev); + return_VALUE(error); } static void |