diff options
| author | Shawn Nematbakhsh <shawnn@google.com> | 2012-10-30 15:17:33 -0700 |
|---|---|---|
| committer | Shawn Nematbakhsh <shawnn@google.com> | 2012-10-31 17:59:21 -0700 |
| commit | d5071c1c30e8a3e30987ee600d8ae0f194ec27ea (patch) | |
| tree | fc0c0918366654df37574b1bedc88704f25e7141 | |
| parent | a9b1248fa6d41a7ff9375c028cbc2235eafd1424 (diff) | |
| download | vboot-d5071c1c30e8a3e30987ee600d8ae0f194ec27ea.tar.gz | |
STOUT-FIRMWARE: vboot_reference: Fix keyboard input lag at confirm screen.
Changing languages is terribly slow at the confirm screen, when
switching from dev to normal. Reduce sleep time to improve user
experience.
BUG=chrome-os-partner:15726
TEST=boot in dev, hit space, hit arrows rapidly to change language,
observe no lag.
BRANCH=butterfly, stout
Change-Id: Ic256b203725d0a096e9129ad2dbb417c5458cec2
Original-Change-Id: I0943debc31d78dcfce87e7f7d4537ae47f5f8cfd
Reviewed-on: https://gerrit.chromium.org/gerrit/36988
Tested-by: Shawn Nematbakhsh <shawnn@google.com>
Reviewed-by: Shawn Nematbakhsh <shawnn@google.com>
| -rw-r--r-- | firmware/lib/vboot_api_kernel.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/firmware/lib/vboot_api_kernel.c b/firmware/lib/vboot_api_kernel.c index 378400f0..87018f1c 100644 --- a/firmware/lib/vboot_api_kernel.c +++ b/firmware/lib/vboot_api_kernel.c @@ -105,6 +105,8 @@ uint32_t VbTryLoadKernel(VbCommonParams* cparams, LoadKernelParams* p, return retval; } +#define CONFIRM_KEY_DELAY 20 /* Check confirm screen keys every 20ms */ + /* Ask the user to confirm something. We should display whatever the question * is first, then call this. ESC is always "no", ENTER is always "yes", and * we'll specify what SPACE means. We don't return until one of those keys is @@ -139,7 +141,7 @@ static int VbUserConfirms(VbCommonParams* cparams, int space_means_no) { default: VbCheckDisplayKey(cparams, key, &vnc); } - VbExSleepMs(1000); + VbExSleepMs(CONFIRM_KEY_DELAY); } /* not reached, but compiler will complain without it */ return -1; |