diff options
author | Duncan Laurie <dlaurie@google.com> | 2018-11-15 16:03:13 -0700 |
---|---|---|
committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2018-11-15 23:20:09 +0000 |
commit | b3bc8409c60eca70abfcab579a6bf3a63aaa4a80 (patch) | |
tree | a6bb77d6c7d1e58dc2c1c6ef91b7d3061f1a4b48 | |
parent | 11997fc7e7755ed44cd65b37ac9bedec8c06c8ef (diff) | |
download | vboot-firmware-eve-9584.B.tar.gz |
Revert "ec_sync: Only do aux fw update if also doing EC update"firmware-eve-9584.B
This reverts commit f5df3b0a6d1788c663f39d94d8a2f3ceba9dd3e5.
Attempt anx3429 update without EC software sync happening.
BUG=b:74336712
BRANCH=eve
TEST=manual
Change-Id: Icae986242bf0b96f11b3decea77f9565d7cb2d10
Signed-off-by: Duncan Laurie <dlaurie@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/1338371
-rw-r--r-- | firmware/lib/ec_sync.c | 3 | ||||
-rw-r--r-- | firmware/lib/ec_sync_all.c | 15 |
2 files changed, 4 insertions, 14 deletions
diff --git a/firmware/lib/ec_sync.c b/firmware/lib/ec_sync.c index f5ceb110..eaedd9d3 100644 --- a/firmware/lib/ec_sync.c +++ b/firmware/lib/ec_sync.c @@ -449,11 +449,8 @@ VbError_t ec_sync_check_aux_fw(struct vb2_context *ctx, VbCommonParams *cparams, VbAuxFwUpdateSeverity_t *severity) { - struct vb2_shared_data *sd = vb2_get_sd(ctx); - /* If we're not updating the EC, skip aux fw syncs as well */ if (!ec_sync_allowed(ctx, cparams) || - !(sd->flags & VB2_SD_FLAG_ECSYNC_EC_RW) || (cparams->gbb->flags & GBB_FLAG_DISABLE_PD_SOFTWARE_SYNC)) { *severity = VB_AUX_FW_NO_UPDATE; return VBERROR_SUCCESS; diff --git a/firmware/lib/ec_sync_all.c b/firmware/lib/ec_sync_all.c index 45da682b..09727152 100644 --- a/firmware/lib/ec_sync_all.c +++ b/firmware/lib/ec_sync_all.c @@ -46,19 +46,12 @@ VbError_t ec_sync_all(struct vb2_context *ctx, struct VbCommonParams *cparams) VbAuxFwUpdateSeverity_t fw_update; VbError_t rv; - /* Do EC sync phase 1; this determines if we need an update */ - VbError_t phase1_rv = ec_sync_phase1(ctx, cparams); - - /* - * Check for AUX firmware updates to be performed. - * Continue on errors to prevent reboot loop on persistent failure. - */ rv = ec_sync_check_aux_fw(ctx, cparams, &fw_update); - if (rv) { - VB2_DEBUG("Skipping AUX FW update.\n"); - fw_update = VB_AUX_FW_NO_UPDATE; - } + if (rv) + return rv; + /* Do EC sync phase 1; this determines if we need an update */ + VbError_t phase1_rv = ec_sync_phase1(ctx, cparams); int need_wait_screen = ec_will_update_slowly(ctx, cparams) || (fw_update == VB_AUX_FW_SLOW_UPDATE); |