Nico Huber posted comments on this change.
View Change
Patch set 4:
(5 comments)
File ich_descriptors.c:
Patch Set #4, Line 371: if (cs == CHIPSET_100_SERIES_SUNRISE_POINT) {
missing C620 here
Patch Set #4, Line 373: EC
EC/BMC?
Patch Set #4, Line 373: unknown
4th region is Device Expansion, DE?
Patch Set #4, Line 373: ,
We have to add a sixth region, "unknown", or we'd run into the
check below.
Patch Set #4, Line 381: msg_pdbg2(" FD BIOS ME GbE Pltf Reg5 Reg6 Reg7 EC Reg9\n");
More regions, sigh.
To view, visit change 20922. To unsubscribe, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: staging
Gerrit-MessageType: comment
Gerrit-Change-Id: I96c89bc28bdfcd953229c17679f2c28f8b874d0b
Gerrit-Change-Number: 20922
Gerrit-PatchSet: 4
Gerrit-Owner: David Hendricks <david.hendricks@gmail.com>
Gerrit-Reviewer: David Hendricks <david.hendricks@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-Comment-Date: Mon, 21 Aug 2017 20:44:39 +0000
Gerrit-HasComments: Yes