Attention is currently required from: Maximilian Brune.
Nico Huber has posted comments on this change by Maximilian Brune. ( https://review.coreboot.org/c/coreboot/+/83617?usp=email )
Change subject: util/cbfstool/cbfs-payload-linux: Add error handling ......................................................................
Patch Set 1: Code-Review+2
(1 comment)
File util/cbfstool/cbfs-payload-linux.c:
https://review.coreboot.org/c/coreboot/+/83617/comment/a1ec07cc_7a18626f?usp... : PS1, Line 177: return ret; Using specific return codes above and then mixing the return code of the compression function in, seems a bit odd. Kind of more work without gaining us anything because the compression function could also return `-1` or `-2` and then the caller still wouldn't know what is what.
So, tldr; I'd probably make this `-3`, or always use `-1`.