Johnny Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42029 )
Change subject: drivers/vpd: Add a function that reads BIOS version from a VPD variable ......................................................................
Patch Set 6:
Patch Set 6:
Updated the commit message to reflect the special usage for updating BIOS version w/o rebuilding from source, it's for internal build and test processes.
If this is simply for testing, I'd say cbfs file would be the better (and more correct) choice than VPD, since VPDs are supposed to be preserved (e.g., no update) during update while CBFS is not (will be updated).
Our plan for VPD usage would be the place where we store the BIOS setting variables (e.g., turbo mode on/off, FSP UPD values .,etc). CB:41732 mentioned this, the RO_VPD would hold the default values and RW_VPD can be updated from user to overwrite the values under Linux. So it's likely we would update VPD for different versions.