On 10/08/2012 10:42 AM, Matthew Garrett wrote: > On Sun, Oct 07, 2012 at 09:00:09PM +0200, Ondrej Zary wrote: > >> Added acpi_read_bit_register there and it seems that SCI_EN is already set! >> >> This patch fixes the problem here. I wonder how this affects systems that >> require SCI_EN to be set. > > I /think/ that this will be safe, but it doesn't match my recollection > of how Windows behaves so it may break something. Any chance you can > find someone with one of the machines mentioned in > https://bugzilla.kernel.org/show_bug.cgi?id=13745 and make sure that > they still work with your patch? yikes. we started with a white-list, then we made setting SCI_EN the default, and here is at least one box that wants to be on a black list so that we don't set SCI_EN:-( -- 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
