Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44695 )
Change subject: superio/winbond/wpcd376i: Resurrect the driver
......................................................................
Patch Set 1: Code-Review+1
Are you planning to add things to the chip.h file? If not, I'd drop that one, since it's currently basically empty. Not too opposed to keeping it though.
The rest looks good to me, but wanted to check back about the chip.h file before giving it a +2
--
To view, visit
https://review.coreboot.org/c/coreboot/+/44695
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0ff1a63c47d5dff2599c83a1cebe1ac5ff2136b1
Gerrit-Change-Number: 44695
Gerrit-PatchSet: 1
Gerrit-Owner: Samuel Holland
samuel@sholland.org
Gerrit-Reviewer: Felix Held
felix-coreboot@felixheld.de
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Comment-Date: Sun, 23 Aug 2020 16:12:58 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment