[coreboot-gerrit] Change in coreboot[master]: security/vboot: Decouple measured boot from verified boot

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35077 )

Change subject: security/vboot: Decouple measured boot from verified boot
......................................................................

Patch Set 70: Code-Review+2

(3 comments)

https://review.coreboot.org/c/coreboot/+/35077/70/src/security/tpm/tspi/crtm... 
File src/security/tpm/tspi/crtm.c:

https://review.coreboot.org/c/coreboot/+/35077/70/src/security/tpm/tspi/crtm... 
PS70, Line 30: static const char * const fmap_cbfs_names[] = {
             : 	"COREBOOT",
             : 	"FW_MAIN_A",
             : 	"FW_MAIN_B",
             : 	"RW_LEGACY"};
this isn't really compliant with the code style. How about:

static const char *const fmap_cbfs_names[] = {
    "COREBOOT",
    "FW_MAIN_A",
    "FW_MAIN_B",
    "RW_LEGACY",
 };

https://review.coreboot.org/c/coreboot/+/35077/70/src/security/tpm/tspi/tspi... 
File src/security/tpm/tspi/tspi.c:

https://review.coreboot.org/c/coreboot/+/35077/70/src/security/tpm/tspi/tspi... 
PS70, Line 235: 		printk(BIOS_DEBUG, "TPM: Extending digest for"
This string fits in one line without breaking (96-char limit)

https://review.coreboot.org/c/coreboot/+/35077/70/src/security/tpm/tspi/tspi... 
PS70, Line 302: "measur" : "logg"
Ehm, I don't think saving two characters is that much of a big deal... Could we have full words in both cases of the ternary operator, please?