Re: Fix for dell inspiron 8100 maestro3

[Javier Conti <av@labcc.ch>]

[-- Attachment #1: Type: text/plain, Size: 743 bytes --]

On Friday 16 May 2003 17:01, Takashi Iwai wrote:
> are you sure that the patch is correct?
> your patch adds the same vendor an device id with the existing
> inspiron 8000 (1028:00a4).

Ok, I found the correct values, repatched and now the modules tells me the 
hack is enabled. With the other values, that didn't happen. Seen that, I 
tried to recompile a vanilla alsa-driver version, to see if my previous patch 
was doing some difference. The weird thing is that it still didn't work!
I'm puzzled, since the wrong patch fixed the problem anyway...

I'm sorry for the "noise" but hope to be helpful by sending this patch which 
includes the correct values.

Bye, Javier
-- 
Javier Conti - Labcc
Tel: +41(0)91 690 0703
Fax: +41(0)91 690 0702

[-- Attachment #2: maestro3.patch --]
[-- Type: text/x-diff, Size: 534 bytes --]

--- maestro3.c	2003-04-23 12:06:36.000000000 +0200
+++ /usr/src/modules/alsa-driver/alsa-kernel/pci/maestro3.c	2003-05-17 04:37:29.000000000 +0200
@@ -947,8 +947,14 @@
 		.amp_gpio = -1,
 		.irda_workaround = 1,
 	},
-	/* FIXME: Inspiron 8100 id should probably be here, too
-	 * (8200 irrelevant: has intel8x0 with CS4205) */
+	/* Dell Inspiron 8100 */
+	{
+		.name = "Dell Inspiron 8100",
+		.vendor = 0x1028,
+		.device = 0x00e6,
+		.amp_gpio = -1,
+		.irda_workaround = 1,
+	},
 	/* NEC LM800J/7 */
 	{
 		.name = "NEC LM800J/7",