On Thu, 2009-11-26 at 05:39 +0800, Len Brown wrote: > On Wed, 25 Nov 2009, Zhang Rui wrote: > > > > > Some buggy BIOS exports multiple ACPI video bus devices for the same > > VGA controller, and multiple backlight control methods as well. > > This may mess up the ACPI video backlight control. > > http://bugzilla.kernel.org/show_bug.cgi?id=13577 > > > > We should at least send out a warning message in this case. > > > > Signed-off-by: Zhang Rui <rui.zhang@xxxxxxxxx> > > --- > > drivers/acpi/video.c | 24 ++++++++++++++++++++++++ > > 1 file changed, 24 insertions(+) > > > > Index: linux-2.6/drivers/acpi/video.c > > =================================================================== > > --- linux-2.6.orig/drivers/acpi/video.c > > +++ linux-2.6/drivers/acpi/video.c > > @@ -2233,12 +2233,36 @@ static int acpi_video_resume(struct acpi > > return AE_OK; > > } > > > > +static acpi_status > > +acpi_video_bus_match(acpi_handle handle, u32 level, void *context, > > + void **return_value) > > +{ > > + struct acpi_device *device = context; > > + struct acpi_device *sibling; > > + int result; > > + > > + if (handle == device->handle) > > + return AE_CTRL_TERMINATE; > > + > > + result = acpi_bus_get_device(handle, &sibling); > > + if (result) > > + return AE_OK; > > + > > + if (!strcmp(acpi_device_name(sibling), ACPI_VIDEO_BUS_NAME)) > > + printk(KERN_WARNING FW_BUG "Duplicate ACPI video bus " > > + "devices for the same VGA controller\n"); > > what is somebody supposed to _do_ when they see this message? > I think this warning message gives us clues when debugging backlight issues. Say we see a bug report that brightness switching doesn't work and find out that there is such a warning message in the dmesg output, then we can see which video bus device under this VGA controller actually works. If this bug is not rare, then we can set up a dmi list to choose the working video bus device for these laptops. thanks, rui -- 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
