[coreboot-gerrit] Change in coreboot[master]: util/cbfstool: new utility pnortool

[Marty E. Plummer (Code Review)]

Marty E. Plummer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34932 )

Change subject: util/cbfstool: new utility pnortool
......................................................................

Patch Set 2:
> 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).

ended up using <commonlib/endian.h>, managed to do the trick.