Clay Daniels has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49112 )
Change subject: superiotool: Add ID for Nuvoton NCT6797D
......................................................................
Patch Set 3:
Patch Set 2:
(4 comments)
Welcome to coreboot, and thank you for the patch.
Thanks for the help! This is my first commit. I think I made the changes you suggested, and they make good sense. I did not find the datasheet for the NCT6797D but based it on the NTC6791D which is similar. It works for me when I tested it on my computer. Let me know what more I need to do.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/49112
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1a5f962f2fd9dc479ddbbaf5e1bebea2c7c9e03f
Gerrit-Change-Number: 49112
Gerrit-PatchSet: 3
Gerrit-Owner: Clay Daniels
clay.daniels.jr@gmail.com
Gerrit-Reviewer: Felix Held
felix-coreboot@felixheld.de
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Tue, 05 Jan 2021 18:08:06 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment