[coreboot-gerrit] Change in coreboot[master]: security/vboot: Add vboot callbacks to support EC software sync

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36208 )

Change subject: security/vboot: Add vboot callbacks to support EC software sync
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36208/8/src/security/vboot/ec_sync.... 
File src/security/vboot/ec_sync.c:

https://review.coreboot.org/c/coreboot/+/36208/8/src/security/vboot/ec_sync.... 
PS8, Line 232: int
> Note that we have a similar discussion going on here: […]

I think it makes sense to be consistent per-file. This file is vboot-specific so there's nothing wrong with still using vboot errors in here (and in some cases it's relevant, for the REBOOT_TO_RO_REQUIRED stuff).