[coreboot-gerrit] Change in coreboot[master]: util/superiotool/fintek.c: Add support for F71889A

Marty Plummer (Code Review) gerrit at coreboot.org
Thu May 11 23:59:24 CEST 2017


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 at gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot at felixheld.de>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki at gmail.com>
Gerrit-Reviewer: Marty Plummer <ntzrmtthihu777 at gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h at gmx.de>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-HasComments: No



More information about the coreboot-gerrit mailing list