[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 19:

(2 comments)

https://review.coreboot.org/c/coreboot/+/36209/19/src/commonlib/include/comm... 
File src/commonlib/include/commonlib/timestamp_serialized.h:

https://review.coreboot.org/c/coreboot/+/36209/19/src/commonlib/include/comm... 
PS19, Line 74: 	TS_START_HASH_BODY = 507,
             : 	TS_DONE_LOADING = 508,
             : 	TS_DONE_HASHING_FW_MAIN = 509,
> Yes, but I liked "BODY" better as the terminology here tbh.

Done

https://review.coreboot.org/c/coreboot/+/36209/19/src/commonlib/include/comm... 
PS19, Line 199: 	{ TS_END_HASH_BODY,	"finished verifying body signature (RSA)" },
> All these timestamps talk about "body" either change them all or none of them (I don't see why it ne […]

It's all the same to me, I'll change it back.