diff options
| author | Kenji Chen <kenji.chen@intel.com> | 2014-10-26 21:58:40 +0800 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-10-30 23:49:57 +0000 |
| commit | 73c71881d296198a0ade18b6226f8a546b0d52b9 (patch) | |
| tree | ceb4fd83948c05d9d082fdbc9ac461e904a43629 | |
| parent | 929cdb94e39e39e66b7c34795febfa1db1782de0 (diff) | |
| download | chrome-ec-73c71881d296198a0ade18b6226f8a546b0d52b9.tar.gz | |
EC:KBC: Wait until LPC host senses the IRQ and gets the character.
BRANCH=master
BUG=chrome-os-partner:29139
TEST=Buiid an EC FW image and run on Rambi to test if key loss is
improved and any side effect somes with this change. Need more test
units to confirm this.
Signed-off-by: Kenji Chen <kenji.chen@intel.com>
Change-Id: I2399e33d2ca3defe8cd9b1f94ab0af1db7f84635
Reviewed-on: https://chromium-review.googlesource.com/225557
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Reviewed-by: Mohammed Habibulla <moch@chromium.org>
(cherry picked from commit 8a1f1b045a2bffc0c3bf3941709acf4ba99d32e7)
Reviewed-on: https://chromium-review.googlesource.com/226700
| -rw-r--r-- | chip/lm4/lpc.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/chip/lm4/lpc.c b/chip/lm4/lpc.c index 99f212af19..536b31fca3 100644 --- a/chip/lm4/lpc.c +++ b/chip/lm4/lpc.c @@ -92,11 +92,16 @@ static void keyboard_irq_assert(void) * following falling edge, regardless of the line state before this * function call. */ + uint64_t tstop = get_time().val + MSEC; gpio_set_level(CONFIG_KEYBOARD_IRQ_GPIO, 1); udelay(4); /* Generate a falling edge */ gpio_set_level(CONFIG_KEYBOARD_IRQ_GPIO, 0); - udelay(4); + /* Wait for host senses the interrupt and gets the char. */ + do { + if (get_time().val > tstop) + break; + } while (lpc_keyboard_has_char()); /* Set signal high, now that we've generated the edge */ gpio_set_level(CONFIG_KEYBOARD_IRQ_GPIO, 1); } |