| commit | 6caf4cd6225fcd1a636fd681970bd2221d63455c | [log] [tgz] |
|---|---|---|
| author | Subrata Banik <[email protected]> | Fri Feb 07 16:05:15 2025 +0530 |
| committer | Chromeos LUCI <[email protected]> | Fri Feb 07 18:22:35 2025 -0800 |
| tree | ffb385389c3ed76f0d7903d5b0474e65814e940e | |
| parent | 1f8153d95a931dca93f710531cdd333207c57982 [diff] |
firmware/2lib: Remove redundant EC sync check in recovery This patch removes the redundant EC sync check in recovery mode. The check is already performed by vb2api_ec_sync, the caller of `ec_sync_phase2`/`ec_sync_phase1`. This change also allows ec_sync_allowed to be used by `ec_sync_phase1` to ensure EC sync prerequisites are met. BUG=b:395038430 TEST=Built and booted google/fatcat. Change-Id: If64f1c5c768c4e7ddbcbae54500ce69f5631a07b Signed-off-by: Subrata Banik <[email protected]> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/6242478 Commit-Queue: Subrata Banik <[email protected]> Tested-by: Subrata Banik <[email protected]> Reviewed-by: Julius Werner <[email protected]>
diff --git a/firmware/2lib/2ec_sync.c b/firmware/2lib/2ec_sync.c index 6475dc4..911278e 100644 --- a/firmware/2lib/2ec_sync.c +++ b/firmware/2lib/2ec_sync.c
@@ -313,8 +313,7 @@ return 0; if (gbb->flags & VB2_GBB_FLAG_DISABLE_EC_SOFTWARE_SYNC) return 0; - if (ctx->flags & VB2_CONTEXT_RECOVERY_MODE) - return 0; + return 1; }