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).

I hate the endian conversions too but I think this fails with the same problem... CBFS is just too much of a standard and has been reimplemented in too many places to make this worthwhile. I think most coreboot users would probably prefer if we keep the slightly gnarly but downwards-compatible thing we have. (If you really want to make such a big change you should probably bring it up on the mailing list first and see what others think.)

View Change

To view, visit change 36346. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Idc99b0c133faa2aa15d06f998e7371d332ffa490
Gerrit-Change-Number: 36346
Gerrit-PatchSet: 1
Gerrit-Owner: Marty E. Plummer <hanetzer@startmail.com>
Gerrit-Reviewer: Julius Werner <jwerner@chromium.org>
Gerrit-Reviewer: Marty E. Plummer <hanetzer@startmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus@gmail.com>
Gerrit-Comment-Date: Sat, 26 Oct 2019 01:38:58 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment