[coreboot-gerrit] Change in coreboot[master]: coreboot-id: Add coreboot id based on FMAP entries

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40377 )

Change subject: coreboot-id: Add coreboot id based on FMAP entries
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/40377/2/src/mainboard/siemens/mc_ap... 
File src/mainboard/siemens/mc_apl1/mc_apl_vboot.fmd:

https://review.coreboot.org/c/coreboot/+/40377/2/src/mainboard/siemens/mc_ap... 
PS2, Line 9: COREBOOT_ID 0x40
           : 			RO_FRID_PAD 0x780
> I have built this patchset for mc_apl5 which uses a vboot scheme. […]

You are right. With this change, there is not much value in adding the ID for vboot-enabled boards since it is just adding redundant info. I can skip it for vboot enabled boards in general.