build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33300 )
Change subject: [RFC and WIP]Add an option to align FMAP regions ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/33300/1/util/cbfstool/fmd.h File util/cbfstool/fmd.h:
https://review.coreboot.org/#/c/33300/1/util/cbfstool/fmd.h@57 PS1, Line 57: unsigned alignment; Prefer 'unsigned int' to bare use of 'unsigned'
https://review.coreboot.org/#/c/33300/1/util/cbfstool/fmd.c File util/cbfstool/fmd.c:
https://review.coreboot.org/#/c/33300/1/util/cbfstool/fmd.c@252 PS1, Line 252: cur_section->offset = (cur_section->offset + cur_section->alignment - 1) line over 80 characters