[coreboot] superiotool: add register definitions for W83627HF

Andriy Gapon avg at icyb.net.ua
Fri Oct 17 17:26:52 CEST 2008


Add register definitions for W83627HF based on publicly available
specification and local testing.
Also tweak a little bit algorithm for (internal) device id calculation:
chips from W83627HF/F/HG/G family have id of 0x52 and a multitude of
revisions (0x1x, 0x3A, 0x41, maybe more), chips from W83627HF/GF family
have the same device id but revisions 0xFx.

Signed-off-by: Andriy Gapon <avg at icyb.net.ua>

---

What about this patch (in addition to my previous question/rant)?

Please note that the last line of the patch simply fixes the comment
about internal device id composition (upper half of reg 0x21 is used).
I chose the most conservative way of detecting W83627HF - only if reg
0x21 value matches 0xFx we skip the previous logic and keep using it for
all other revisions.

Here's an output of patched superiotool on my system, Delta MP2-BX-X:

superiotool r3658
Found Winbond W83977EF/EG (id=0x52, rev=0xf4) at 0x3f0
Register dump:
idx 02 20 21 22 23 24 25 26  28 2a 2b 2c 2d 2e 2f
val ff 52 f4 ff fe 84 00 00  00 00 00 00 00 00 ff
def RR 52 NA ff fe MM 00 MM  00 00 00 00 RR RR RR
LDN 0x00 (Floppy)
idx 30 60 61 70 74 f0 f1 f2  f4 f5
val 00 00 00 00 04 0c 00 ff  00 00
def 01 03 f0 06 02 0e 00 ff  00 00
LDN 0x01 (Parallel port)
idx 30 60 61 70 74 f0
val 00 00 00 00 04 00
def 01 03 78 07 04 3f
LDN 0x02 (COM1)
idx 30 60 61 70 f0
val 01 03 f8 04 00
def 01 03 f8 04 00
LDN 0x03 (COM2)
idx 30 60 61 70 f0 f1
val 01 02 e8 03 00 00
def 01 02 f8 03 00 00
LDN 0x05 (Keyboard)
idx 30 60 61 62 63 70 72 f0
val 01 00 60 00 64 01 0c 40
def 01 00 60 00 64 01 0c 83
LDN 0x07 (GPIO 1)
idx 30 60 61 62 63 64 65 70  72 e0 e1 e2 e3 e4 e5 e6  e7 f1
val 01 00 00 00 00 00 00 00  00 01 01 01 01 01 01 01  01 00
def 00 00 00 00 00 00 00 00  00 01 01 01 01 01 01 01  01 00
LDN 0x08 (GPIO 2)
idx 30 60 61 70 72 e8 e9 ea  eb ec ed f0 f1 f2 f3 f4
val 01 00 00 00 00 10 01 01  01 01 08 00 ff 00 00 00
def 00 00 00 00 00 01 01 01  01 01 01 00 RR 00 00 00
LDN 0x0a (ACPI)
idx 30 70 e0 e1 e2 e3 e4 e5  e6 e7 f0 f1 f3 f4 f6 f7  f9 fe ff
val 00 00 00 00 00 00 00 00  00 00 00 8f 30 00 00 00  00 00 00
def 00 00 00 00 MM MM MM 00  00 00 00 00 00 00 00 00  00 RR RR


-- 
Andriy Gapon
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: superiotool.diff
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20081017/0efba3b7/attachment.ksh>


More information about the coreboot mailing list