On Wednesday 07 April 2010, Zhang Rui wrote: > 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... Yes, exactly. If more systems were affected, I'd say it would be worth considering, but not if there's only one. Thanks, 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
