Andrey Pronin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46772 )
Change subject: security/vboot: fix policy digest for nvmem spaces ......................................................................
Patch Set 4:
(2 comments)
https://review.coreboot.org/c/coreboot/+/46772/3/src/security/vboot/secdata_... File src/security/vboot/secdata_tpm.c:
https://review.coreboot.org/c/coreboot/+/46772/3/src/security/vboot/secdata_... PS3, Line 111: #if CONFIG(CHROMEOS)
You don't need to add this, just update the code with the new version. […]
Done
https://review.coreboot.org/c/coreboot/+/46772/3/src/security/vboot/secdata_... PS3, Line 118: * 1) all zeros = initial, unextended state,
nit: Just to make things easier to reconstruct for people maybe also write each result digest here ( […]
Done