David Hendricks has posted comments on this change. ( https://review.coreboot.org/20922 )
Change subject: chispet_enable: Add support for C620-series Lewisburg PCH
......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/20922/2/ichspi.c
File ichspi.c:
https://review.coreboot.org/#/c/20922/2/ichspi.c@1704
PS2, Line 1704: 6; /* FIXME: should be 5? */
num_pr = 6 seemed wrong for both Sunrise Point and Lewisburg, but maybe I am missing something...
--
To view, visit
https://review.coreboot.org/20922
To unsubscribe, visit
https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: staging
Gerrit-MessageType: comment
Gerrit-Change-Id: I96c89bc28bdfcd953229c17679f2c28f8b874d0b
Gerrit-Change-Number: 20922
Gerrit-PatchSet: 2
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: Sat, 19 Aug 2017 03:15:59 +0000
Gerrit-HasComments: Yes