Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31346 )
Change subject: libpayload/sys/types.h: Add definition for off_t ......................................................................
Patch Set 2: Code-Review+2
But wouldn't a relation between `ssize_t` and `off_t` make it even more confusing? Generally, you don't know their size. So planting the idea that they are the same here, might con- fuse us again when we work with other definitions.
ssize_t is defined to be machine-word size (i.e. the same as uintptr_t)... at least on all systems that are and ever will be relevant to coreboot, I think. So for me that's a pretty clear one and way more tangible than "long".
Anyway, was just a suggestion, I'm fine with this as well if you prefer it.