On Tuesday 06 November 2007 14:11, Alex Williamson wrote: > > Seems like we don't need to print a big ugly warning when osi_linux > is already properly enabled. Thanks, True if "properly enabled" were not an oxymoron:-) i need to hack on this code a bit so that it doesn't complain when DMI or the user enabled it, but otherwise does. thanks, -Len > Alex > > Signed-off-by: Alex Williamson <alex.williamson@xxxxxx> > -- > > diff -r c7f1be4e5832 drivers/acpi/osl.c > --- a/drivers/acpi/osl.c Thu Nov 01 12:09:33 2007 -0700 > +++ b/drivers/acpi/osl.c Tue Nov 06 12:08:38 2007 -0700 > @@ -1167,14 +1167,15 @@ acpi_os_validate_interface (char *interf > if (!strncmp(osi_additional_string, interface, OSI_STRING_LENGTH_MAX)) > return AE_OK; > if (!strcmp("Linux", interface)) { > + if (osi_linux) > + return AE_OK; > + > printk(KERN_WARNING PREFIX > "System BIOS is requesting _OSI(Linux)\n"); > printk(KERN_WARNING PREFIX > "If \"acpi_osi=Linux\" works better,\n" > "Please send dmidecode " > "to linux-acpi@xxxxxxxxxxxxxxx\n"); > - if(osi_linux) > - return AE_OK; > } > return AE_SUPPORT; > } > > - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html