Shelley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46432 )
Change subject: security/vboot: Make mrc_cache hash functions generic ......................................................................
Patch Set 4:
(4 comments)
https://review.coreboot.org/c/coreboot/+/46432/3/src/security/vboot/mrc_cach... File src/security/vboot/mrc_cache_hash_tpm.c:
https://review.coreboot.org/c/coreboot/+/46432/3/src/security/vboot/mrc_cach... PS3, Line 92: "
Missing the '\n' now.
Done
https://review.coreboot.org/c/coreboot/+/46432/1/src/security/vboot/secdata_... File src/security/vboot/secdata_tpm.c:
https://review.coreboot.org/c/coreboot/+/46432/1/src/security/vboot/secdata_... PS1, Line 242: static uint32_t set_hash_space(uint32_t index, const uint8_t *data)
(Still open, btw)
Done in https://review.coreboot.org/c/coreboot/+/46434
https://review.coreboot.org/c/coreboot/+/46432/3/src/security/vboot/secdata_... File src/security/vboot/secdata_tpm.c:
https://review.coreboot.org/c/coreboot/+/46432/3/src/security/vboot/secdata_... PS3, Line 167: recovery_boot
nit: "recovery_boot" sounds a bit odd here (the important part is what kind of space we want to crea […]
Adding the if condition removed the need for this variable.
https://review.coreboot.org/c/coreboot/+/46432/3/src/security/vboot/secdata_... PS3, Line 168: "MRC Hash"
Might also be nice to change this... […]
Done