David Hendricks has posted comments on this change. ( https://review.coreboot.org/20937 )
Change subject: ich_descriptors: Fix off-by-one error
......................................................................
Patch Set 1: Code-Review-2
Out of caution, I went ahead and added Lewisburg as its own ICH in https://review.coreboot.org/#/c/20922/ . So far the only notable differences I've come across are the number of masters and maybe the number of protected range registers. Let's discuss further in 20922.
--
To view, visit
https://review.coreboot.org/20937
To unsubscribe, visit
https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: staging
Gerrit-MessageType: comment
Gerrit-Change-Id: I92627332265cf79720b98241d73bc36b1f6a7167
Gerrit-Change-Number: 20937
Gerrit-PatchSet: 1
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: Stefan Tauner
stefan.tauner@gmx.at
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Comment-Date: Sat, 19 Aug 2017 03:18:13 +0000
Gerrit-HasComments: No