[coreboot-gerrit] Change in coreboot[master]: ifdtool: port the feature to set AltMeDisable/HAP bit here
build bot (Jenkins) (Code Review)
gerrit at coreboot.org
Mon Aug 13 08:54:50 CEST 2018
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21437 )
Change subject: ifdtool: port the feature to set AltMeDisable/HAP bit here
......................................................................
Patch Set 63:
(4 comments)
https://review.coreboot.org/#/c/21437/63/util/ifdtool/ifdtool.c
File util/ifdtool/ifdtool.c:
https://review.coreboot.org/#/c/21437/63/util/ifdtool/ifdtool.c@188
PS63, Line 188: printf("Peculiar firmware descriptor, assuming Ibex Peak compatibility.\n");
line over 80 characters
https://review.coreboot.org/#/c/21437/63/util/ifdtool/ifdtool.c@197
PS63, Line 197: printf("Peculiar firmware descriptor, assuming Wildcat Point compatibility.\n");
line over 80 characters
https://review.coreboot.org/#/c/21437/63/util/ifdtool/ifdtool.c@1650
PS63, Line 1650: fpsba_t* fpsba = find_fpsba(image, size);
"foo* bar" should be "foo *bar"
https://review.coreboot.org/#/c/21437/63/util/ifdtool/ifdtool.c@1651
PS63, Line 1651: fmsba_t* fmsba = find_fmsba(image, size);
"foo* bar" should be "foo *bar"
--
To view, visit https://review.coreboot.org/21437
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a2ecc60cfbb9ee9d96f15be3d53226cb428729a
Gerrit-Change-Number: 21437
Gerrit-PatchSet: 63
Gerrit-Owner: Bill XIE <persmule at gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Nicola Corna <nicola at corna.info>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-CC: Stefan Tauner <stefan.tauner at gmx.at>
Gerrit-Comment-Date: Mon, 13 Aug 2018 06:54:50 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180813/336928a2/attachment.html>
More information about the coreboot-gerrit
mailing list