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

[build bot (Jenkins) (Code Review)]

build bot (Jenkins) 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 57:

(3 comments)

https://review.coreboot.org/c/coreboot/+/35077/57/src/include/memlayout.h 
File src/include/memlayout.h:

https://review.coreboot.org/c/coreboot/+/35077/57/src/include/memlayout.h@16... 
PS57, Line 163: #define TPM_TCPA_LOG(addr, size) \
Macros with multiple statements should be enclosed in a do - while loop

https://review.coreboot.org/c/coreboot/+/35077/57/src/include/memlayout.h@16... 
PS57, Line 163: #define TPM_TCPA_LOG(addr, size) \
macros should not use a trailing semicolon

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

https://review.coreboot.org/c/coreboot/+/35077/57/src/security/tpm/tspi/crtm... 
PS57, Line 128: 		} else {
else is not generally useful after a break or return