Julius Werner 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?
https://review.coreboot.org/c/coreboot/+/36209/9/src/security/vboot/ec_sync.... PS9, Line 425: ...and maybe one here so we can see how long we looped over limit power?