Attention is currently required from: Filip Brozovic, Matt DeVillier, Sean Rhodes.
Angel Pons has posted comments on this change by Filip Brozovic. ( https://review.coreboot.org/c/coreboot/+/86080?usp=email )
Change subject: CFR: Add version field to root struct ......................................................................
Patch Set 1: Code-Review+1
(2 comments)
File src/commonlib/include/commonlib/cfr.h:
https://review.coreboot.org/c/coreboot/+/86080/comment/8ac37d14_00afbbf5?usp... : PS1, Line 52: 0x00010000 We can start at 0
File src/commonlib/include/commonlib/coreboot_tables.h:
https://review.coreboot.org/c/coreboot/+/86080/comment/89bdf1e4_b6e76f14?usp... : PS1, Line 605: major and minor I don't think we need a major/minor distinction. We only need this to be monotonically incremented every time some breaking change happens