On Tuesday 16 February 2010 22:53:50 Matthew Garrett wrote: > Right now, if _BQC returns a value we don't understand we immediately > invalidate it. Change this behaviour so we only invalidate it if it > continues to give an invalid answer after we've already set a brightness. > > + if (!init) { > + /* > + * BQC returned an invalid level. > + * Stop using it. > + */ > + ACPI_WARNING((AE_INFO, > + "%s returned an invalid level", > + buf)); Can you use: printk(KERN_WARNING FW_WARN "%s returned an invalid level", buf); instead please. It would be great if major kernel contributors, especially those working near the BIOS do make use of the FW_* strings more often! Cleaning up existing messages is one (work intensive) thing, but please use it to identify new BIOS issues. Thanks, Thomas -- 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