[coreboot-gerrit] Change in ...coreboot[master]: util/superiotool: detect Fintek F81866, F8196*

[Felix Held (Code Review)]

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

Change subject: util/superiotool: detect Fintek F81866, F8196*
......................................................................

Patch Set 2: Code-Review+1
> F8196* detection is not actually tested. If that's a problem, I'll remove it and leave just the F81866 ID.

So all F8196* chips share the same ID, right? If they all share one ID (I wonder if those are the same silicon, but different packages), then I'm ok with the patch in its current state.
It would be nice if you'd add the registers and their defaults in a follow-up patch