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-28 22:30:19 +0000 |
commit | 8a1f1b045a2bffc0c3bf3941709acf4ba99d32e7 (patch) | |
tree | 849a633444c8461a1a1f3a27586f9a0f5361fa61 | |
parent | f899a3445d037658fd9a3fe5191cb6f17a553e0c (diff) | |
download | chrome-ec-8a1f1b045a2bffc0c3bf3941709acf4ba99d32e7.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>
-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 8f8c5434a4..6bab4fb761 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); } |