Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36209 )
Change subject: security/vboot/sync_ec: Add timestamps ......................................................................
Patch Set 9:
(2 comments)
https://review.coreboot.org/c/coreboot/+/36209/9/src/security/vboot/ec_sync.... File src/security/vboot/ec_sync.c:
https://review.coreboot.org/c/coreboot/+/36209/9/src/security/vboot/ec_sync.... PS9, Line 148:
Might be worth to put another one here to see how long the hash took to become ready?
Sounds reasonable to me.
https://review.coreboot.org/c/coreboot/+/36209/9/src/security/vboot/ec_sync.... PS9, Line 425:
... […]
Also sounds reasonable to me.