Hi, thomas, we got a couple of regressions caused by the acpi video detect patch set. IMO, the reason is that they have buggy ACPI video extension implemented. and these laptops used to control the backlight via platform drivers. you can refer to this thread or the bug: http://bugzilla.kernel.org/show_bug.cgi?id=12037 http://bugzilla.kernel.org/show_bug.cgi?id=12235 http://bugzilla.kernel.org/show_bug.cgi?id=12249 http://bugzilla.kernel.org/show_bug.cgi?id=12302 I have fixed one by overriding the DSDT. the source code of the customized DSDT can be found at http://bugzilla.kernel.org/show_bug.cgi?id=12249#c32 so IMO, acpi_video_backlight_support should be able to verify if BIOS offers buggy ACPI backlight control methods and returns false in this case. but then we need to move a lot of ACPI video code (acpi_video_init_brightness) to video_detect.c, which doesn't sound quite good. what do you think? thanks, rui On Wed, 2008-12-31 at 11:33 +0800, Zhang Rui wrote: > the ACPI video backlight I/F on your laptop is broken. > could you please boot with acpi_backlight=vendor and see if it works at > before? > > BTW: please attach the output of "grep . /proc/acpi/video/*/*/*" in > 1. 2.6.28 kenrel & boot with "acpi_backlight=vendor" > 2. an earlier working kernel, say 2.6.26. > > thanks, > rui > > > On Wed, 2008-12-31 at 00:02 +0800, Kel Modderman wrote: > > On Tuesday 30 December 2008 11:37:47 Zhang Rui wrote: > > > On Mon, 2008-12-29 at 16:39 +0800, Kel Modderman wrote: > > > > On Monday 29 December 2008 11:07:32 Zhang Rui wrote: > > > > > hi, > > > > > > > > > > This is a duplicate of bug > > > > > http://bugzilla.kernel.org/show_bug.cgi?id=12302 > > > > > please try the patch at > > > > > http://bugzilla.kernel.org/show_bug.cgi?id=12037#c5 > > > > > > > > When patch is applied backlight control returned to normality. > > > > > > > please attach the output of "grep . /proc/acpi/video/*/*/*" both before > > > and after applying this patch. > > > > Attached. > > > > Thanks, Kel > > -- > 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
