Julius Werner 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 4:
If you use FMAP, define a version section explicitly, and fill the section in your build procedure where you can still access CONFIG_LOCAL_VERSION.
If FMAP is not an option, create a file like 'coreboot_version' and put that into CBFS may be easier, so you can still extract and read using cbfstool.
Anyone who wants to do this, please check the existing discussions in CB:40376 and CB:40377 first, as there are some opinions about how best to do this. I still maintain (like explained there) that I think this should definitely be a CBFS file and not an FMAP section. Not sure if there was agreement about that in the end and Furquan just stopped working on it or the issue is still contentious.