[coreboot-gerrit] Change in coreboot[master]: vboot/secdata_tpm: Create FWMP space in coreboot

[Julius Werner (Code Review)]

Attention is currently required from: Joel Kitching, Andrey Pronin, Aseda Aboagye, Aaron Durbin.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52919 )

Change subject: vboot/secdata_tpm: Create FWMP space in coreboot
......................................................................

Patch Set 7: Code-Review+2

(1 comment)

File src/security/vboot/secdata_tpm.c:

https://review.coreboot.org/c/coreboot/+/52919/comment/d894f8e6_4e2cd975 
PS1, Line 420: 	/*
             : 	 * Set initial values of secdata_firmware space.
             : 	 * kernel space is created in _factory_initialize_tpm().
             : 	 */
             : 	vb2api_secdata_firmware_create(ctx);
> Does anyone know if there's any particular reason this is done prior to _factory_initialize_tpm()? I […]

No, shouldn't be. I think this was just put here to minimize duplication between TPM1/TPM2 parts.