The patch titled acpi_boot_init() making bad check on return code has been added to the -mm tree. Its filename is acpi_boot_init-making-bad-check-on-return-code.patch *** Remember to use Documentation/SubmitChecklist when testing your code *** See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find out what to do about this ------------------------------------------------------ Subject: acpi_boot_init() making bad check on return code From: John Keller <jpk@xxxxxxx> acpi_boot_init() is making a bad check on the return status from acpi_table_parse(). acpi_table_parse() now returns zero on success, one on failure. Signed-off-by: Aaron Young <ayoung@xxxxxxx> Cc: Len Brown <lenb@xxxxxxxxxx> Cc: "Luck, Tony" <tony.luck@xxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- arch/ia64/kernel/acpi.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff -puN arch/ia64/kernel/acpi.c~acpi_boot_init-making-bad-check-on-return-code arch/ia64/kernel/acpi.c --- a/arch/ia64/kernel/acpi.c~acpi_boot_init-making-bad-check-on-return-code +++ a/arch/ia64/kernel/acpi.c @@ -651,7 +651,7 @@ int __init acpi_boot_init(void) * information -- the successor to MPS tables. */ - if (acpi_table_parse(ACPI_SIG_MADT, acpi_parse_madt) < 1) { + if (acpi_table_parse(ACPI_SIG_MADT, acpi_parse_madt)) { printk(KERN_ERR PREFIX "Can't find MADT\n"); goto skip_madt; } @@ -702,7 +702,7 @@ int __init acpi_boot_init(void) * gets interrupts such as power and sleep buttons. If it's not * on a Legacy interrupt, it needs to be setup. */ - if (acpi_table_parse(ACPI_SIG_FADT, acpi_parse_fadt) < 1) + if (acpi_table_parse(ACPI_SIG_FADT, acpi_parse_fadt)) printk(KERN_ERR PREFIX "Can't find FADT\n"); #ifdef CONFIG_SMP _ Patches currently in -mm which might be from jpk@xxxxxxx are origin.patch git-acpi.patch acpi_boot_init-making-bad-check-on-return-code.patch - To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html