diff options
author | Yu-Ping Wu <yupingso@chromium.org> | 2020-06-09 17:07:14 +0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-06-23 04:29:53 +0000 |
commit | be49fbfdf0b08df8e8a840aad1b91b4a7b6010db (patch) | |
tree | 9aa7c833df6ae2ae153820e4a9a58f0d194cf39b | |
parent | 23f72259612eccc4557ee7e2c40f3ea2fabf9c11 (diff) | |
download | vboot-be49fbfdf0b08df8e8a840aad1b91b4a7b6010db.tar.gz |
vboot/ui: add power off options
Add power off options to screens.
BRANCH=none
BUG=b:146399181, b:145098577
TEST=make runtests
TEST=USE="menu_ui" emerge-nami depthcharge
Cq-Depend: chromium:2237367
Change-Id: I356c56361a4282b456c3a4661e518a84944e983b
Signed-off-by: Yu-Ping Wu <yupingso@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2237366
Reviewed-by: Joel Kitching <kitching@chromium.org>
-rw-r--r-- | firmware/2lib/2ui_screens.c | 25 | ||||
-rw-r--r-- | tests/vb2_ui_tests.c | 17 |
2 files changed, 36 insertions, 6 deletions
diff --git a/firmware/2lib/2ui_screens.c b/firmware/2lib/2ui_screens.c index 2ac3f5e9..b41146d7 100644 --- a/firmware/2lib/2ui_screens.c +++ b/firmware/2lib/2ui_screens.c @@ -40,6 +40,17 @@ .target = VB2_SCREEN_ADVANCED_OPTIONS, \ } +/* Action that will power off the device. */ +static vb2_error_t power_off_action(struct vb2_ui_context *ui) +{ + return VB2_REQUEST_SHUTDOWN; +} + +#define POWER_OFF_ITEM { \ + .text = "Power off", \ + .action = power_off_action, \ +} + /******************************************************************************/ /* VB2_SCREEN_BLANK */ @@ -131,6 +142,7 @@ static const struct vb2_screen_info language_select_screen = { static const struct vb2_menu_item recovery_broken_items[] = { LANGUAGE_SELECT_ITEM, ADVANCED_OPTIONS_ITEM, + POWER_OFF_ITEM, }; static const struct vb2_screen_info recovery_broken_screen = { @@ -165,6 +177,7 @@ static const struct vb2_menu_item advanced_options_items[] = { .target = VB2_SCREEN_RECOVERY_TO_DEV, }, [ADVANCED_OPTIONS_ITEM_BACK] = BACK_ITEM, + POWER_OFF_ITEM, }; static const struct vb2_screen_info advanced_options_screen = { @@ -203,6 +216,7 @@ static const struct vb2_menu_item recovery_select_items[] = { .target = VB2_SCREEN_RECOVERY_DISK_STEP1, }, ADVANCED_OPTIONS_ITEM, + POWER_OFF_ITEM, }; static const struct vb2_screen_info recovery_select_screen = { @@ -217,6 +231,7 @@ static const struct vb2_screen_info recovery_select_screen = { static const struct vb2_menu_item recovery_invalid_items[] = { LANGUAGE_SELECT_ITEM, + POWER_OFF_ITEM, }; static const struct vb2_screen_info recovery_invalid_screen = { @@ -321,6 +336,7 @@ static const struct vb2_menu_item recovery_to_dev_items[] = { .text = "Cancel", .action = vb2_ui_change_root, }, + POWER_OFF_ITEM, }; static const struct vb2_screen_info recovery_to_dev_screen = { @@ -338,6 +354,7 @@ static const struct vb2_menu_item recovery_phone_step1_items[] = { LANGUAGE_SELECT_ITEM, NEXT_ITEM(VB2_SCREEN_RECOVERY_PHONE_STEP2), BACK_ITEM, + POWER_OFF_ITEM, }; static const struct vb2_screen_info recovery_phone_step1_screen = { @@ -352,6 +369,7 @@ static const struct vb2_screen_info recovery_phone_step1_screen = { static const struct vb2_menu_item recovery_phone_step2_items[] = { LANGUAGE_SELECT_ITEM, BACK_ITEM, + POWER_OFF_ITEM, }; static const struct vb2_screen_info recovery_phone_step2_screen = { @@ -367,6 +385,7 @@ static const struct vb2_menu_item recovery_disk_step1_items[] = { LANGUAGE_SELECT_ITEM, NEXT_ITEM(VB2_SCREEN_RECOVERY_DISK_STEP2), BACK_ITEM, + POWER_OFF_ITEM, }; static const struct vb2_screen_info recovery_disk_step1_screen = { @@ -382,6 +401,7 @@ static const struct vb2_menu_item recovery_disk_step2_items[] = { LANGUAGE_SELECT_ITEM, NEXT_ITEM(VB2_SCREEN_RECOVERY_DISK_STEP3), BACK_ITEM, + POWER_OFF_ITEM, }; static const struct vb2_screen_info recovery_disk_step2_screen = { @@ -396,6 +416,7 @@ static const struct vb2_screen_info recovery_disk_step2_screen = { static const struct vb2_menu_item recovery_disk_step3_items[] = { LANGUAGE_SELECT_ITEM, BACK_ITEM, + POWER_OFF_ITEM, }; static const struct vb2_screen_info recovery_disk_step3_screen = { @@ -539,6 +560,7 @@ static const struct vb2_menu_item developer_mode_items[] = { .action = vb2_ui_developer_mode_boot_external_action, }, ADVANCED_OPTIONS_ITEM, + POWER_OFF_ITEM, }; static const struct vb2_screen_info developer_mode_screen = { @@ -574,6 +596,7 @@ static const struct vb2_menu_item developer_to_norm_items[] = { .text = "Cancel", .action = vb2_ui_change_root, }, + POWER_OFF_ITEM, }; static const struct vb2_screen_info developer_to_norm_screen = { @@ -588,6 +611,7 @@ static const struct vb2_screen_info developer_to_norm_screen = { static const struct vb2_menu_item developer_boot_external_items[] = { LANGUAGE_SELECT_ITEM, BACK_ITEM, + POWER_OFF_ITEM, }; static const struct vb2_screen_info developer_boot_external_screen = { @@ -603,6 +627,7 @@ static const struct vb2_screen_info developer_boot_external_screen = { static const struct vb2_menu_item developer_invalid_disk_items[] = { LANGUAGE_SELECT_ITEM, BACK_ITEM, + POWER_OFF_ITEM, }; static const struct vb2_screen_info developer_invalid_disk_screen = { diff --git a/tests/vb2_ui_tests.c b/tests/vb2_ui_tests.c index 676f16fa..407a081f 100644 --- a/tests/vb2_ui_tests.c +++ b/tests/vb2_ui_tests.c @@ -694,6 +694,17 @@ static void developer_tests(void) TEST_EQ(vb2_nv_get(ctx, VB2_NV_DISABLE_DEV_REQUEST), 0, " disable dev request"); + /* Power off */ + reset_common_data(FOR_DEVELOPER); + add_mock_vbtlk(VB2_SUCCESS, VB_DISK_FLAG_FIXED); + /* Navigate to the bottom most menu item */ + add_mock_keypress(VB_KEY_DOWN); + add_mock_keypress(VB_KEY_DOWN); + add_mock_keypress(VB_KEY_DOWN); + add_mock_keypress(VB_KEY_ENTER); + TEST_EQ(vb2_developer_menu(ctx), VB2_REQUEST_SHUTDOWN, + "select power off"); + VB2_DEBUG("...done.\n"); } @@ -1077,7 +1088,6 @@ static void developer_screen_tests(void) add_mock_keypress(VB_KEY_ESC); add_mock_keypress(VB_KEY_DOWN); add_mock_keypress(VB_KEY_DOWN); - add_mock_keypress(VB_KEY_DOWN); /* Blocked */ TEST_EQ(vb2_developer_menu(ctx), VB2_REQUEST_SHUTDOWN, "dev mode screen"); /* #4: Advanced options */ @@ -1113,7 +1123,6 @@ static void developer_screen_tests(void) add_mock_keypress(VB_KEY_DOWN); add_mock_keypress(VB_KEY_DOWN); add_mock_keypress(VB_KEY_ENTER); - add_mock_keypress(VB_KEY_DOWN); extend_calls_until_shutdown(); TEST_EQ(vb2_developer_menu(ctx), VB2_REQUEST_SHUTDOWN, "advanced options screen"); @@ -1157,7 +1166,6 @@ static void broken_recovery_screen_tests(void) add_mock_keypress(VB_KEY_ENTER); /* End of menu */ add_mock_keypress(VB_KEY_ESC); - add_mock_keypress(VB_KEY_DOWN); /* Blocked */ extend_calls_until_shutdown(); TEST_EQ(vb2_broken_recovery_menu(ctx), VB2_REQUEST_SHUTDOWN, "broken screen"); @@ -1190,7 +1198,6 @@ static void broken_recovery_screen_tests(void) add_mock_keypress(VB_KEY_ENTER); /* End of menu */ add_mock_keypress(VB_KEY_ENTER); - add_mock_keypress(VB_KEY_DOWN); /* Blocked */ extend_calls_until_shutdown(); TEST_EQ(vb2_broken_recovery_menu(ctx), VB2_REQUEST_SHUTDOWN, "advanced options screen"); @@ -1239,7 +1246,6 @@ static void manual_recovery_screen_tests(void) add_mock_keypress(VB_KEY_ESC); add_mock_keypress(VB_KEY_DOWN); add_mock_keypress(VB_KEY_DOWN); - add_mock_keypress(VB_KEY_DOWN); /* Blocked */ extend_calls_until_shutdown(); TEST_EQ(vb2_manual_recovery_menu(ctx), VB2_REQUEST_SHUTDOWN, "recovery select screen"); @@ -1300,7 +1306,6 @@ static void manual_recovery_screen_tests(void) add_mock_keypress(VB_KEY_DOWN); add_mock_keypress(VB_KEY_ENTER); add_mock_keypress(VB_KEY_DOWN); - add_mock_keypress(VB_KEY_DOWN); /* Blocked */ extend_calls_until_shutdown(); TEST_EQ(vb2_manual_recovery_menu(ctx), VB2_REQUEST_SHUTDOWN, "advanced options screen"); |