Subject: Re: AC97 patch for ICH5 (native mode) & AD1985 codec
To: TAMURA Kent <email@example.com>
From: Lars Heidieker <firstname.lastname@example.org>
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_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.