summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaisuke Nojiri <dnojiri@chromium.org>2020-07-10 14:17:18 -0700
committerCommit Bot <commit-bot@chromium.org>2020-07-14 03:03:49 +0000
commited23c08440dae6657d026a30b504578e03e0426f (patch)
tree1919e74b713f0a71b9b10605249982144bec7aab
parent0ae1e58736b59189cc336f41510e7a2a91489465 (diff)
downloadvboot-stabilize-13360.B.tar.gz
Reset EC when transitioning to dev modestabilize-13360.B
Currently, we reboot the AP (a.k.a. warm-reboot) to transition to dev mode in legacy_menu_ui; and reboot the EC (a.k.a. cold-reboot) in menu_ui and legacy_clamshell_ui. This patch makes legacy_menu_ui also reboot the EC. BUG=b:154228771 BRANCH=none TEST=make runtests Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org> Change-Id: Ie48c8ef5e8744102c8dbafb7929523c54b14d81d Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2293050 Reviewed-by: Joel Kitching <kitching@chromium.org>
-rw-r--r--firmware/lib/vboot_ui_legacy_menu.c2
-rw-r--r--tests/vboot_ui_legacy_menu_tests.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/firmware/lib/vboot_ui_legacy_menu.c b/firmware/lib/vboot_ui_legacy_menu.c
index c11e845f..f5165ad7 100644
--- a/firmware/lib/vboot_ui_legacy_menu.c
+++ b/firmware/lib/vboot_ui_legacy_menu.c
@@ -396,7 +396,7 @@ static vb2_error_t to_dev_action(struct vb2_context *ctx)
vb2_enable_developer_mode(ctx);
- return VB2_REQUEST_REBOOT;
+ return VB2_REQUEST_REBOOT_EC_TO_RO;
}
/* Action that disables developer mode, shows TO_NORM_CONFIRMED and reboots. */
diff --git a/tests/vboot_ui_legacy_menu_tests.c b/tests/vboot_ui_legacy_menu_tests.c
index fee6a3f9..1dc92535 100644
--- a/tests/vboot_ui_legacy_menu_tests.c
+++ b/tests/vboot_ui_legacy_menu_tests.c
@@ -1644,7 +1644,7 @@ static void VbBootRecTest(void)
mock_keyflags[i] = VB_KEY_FLAG_TRUSTED_KEYBOARD;
mock_keypress[i++] = VB_BUTTON_VOL_UP_SHORT_PRESS;
mock_keypress[i++] = VB_BUTTON_POWER_SHORT_PRESS;
- TEST_EQ(VbBootRecoveryLegacyMenu(ctx), VB2_REQUEST_REBOOT,
+ TEST_EQ(VbBootRecoveryLegacyMenu(ctx), VB2_REQUEST_REBOOT_EC_TO_RO,
"go to TO_DEV screen and confirm");
TEST_EQ(vb2_nv_get(ctx, VB2_NV_RECOVERY_REQUEST), 0, " no recovery");
TEST_EQ(debug_info_displayed, 0, " no debug info");