build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56127 )
Change subject: util/cbfstool: Catch baseaddress parameter errors early ......................................................................
Patch Set 1:
(3 comments)
File util/cbfstool/cbfstool.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-123549): https://review.coreboot.org/c/coreboot/+/56127/comment/fe5b5a82_fffc681b PS1, Line 811: ERROR("Wrong -b parameter: FSP non-XIP has to be located outside of flash\n"); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-123549): https://review.coreboot.org/c/coreboot/+/56127/comment/686bbb83_7e6ef1bd PS1, Line 817: ERROR("Wrong -b parameter: base not mapped inside flash\n"); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-123549): https://review.coreboot.org/c/coreboot/+/56127/comment/8ec9c59c_7ce25d5a PS1, Line 822: ERROR("Wrong -b parameter: base bigger than %s region\n", line over 96 characters