Youness Alaoui has posted comments on this change. ( https://review.coreboot.org/18883 )
Change subject: Handle Intel Wildcat Point *LP* like Lynx Point LP
......................................................................
Patch Set 3:
The ich_descriptors.c has 2 other functions that refer to
WILDCAT_POINT but not WILDCAT_POINT_LP.
Thanks, no idea how I could have missed that ;)
No problem, it happens, that's what reviews are for :)
You caught one of them in a later commit (line 230 of https://review.coreboot.org/#/c/18973/5/ich_descriptors.c)
But the second one was never caught though.
--
To view, visit
https://review.coreboot.org/18883
To unsubscribe, visit
https://review.coreboot.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I6b7ca3c0bde111b04ed7c745ed76d28d3d05f01c
Gerrit-PatchSet: 3
Gerrit-Project: flashrom
Gerrit-Branch: staging
Gerrit-Owner: Nico Huber
nico.h@gmx.de
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: Youness Alaoui
snifikino@gmail.com
Gerrit-Reviewer: build bot (Jenkins)
Gerrit-HasComments: No