On Tuesday 09 October 2012, Len Brown wrote: > 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:-( I wonder what Windows does here. Haven't tested it on this machine but I assume that it works. -- Ondrej Zary -- 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
