commit d5071c1c30e8a3e30987ee600d8ae0f194ec27ea
author Shawn Nematbakhsh <shawnn@google.com> Tue Oct 30 15:17:33 2012 -0700
committer Shawn Nematbakhsh <shawnn@google.com> Wed Oct 31 17:59:21 2012 -0700
tree fc0c0918366654df37574b1bedc88704f25e7141
parent a9b1248fa6d41a7ff9375c028cbc2235eafd1424

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>

firmware/lib/vboot_api_kernel.c

diff --git a/firmware/lib/vboot_api_kernel.c b/firmware/lib/vboot_api_kernel.c
index 378400f..87018f1 100644
--- a/firmware/lib/vboot_api_kernel.c
+++ b/firmware/lib/vboot_api_kernel.c
@@ -105,6 +105,8 @@
   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 @@
     default:
       VbCheckDisplayKey(cparams, key, &vnc);
     }
-    VbExSleepMs(1000);
+    VbExSleepMs(CONFIRM_KEY_DELAY);
   }
   /* not reached, but compiler will complain without it */
   return -1;