Re: Fix for dell inspiron 8100 maestro3

[Takashi Iwai <tiwai@suse.de>]

At Sat, 17 May 2003 04:45:12 +0200,
Javier Conti wrote:
> 
> [1  <text/plain; iso-8859-1 (7bit)>]
> 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...

ok, i'll add your patch.

the irda_workaround field specifies that the workaround for touching
the bit 0x10 of GPIO direction.  depending on the device, the GPIO
direction for the same purpose might be different.

about debugging this problem:

please check the register GPIO_DIRECTION by adding printk() around
line 1922 of maestro3.c, such as

	for (i = 0; i < 5; i++) {
		dir = inw(io + GPIO_DIRECTION);
==>		printk("GPIO_DIRECTION = 0x%x\n", dir);
		if (! chip->quirk || ! chip->quirk->irda_workaround)

and compare the values between the working and non-working states.


Takashi


-------------------------------------------------------
This SF.net email is sponsored by: If flattening out C++ or Java
code to make your application fit in a relational database is painful, 
don't do it! Check out ObjectStore. Now part of Progress Software.
http://www.objectstore.net/sourceforge