[PATCH] Input: atkbd - broaden the Dell DMI signatures

[PATCH] Input: atkbd - broaden the Dell DMI signatures

[Matthew Garrett]

Some Dells need the dell input quirk applied but have a different vendor
string in their DMI tables. Add an extra entry to cover these machines as
well.
    
Signed-off-by: Matthew Garrett <mjg@redhat.com>

---

Dmitry, any chance of getting this queued for .29? Probably suitable for 
-stable as well.

diff --git a/drivers/input/keyboard/atkbd.c b/drivers/input/keyboard/atkbd.c
index 379b7ff..d48de0c 100644
--- a/drivers/input/keyboard/atkbd.c
+++ b/drivers/input/keyboard/atkbd.c
@@ -1476,6 +1476,15 @@ static struct dmi_system_id atkbd_dmi_quirk_table[] __initdata = {
 		.driver_data = atkbd_dell_laptop_keymap_fixup,
 	},
 	{
+		.ident = "Dell Laptop",
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "Dell Computer Corporation"),
+			DMI_MATCH(DMI_CHASSIS_TYPE, "8"), /* Portable */
+		},
+		.callback = atkbd_setup_fixup,
+		.driver_data = atkbd_dell_laptop_keymap_fixup,
+	},
+	{
 		.ident = "HP 2133",
 		.matches = {
 			DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),

-- 
Matthew Garrett | mjg59@srcf.ucam.org

Re: [PATCH] Input: atkbd - broaden the Dell DMI signatures

[Dmitry Torokhov]

On Monday 29 December 2008 06:29:02 Matthew Garrett wrote:
> Some Dells need the dell input quirk applied but have a different vendor
> string in their DMI tables. Add an extra entry to cover these machines as
> well.
>
> Signed-off-by: Matthew Garrett <mjg@redhat.com>
>
> ---
>
> Dmitry, any chance of getting this queued for .29? Probably suitable for
> -stable as well.

Yep, will send with the rest of stuff for .29. Thank you Matthew.

-- 
Dmitry

Re: [PATCH] Input: atkbd - broaden the Dell DMI signatures

[Nick Warne]

Hi Matthew, Dmitry,

On Mon, 29 Dec 2008 20:15:33 -0800
Dmitry Torokhov <dtor@mail.ru> wrote:

> On Monday 29 December 2008 06:29:02 Matthew Garrett wrote:
> > Some Dells need the dell input quirk applied but have a different
> > vendor string in their DMI tables. Add an extra entry to cover
> > these machines as well.
> >
> > Signed-off-by: Matthew Garrett <mjg@redhat.com>
> >
> > ---
> >
> > Dmitry, any chance of getting this queued for .29? Probably
> > suitable for -stable as well.
> 
> Yep, will send with the rest of stuff for .29. Thank you Matthew.

I am mailing off LKML here, as I don't know what to think, but somebody
has reported a new ID for these Dell laptops:




http://ubuntuforums.org/showpost.php?p=6488632&postcount=37
===
You say you're looking for people whose Vendor IDs don't say "Dell"?

Looks like you're looking for me.

261-BIOS Information
278: Vendor: Phoenix Technologies LTD
312- Version: A04
809- Manufacturer: Dell Inc.
834: Product Name: Inspiron 700m
869- Version: -1
--
1050- Manufacturer: DELL SYSTEM
1077: Product Name: 0D9593
1099- Version:
===


I dunno if this is a rogue post, but have no reason to assume otherwise.

Will this never end?

Nick
-- 
Free Software Foundation Associate Member 5508
http://linicks.net/