On Wed, 2010-04-07 at 02:51 +0800, Rafael J. Wysocki wrote: > On Tuesday 06 April 2010, Zhang Rui wrote: > > On Mon, 2010-04-05 at 07:19 +0800, Rafael J. Wysocki wrote: > > > On Friday 02 April 2010, Rafael J. Wysocki wrote: > > > > On Friday 02 April 2010, Zhang Rui wrote: > > > > > On Fri, 2010-04-02 at 03:45 +0800, Rafael J. Wysocki wrote: > > > > > > On Thursday 01 April 2010, Zhang Rui wrote: > > > > > > > On Thu, 2010-04-01 at 09:34 +0800, Zhang Rui wrote: > > > > > > > > On Thu, 2010-04-01 at 09:19 +0800, Rafael J. Wysocki wrote: > > > > > > This patch may not be necessary after all. > > > > > > I'll repost it if I find it's still needed. > > > > > okay. > > I re-read the code and find out that you're right. sorry for the noise. > > > > please feel free to add an Acked-by from me next time you send it to the > > list. > > Thanks, but it looks like only one box is affected and because its BIOS returns > the minimal brightness value in the initial _BQC. > I'm not sure if it's worth > changing the current behavior because of that. > well. in the ACPI video driver, we assume that BIOS sets a PROPER brightness level when loading OS. so we use the _BQC return value as the default backlight, but apparently this doesn't work well in this BIOS. I would rather say this is a BIOS problem, which I'm not sure if we should fix it in Linux kernel or not... thanks, rui > Rafael > -- > 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 -- 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
