[coreboot-gerrit] Change in coreboot[master]: security/vboot/sync_ec: Add timestamps

[Tim Wawrzynczak (Code Review)]

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 15:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36209/13/src/security/vboot/ec_sync... 
File src/security/vboot/ec_sync.c:

https://review.coreboot.org/c/coreboot/+/36209/13/src/security/vboot/ec_sync... 
PS13, Line 159: timestamp_add_now(TS_EC_HASH_READY);
> Should we add it only when we return success? That will be consistent with other error scenarios whe […]

I figured leave it in, even if it returns the wrong type or doesn't finish, we'll have a timestamp saying it at least went through this path.  Thoughts?