Julius Werner has posted comments on this change. ( https://review.coreboot.org/21598 )
Change subject: util/cbfstool: Add "expand" command to make CBFS span an fmap region ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/21598/1/util/cbfstool/cbfstool.c File util/cbfstool/cbfstool.c:
https://review.coreboot.org/#/c/21598/1/util/cbfstool/cbfstool.c@1300 PS1, Line 1300: "Expand CBFS to span entire region\n" Don't we want to kill two birds with one stone and implement a command that can both expand and truncate here? The "parse cbfstool output with sed" solution we currently use to truncate in the ebuild doesn't really inspire that much confidence anyway, I think while we're redoing this anyway we might as well do away with that. (I'd call the command cbfstool truncate then, with behavior similar to the truncate POSIX tool.)