Attention is currently required from: Paul Menzel, Clay Daniels, Felix Held. Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49112 )
Change subject: superiotool: Add ID for Nuvoton NCT6797D ......................................................................
Patch Set 10: Code-Review+1
(2 comments)
Patchset:
PS10: Your superiotool dump [1] looks very reasonable. Please update the NANA fields as done in CB:49241 and adapt the printed name to make people aware of the unfortunate lack of a datasheet. I see no reason not to merge it then.
[1] https://paste.flashrom.org/view.php?id=3423
File util/superiotool/nuvoton.c:
https://review.coreboot.org/c/coreboot/+/49112/comment/e54ac78a_cf89728d PS10, Line 570: NCT6797D As we can never be 100% without a datasheet, how about naming this like
NCT6797D (experimental)
or
NCT6797D (potential subset)
or something like that.