Marty E. Plummer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36346 )
Change subject: cbfs: store header offset as absolute value BE from fmap_top ......................................................................
Patch Set 1:
Hell, I'm even up for the idea of a bigish overhaul of cbfs, in that it will store these header and other similar values in be/le depending on the build arch (eg, for most everything the values will be stored as LE and for big-endian ppc64 it would store them as be). Would take out a lot of the ntohl and so on in the coreboot code itself and reduce some of the overhead (granted, byte swizzling is pretty low cost in terms of instructions).