Marty Plummer has posted comments on this change. ( https://review.coreboot.org/17442 )
Change subject: util/superiotool/fintek.c: Add support for F71889A
......................................................................
Patch Set 7:
are you planning to address the comment regarding the many NANA
values I made some months ago?
Ah, entirely sorry about that. I've been rather busy with realworld concerns of late and have not had time to tinker with this. Treat reserved values as returning '0' should be sufficient for now, correct?
--
To view, visit
https://review.coreboot.org/17442
To unsubscribe, visit
https://review.coreboot.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifc5035ebe2c1c66213069567da757886923e5596
Gerrit-PatchSet: 7
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Marty Plummer
ntzrmtthihu777@gmail.com
Gerrit-Reviewer: Felix Held
felix-coreboot@felixheld.de
Gerrit-Reviewer: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Reviewer: Marty Plummer
ntzrmtthihu777@gmail.com
Gerrit-Reviewer: Nico Huber
nico.h@gmx.de
Gerrit-Reviewer: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-HasComments: No