This patch leads to a compile failure with glibc-headers-2.12-1.212 (currently installed on one of my systems) due to old endian.h in this package.

Is it missing le32toh()? I intentionally tried to standardize on something that I thought would be more widely available with this new commonlib/bsd code (as opposed to things like le32_to_cpu() which we also have in coreboot but I think are less widespread).

AFAIK there's no fully official POSIX API for little-endian stuff (only ntohl() for big-endian), so I think this is the next best thing we can try to rely on. Is this a big problem or are you okay with just requiring the newer version?

View Change

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