Attention is currently required from: Julius Werner, Yu-Ping Wu.
Hello Julius Werner, Yu-Ping Wu,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/75521?usp=email
to look at the new patch set (#2).
Change subject: vboot: Drop argument to select slot from `vb2ex_ec_protect()` ......................................................................
vboot: Drop argument to select slot from `vb2ex_ec_protect()`
vboot code changes has eliminated the redundant call to WP the EC-RO region as protecting RW flash implies protecting both RO and RW flash, so the call to protect RO is redundant. google/rex is currently takes about 17 ms to lock down the EC.
Along with vboot changes, this patch drops argument to choose between RO and RW slot to protect while calling into `vb2ex_ec_protect()`. It ensures vb2ex_ec_protect() is explicitly meant for protecting RW regions.
BUG=b:281918204 TEST=Able to build and boot google/rex and observed boot time savings.
Change-Id: I2974f0cb43ba800c2aaeac4876ebaa052b5ee793 Signed-off-by: Subrata Banik subratabanik@google.com --- M src/security/vboot/ec_sync.c 1 file changed, 4 insertions(+), 7 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/21/75521/2