On Sat, Jan 07, 2012 at 03:12:38PM +0100, Corentin Chary wrote: > On these laptops, the ACPI video is not functional, and very unlikely > to be fixed by the vendor. Note that intel_backlight works for some > of these laptops, and the backlight from samsung-laptop always work. > > The good news is that newer laptops have functional ACPI video device > and won't end up growing this list. > > Signed-off-by: Corentin Chary <[email protected]> > --- > > Could the concerned people test this patch and check that it correctly > disable the acpi_video backlight ? > > drivers/acpi/video_detect.c | 40 ++++++++++++++++++++++++++++++++++++++++ > 1 files changed, 40 insertions(+), 0 deletions(-) > > diff --git a/drivers/acpi/video_detect.c b/drivers/acpi/video_detect.c > index 45d8097..376bce2 100644 > --- a/drivers/acpi/video_detect.c > +++ b/drivers/acpi/video_detect.c > @@ -132,6 +132,44 @@ find_video(acpi_handle handle, u32 lvl, void *context, > void **rv) > return AE_OK; > } > > +/* Force to use vendor driver when the ACPI device is known to be > + * buggy */ > +static int video_detect_force_vendor(const struct dmi_system_id *d) > +{ > + acpi_video_support |= ACPI_VIDEO_BACKLIGHT_DMI_VENDOR; > + return 0; > +} > + > +static struct dmi_system_id video_detect_dmi_table[] = { > + { > + .callback = video_detect_force_vendor, > + .ident = "N150P", > + .matches = { > + DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."), > + DMI_MATCH(DMI_PRODUCT_NAME, "N150P"), > + DMI_MATCH(DMI_BOARD_NAME, "N150P"), > + }, > + }, > + { > + .callback = video_detect_force_vendor, > + .ident = "N145P/N250P/N260P", > + .matches = { > + DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."), > + DMI_MATCH(DMI_PRODUCT_NAME, "N145P/N250P/N260P"), > + DMI_MATCH(DMI_BOARD_NAME, "N145P/N250P/N260P"), > + }, > + }, > + { > + .callback = video_detect_force_vendor, > + .ident = "N150/N210/N220", > + .matches = { > + DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."), > + DMI_MATCH(DMI_PRODUCT_NAME, "N150/N210/N220"), > + DMI_MATCH(DMI_BOARD_NAME, "N150/N210/N220"), > + }, > + }, > +}; > +
I don't see NF110/NF210/NF310 in this list. Was that an oversight? -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
