diff mbox

[trusty] SAUCE: Add use native backlight quirk for Dell Inspiron 5547/5447

Message ID 1404968198-30610-1-git-send-email-yidi.lin@canonical.com
State New
Headers show

Commit Message

Edward Lin July 10, 2014, 4:56 a.m. UTC
BugLink: http://bugs.launchpad.net/bugs/1332437

The ACPI backlight control is broken on these 2 models.
Since backlight control works on 3.16.0-999-generic #201406200205,
and also works on 3.13.0-29 with video.use_native_backlight=1.
Add use native backlight quirk for these 2 models as the workaround.

This patch can be ignored after 3.16-r2.

Signed-off-by: Edward Lin <yidi.lin@canonical.com>
---
 drivers/acpi/video.c |   24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

Comments

Edward Lin July 10, 2014, 6:11 a.m. UTC | #1
Please ignore this patch.

I will re-send a patch which excludes Dell Inspiron 5737

On Thu, Jul 10, 2014 at 12:56 PM, Edward Lin <yidi.lin@canonical.com> wrote:
> BugLink: http://bugs.launchpad.net/bugs/1332437
>
> The ACPI backlight control is broken on these 2 models.
> Since backlight control works on 3.16.0-999-generic #201406200205,
> and also works on 3.13.0-29 with video.use_native_backlight=1.
> Add use native backlight quirk for these 2 models as the workaround.
>
> This patch can be ignored after 3.16-r2.
>
> Signed-off-by: Edward Lin <yidi.lin@canonical.com>
> ---
>  drivers/acpi/video.c |   24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
>
> diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c
> index 5b6f970..a0b124b 100644
> --- a/drivers/acpi/video.c
> +++ b/drivers/acpi/video.c
> @@ -500,6 +500,30 @@ static struct dmi_system_id video_dmi_table[] __initdata = {
>         },
>         {
>          .callback = video_set_use_native_backlight,
> +        .ident = "Dell Inspiron 5737",
> +        .matches = {
> +               DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> +               DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 5737"),
> +               },
> +       },
> +       {
> +        .callback = video_set_use_native_backlight,
> +        .ident = "Dell Inspiron 5547",
> +        .matches = {
> +               DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> +               DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 5547"),
> +               },
> +       },
> +       {
> +        .callback = video_set_use_native_backlight,
> +        .ident = "Dell Inspiron 5447",
> +        .matches = {
> +               DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> +               DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 5447"),
> +               },
> +       },
> +       {
> +        .callback = video_set_use_native_backlight,
>          .ident = "Acer Aspire 5733Z",
>          .matches = {
>                 DMI_MATCH(DMI_SYS_VENDOR, "Acer"),
> --
> 1.7.9.5
>
diff mbox

Patch

diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c
index 5b6f970..a0b124b 100644
--- a/drivers/acpi/video.c
+++ b/drivers/acpi/video.c
@@ -500,6 +500,30 @@  static struct dmi_system_id video_dmi_table[] __initdata = {
 	},
 	{
 	 .callback = video_set_use_native_backlight,
+	 .ident = "Dell Inspiron 5737",
+	 .matches = {
+		DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+		DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 5737"),
+		},
+	},
+	{
+	 .callback = video_set_use_native_backlight,
+	 .ident = "Dell Inspiron 5547",
+	 .matches = {
+		DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+		DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 5547"),
+		},
+	},
+	{
+	 .callback = video_set_use_native_backlight,
+	 .ident = "Dell Inspiron 5447",
+	 .matches = {
+		DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+		DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 5447"),
+		},
+	},
+	{
+	 .callback = video_set_use_native_backlight,
 	 .ident = "Acer Aspire 5733Z",
 	 .matches = {
 		DMI_MATCH(DMI_SYS_VENDOR, "Acer"),