On Tue, Oct 09, 2007 at 03:29:12PM +0200, Thomas Renninger wrote: > static const struct acpi_device_id dummy_vid_device_ids[] = { > {"LNXVIDEO", 0}, > {"", 0}, > }; No. This will match if any of the video extension is implemented. We only want it to match if backlight control is implemented. There are plenty of Thinkpads that implement a subset of the video extension but still need backlight control to be handled via the Thinkpad-specific routes. -- Matthew Garrett | mjg59@xxxxxxxxxxxxx - 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