build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/26836 )
Change subject: cbfstool: Update FIT entries in the second bootblock ......................................................................
Patch Set 17:
(4 comments)
https://review.coreboot.org/#/c/26836/17/util/cbfstool/cbfstool.c File util/cbfstool/cbfstool.c:
https://review.coreboot.org/#/c/26836/17/util/cbfstool/cbfstool.c@1486 PS17, Line 1486: " -x EMTPY_FIT_ENTRIES \ \n" unnecessary whitespace before a quoted newline
https://review.coreboot.org/#/c/26836/17/util/cbfstool/cbfstool.c@1490 PS17, Line 1490: " ucode-region is a region in the FMAP, its address is \n" unnecessary whitespace before a quoted newline
https://review.coreboot.org/#/c/26836/17/util/cbfstool/cbfstool.c@1492 PS17, Line 1492: " inserted as the first entry in the topswap FIT. \n" unnecessary whitespace before a quoted newline
https://review.coreboot.org/#/c/26836/17/util/cbfstool/fit.c File util/cbfstool/fit.c:
https://review.coreboot.org/#/c/26836/17/util/cbfstool/fit.c@348 PS17, Line 348: ERROR("No room, blob mcus = %d, total entries = %d\n", line over 80 characters