[Xen-changelog] [xen-4.2-testing] ACPI: acpi_table_parse() should return handler's error code

[Xen patchbot-4.2-testing <patchbot@xxxxxxx>]

# HG changeset patch
# User Boris Ostrovsky <boris.ostrovsky@xxxxxxx>
# Date 1360074543 -3600
# Node ID f3725a1da1931411ce9d8f2d2cc514794708c26f
# Parent  7c04074a0a0f897f6745fb3328746bc50bb91d71
ACPI: acpi_table_parse() should return handler's error code

Currently, the error code returned by acpi_table_parse()'s handler
is ignored. This patch will propagate handler's return value to
acpi_table_parse()'s caller.

Signed-off-by: Boris Ostrovsky <boris.ostrovsky@xxxxxxx>
Committed-by: Jan Beulich <jbeulich@xxxxxxxx>
xen-unstable changeset: 26516:32d4516a97f0
xen-unstable date: Tue Feb  5 14:18:18 UTC 2013
---


diff -r 7c04074a0a0f -r f3725a1da193 xen/drivers/acpi/tables.c
--- a/xen/drivers/acpi/tables.c Wed Jan 23 11:52:44 2013 +0100
+++ b/xen/drivers/acpi/tables.c Tue Feb 05 15:29:03 2013 +0100
@@ -267,7 +267,7 @@ acpi_table_parse_madt(enum acpi_madt_typ
  * @handler: handler to run
  *
  * Scan the ACPI System Descriptor Table (STD) for a table matching @id,
- * run @handler on it.  Return 0 if table found, return on if not.
+ * run @handler on it.
  */
 int __init acpi_table_parse(char *id, acpi_table_handler handler)
 {
@@ -282,8 +282,7 @@ int __init acpi_table_parse(char *id, ac
                acpi_get_table(id, 0, &table);
 
        if (table) {
-               handler(table);
-               return 0;
+               return handler(table);
        } else
                return 1;
 }

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog