[coreboot-gerrit] Change in coreboot[master]: lib/coreboot_table: Add Intel FSP version to coreboot table

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41809 )

Change subject: lib/coreboot_table: Add Intel FSP version to coreboot table
......................................................................

Patch Set 16:

(1 comment)

https://review.coreboot.org/c/coreboot/+/41809/16/src/commonlib/include/comm... 
File src/commonlib/include/commonlib/coreboot_tables.h:

https://review.coreboot.org/c/coreboot/+/41809/16/src/commonlib/include/comm... 
PS16, Line 82: 	LB_TAG_CMOS_OPTION_TABLE	= 0x00c8,
             : 	LB_TAG_OPTION			= 0x00c9,
             : 	LB_TAG_OPTION_ENUM		= 0x00ca,
             : 	LB_TAG_OPTION_DEFAULTS		= 0x00cb,
             : 	LB_TAG_OPTION_CHECKSUM		= 0x00cc,
These are actually a group of tags starting from 200 (somehow this
information got lost during refactoring). The new entry should be added
above as 0x38.

And I would still prefer to use a vendor-neutral tag name, e.g.
`LB_TAG_PLATFORM_BLOB_VERSION`.