[coreboot-gerrit] Change in coreboot[master]: superio/fintek: Add f81803a

[Richard Spiegel (Code Review)]

Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33623 )

Change subject: superio/fintek: Add f81803a
......................................................................

Patch Set 7:

(1 comment)

This whole code has wrong things, and I was not aware (and apparently neither was Marc, who started the SIO code). There's a lot to study before I make any change. Anyway, thanks for pointing it out.
Is there any document I can use for reference?

https://review.coreboot.org/c/coreboot/+/33623/7/src/superio/fintek/f81803a/... 
File src/superio/fintek/f81803a/f81803a.h:

https://review.coreboot.org/c/coreboot/+/33623/7/src/superio/fintek/f81803a/... 
PS7, Line 79: #define FINTEK_BASE_ADDRESS		0x4E
> can probably also be 0x2e; this is usually configured in the devicetree

Only through power-on strap. By default it's 4e. Was not aware of this capability of devicetree.