build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41117 )
Change subject: cbfstool: Use cbfs_serialized.h and standard vboot helpers ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41117/1/util/cbfstool/cbfs.h File util/cbfstool/cbfs.h:
https://review.coreboot.org/c/coreboot/+/41117/1/util/cbfstool/cbfs.h@66 PS1, Line 66: static inline size_t cbfs_file_attr_hash_size(enum vb2_hash_algorithm algo) { open brace '{' following function definitions go on the next line
https://review.coreboot.org/c/coreboot/+/41117/1/util/cbfstool/cbfs_image.c File util/cbfstool/cbfs_image.c:
https://review.coreboot.org/c/coreboot/+/41117/1/util/cbfstool/cbfs_image.c@... PS1, Line 1534: switch (ntohl(entry->type)) { switch and case should be at the same indent