diff options
author | Shelley Chen <shchen@chromium.org> | 2017-02-15 11:06:13 -0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2017-03-02 09:33:57 -0800 |
commit | 05f00584796fc5aca01f4a4587158d60bded9d49 (patch) | |
tree | 2667b6be829fd35f040b8fb9d134606e489b80ea | |
parent | 19a41188cd50b5cdcd6fc50b718031a3d469c025 (diff) | |
download | vboot-05f00584796fc5aca01f4a4587158d60bded9d49.tar.gz |
poppy: use return key for selection
Replacing right arrow key with return for selection
when keyboard is available because it's more obvious.
BUG=b:35585623
BRANCH=None
TEST=reboot and make sure return selects menu item.
Change-Id: I6b2fbd8cddcd98c49638a6b9d79b47da1ca6545f
Signed-off-by: Shelley Chen <shchen@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/444944
Reviewed-by: Randall Spangler <rspangler@chromium.org>
-rw-r--r-- | firmware/lib/vboot_ui_menu.c | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/firmware/lib/vboot_ui_menu.c b/firmware/lib/vboot_ui_menu.c index 7cf18525..ef428610 100644 --- a/firmware/lib/vboot_ui_menu.c +++ b/firmware/lib/vboot_ui_menu.c @@ -666,10 +666,6 @@ VbError_t vb2_developer_menu(struct vb2_context *ctx, VbCommonParams *cparams) case 0: /* Nothing pressed */ break; - case '\r': - /* Only disable virtual dev switch if allowed by GBB */ - if (!(gbb->flags & GBB_FLAG_ENTER_TRIGGERS_TONORM)) - break; case 0x04: /* Ctrl+D = dismiss warning; advance to timeout */ VB2_DEBUG("user pressed Ctrl+D; skip delay\n"); @@ -732,7 +728,7 @@ VbError_t vb2_developer_menu(struct vb2_context *ctx, VbCommonParams *cparams) vb2_print_current_menu(); break; case VB_BUTTON_POWER: - case VB_KEY_RIGHT: + case '\r': selected = 1; ret = vb2_update_menu(); @@ -979,7 +975,6 @@ VbError_t vb2_recovery_menu(struct vb2_context *ctx, VbCommonParams *cparams) break; case VB_BUTTON_VOL_UP: case VB_KEY_UP: - VB2_DEBUG("pressed key VB_KEY_UP\n"); vb2_get_current_menu_size(current_menu, NULL, &menu_size); if (current_menu_idx > 0) current_menu_idx--; @@ -987,15 +982,13 @@ VbError_t vb2_recovery_menu(struct vb2_context *ctx, VbCommonParams *cparams) break; case VB_BUTTON_VOL_DOWN: case VB_KEY_DOWN: - VB2_DEBUG("pressed key VB_KEY_DOWN\n"); vb2_get_current_menu_size(current_menu, NULL, &menu_size); if (current_menu_idx < menu_size-1) current_menu_idx++; vb2_print_current_menu(); break; case VB_BUTTON_POWER: - case VB_KEY_RIGHT: - VB2_DEBUG("pressed key VB_KEY_RIGHT (SELECT)\n"); + case '\r': selected = 1; ret = vb2_update_menu(); |