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.