Jonathan Neuschäfer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34932 )
Change subject: util/cbfstool: new utility pnortool ......................................................................
Patch Set 1:
(1 comment)
About the endianness issue (because I guess not everyone will want to build on a big-endian host ;) : Maybe you can include src/include/endian.h and use cpu_to_le32(), etc. (you might have to move it to src/common/include first, though, I'm not sure). htonl etc. are technically also an option, but I find them hard to read and this isn't per se about "network" byte order but specifically about big endian (which happens to be the same).
https://review.coreboot.org/c/coreboot/+/34932/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/34932/1//COMMIT_MSG@8 PS1, Line 8: A brief introduction of this tool would be nice to have here.