[coreboot-gerrit] [S] Change in coreboot[main]: CFR: Add min/max/step values and hex display flag for number options

[Angel Pons (Code Review)]

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/+/86039?usp=email )

Change subject: CFR: Add min/max/step values and hex display flag for number options
......................................................................

Patch Set 3:

(1 comment)

File src/commonlib/include/commonlib/cfr.h:

https://review.coreboot.org/c/coreboot/+/86039/comment/d30c9277_c94ffd82?usp... :
PS2, Line 139: uint32_t min;
             : 	uint32_t max;
             : 	uint32_t step;
> Should we add a fixed version number to the header in coreboot, change `lb_cfr_numeric_option` to ad […]

Yes, though the main idea of a CFR version number is to prevent parsing incompatible data structures, i.e. if the version number doesn't match, don't even try to parse anything. It's simpler than supporting multiple versions, and everything should be using the latest version anyway.