Subject: Re: AC97 patch for ICH5 (native mode) & AD1985 codec
To: TAMURA Kent <kent@netbsd.org>
From: Lars Heidieker <lars@heidieker.de>
List: current-users
Date: 11/22/2003 09:05:41
TAMURA Kent wrote:
>>*** ic/ac97.c   28 Sep 2003 22:24:09 -0000      1.49
>>--- ic/ac97.c   20 Nov 2003 09:13:37 -0000
>>***************
>>*** 901,906 ****
>>--- 901,908 ----
>>        if (host_if->flags)
>>                as->host_flags = host_if->flags(host_if->arg);
>>
>>+       ac97_read(as, AC97_REG_RESET, &id1);
>>+
>>        ac97_setup_defaults(as);
>>        ac97_read(as, AC97_REG_VENDOR_ID1, &id1);
>>        ac97_read(as, AC97_REG_VENDOR_ID2, &id2);
> 
> 
> This change seems meaningless.  Do you have the expected
> behavior by inserting DELAY() instead of reading AC97_REG_RESET?
> 

I think a DELAY() should do it as well. I'll try that.
Some sort of delay is needed because if it is missing the id1 ends up as 
0xffff and that's bogus.