Patchwork [lucid-SRU,1/2] UBUNTU: dell-laptop: Add another Dell laptop family to the DMI whitelist

login
register
mail settings
Submitter Kamal Mostafa
Date Jan. 25, 2011, 12:05 a.m.
Message ID <1295913918.16789.88.camel@fourier>
Download mbox | patch
Permalink /patch/80294/
State Accepted
Delegated to: Tim Gardner
Headers show

Comments

Kamal Mostafa - Jan. 25, 2011, 12:05 a.m.
From 51c14ac91eedd75f9fbf8a2da4fb67964a52ae1e Mon Sep 17 00:00:00 2001
From: Erik Andren <erik.andren@gmail.com>
Date: Sun, 14 Feb 2010 11:53:23 -0500
Subject: [PATCH 1/2] dell-laptop: Add another Dell laptop to the DMI whitelist

The Latitude C640 has another variation of dell in its DMI vendor entry.
Add it to the whitelist in order to enjoy the sweet fruits of software
backlight toggling.

Signed-off-by: Erik Andren <erik.andren@gmail.com>
(cherry picked from commit cb6a7937f4328a267e0806680ade500ed2c3c01a)

Bring Dell DMI table up-to-date for...
BugLink: http://bugs.launchpad.net/bugs/693078
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
---
 drivers/platform/x86/dell-laptop.c |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)
Stefan Bader - Jan. 25, 2011, 10:57 a.m.
On 01/25/2011 01:05 AM, Kamal Mostafa wrote:
> From 51c14ac91eedd75f9fbf8a2da4fb67964a52ae1e Mon Sep 17 00:00:00 2001
> From: Erik Andren <erik.andren@gmail.com>
> Date: Sun, 14 Feb 2010 11:53:23 -0500
> Subject: [PATCH 1/2] dell-laptop: Add another Dell laptop to the DMI whitelist
> 
> The Latitude C640 has another variation of dell in its DMI vendor entry.
> Add it to the whitelist in order to enjoy the sweet fruits of software
> backlight toggling.
> 
> Signed-off-by: Erik Andren <erik.andren@gmail.com>
> (cherry picked from commit cb6a7937f4328a267e0806680ade500ed2c3c01a)
> 
> Bring Dell DMI table up-to-date for...
> BugLink: http://bugs.launchpad.net/bugs/693078
> Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Acked-by: Stefan Bader <stefan.bader@canonical.com>
> ---
>  drivers/platform/x86/dell-laptop.c |    8 ++++++++
>  1 files changed, 8 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/platform/x86/dell-laptop.c b/drivers/platform/x86/dell-laptop.c
> index a224f49..6c2e1bc 100644
> --- a/drivers/platform/x86/dell-laptop.c
> +++ b/drivers/platform/x86/dell-laptop.c
> @@ -84,6 +84,13 @@ static const struct dmi_system_id __initdata dell_device_table[] = {
>  			DMI_MATCH(DMI_CHASSIS_TYPE, "8"),
>  		},
>  	},
> +	{
> +		.ident = "Dell Computer Corporation",
> +		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "Dell Computer Corporation"),
> +			DMI_MATCH(DMI_CHASSIS_TYPE, "8"),
> +		},
> +	},
>  	{ }
>  };
>  
> @@ -603,3 +610,4 @@ MODULE_AUTHOR("Matthew Garrett <mjg@redhat.com>");
>  MODULE_DESCRIPTION("Dell laptop driver");
>  MODULE_LICENSE("GPL");
>  MODULE_ALIAS("dmi:*svnDellInc.:*:ct8:*");
> +MODULE_ALIAS("dmi:*svnDellComputerCorporation.:*:ct8:*");
>

Patch

diff --git a/drivers/platform/x86/dell-laptop.c b/drivers/platform/x86/dell-laptop.c
index a224f49..6c2e1bc 100644
--- a/drivers/platform/x86/dell-laptop.c
+++ b/drivers/platform/x86/dell-laptop.c
@@ -84,6 +84,13 @@  static const struct dmi_system_id __initdata dell_device_table[] = {
 			DMI_MATCH(DMI_CHASSIS_TYPE, "8"),
 		},
 	},
+	{
+		.ident = "Dell Computer Corporation",
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "Dell Computer Corporation"),
+			DMI_MATCH(DMI_CHASSIS_TYPE, "8"),
+		},
+	},
 	{ }
 };
 
@@ -603,3 +610,4 @@  MODULE_AUTHOR("Matthew Garrett <mjg@redhat.com>");
 MODULE_DESCRIPTION("Dell laptop driver");
 MODULE_LICENSE("GPL");
 MODULE_ALIAS("dmi:*svnDellInc.:*:ct8:*");
+MODULE_ALIAS("dmi:*svnDellComputerCorporation.:*:ct8:*");