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

[Felix Held (Code Review)]

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

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

Patch Set 6:
> I could have created something devicetree based, but the structure would have to be too complicated due to all the parameters that needs to be controlled. I thought that an API breaking the parameters into logically grouped parameters was much easier to understand and program.

Yep
> As I mentioned before, I can't merge Padmelon board until 33615 is merged. Locally, I have build and tested the SIO code... found 1 mistake (caused by trying to enable HWM too early of all things, lol). Once I moved enabling HWM to early ramstage (just before calling the API), it worked as expected.

I know; just wanted to let you know why I haven't +2ed the patch yet, but that I'm happy with its current state