diff options
| author | Mary Ruthven <mruthven@chromium.org> | 2022-04-25 13:36:13 -0500 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-04-25 21:02:13 +0000 |
| commit | eb9f26b0f47129791a69d86deac0b17a2caace28 (patch) | |
| tree | 8a84cf4789b243aa2f52570c69a5a222fff24d9e | |
| parent | 901bd36869257d4e5fadf6e5d63dea998c9300a1 (diff) | |
| download | chrome-ec-eb9f26b0f47129791a69d86deac0b17a2caace28.tar.gz | |
ap_ro_integrity_check: delay sleep after deasserting AP FLASH SEL
Delay sleep to give AP_FLASH_SELECT enough time to discharge. Future CLs
will do more to ensure AP_FLASH_SELECT isn't asserted entering deep
sleep. This CL does the bare minimum to fix AP RO verification.
BUG=b:229974371
TEST=Trigger AP RO verification on Hoglin
Change-Id: Iec10c51dfe8e7df2b1bb2210c4705d90c3c89c54
Signed-off-by: Mary Ruthven <mruthven@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3606093
Reviewed-by: Andrey Pronin <apronin@chromium.org>
| -rw-r--r-- | common/ap_ro_integrity_check.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/common/ap_ro_integrity_check.c b/common/ap_ro_integrity_check.c index 59ce15a25b..7724ae2585 100644 --- a/common/ap_ro_integrity_check.c +++ b/common/ap_ro_integrity_check.c @@ -1390,6 +1390,8 @@ static void release_ec_reset_override(void) { hook_call_deferred(&keep_ec_in_reset_data, -1); deassert_ec_rst(); + /* b/229974371 Give AP_FLASH_SELECT at least 500us to discharge */ + delay_sleep_by(1 * SECOND); enable_sleep(SLEEP_MASK_AP_RO_VERIFICATION); } |