[coreboot] [superiotool] patch for fintek f71889fg

[Uwe Hermann]

Hi,

On Mon, Nov 01, 2010 at 01:02:53PM -0700, David Hendricks wrote:
> The patch (attached) was tested by a user on IRC who had the F71889FG. I
> wrote it using documentation from Fintek's website available here:
> http://www.fintek.com.tw/files/productfiles/F71889_V0.28P.pdf
> 
> This patch also seems to work for the F71889ED, which uses 0x09 and 0x09 for

Both times 0x09? Or is this a typo?


> chip ID bytes 1 & 2. However, I have not been able to find documentation to
> verify that the two chips are identical from superiotool's perspective.
> 
> The F71889 seems popular on current generation platforms with AMD chipsets,
> in case there are folks looking to try it on a presently unsupported
> board...
> 
> Signed-off-by: David Hendricks <dhendrix at google.com>

Are you sure this is the correct patch? It doesn't seem to match the
datasheet in a number of places, e.g. 0x20 and 0x21 (IDs) are incorrect
in NOLDN (as well as most other values in NOLDN), some registers are missing
completely, some LDNs are missing completely etc.

Is this for another Super I/O, or remainders of copying another table?


Uwe.
-- 
http://hermann-uwe.de     | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org