Hung-Te Lin has submitted this change. ( https://review.coreboot.org/c/coreboot/+/47087 )
Change subject: libpayload/keyboard: Add timestamp to track time spent in a state ......................................................................
libpayload/keyboard: Add timestamp to track time spent in a state
Will be used to time out in states that don't always advance.
Change-Id: I28235e7638d8157cedf81fd915a41d28a1fc070b Signed-off-by: Nico Huber nico.h@gmx.de Reviewed-on: https://review.coreboot.org/c/coreboot/+/47087 Reviewed-by: Angel Pons th3fanbus@gmail.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M payloads/libpayload/drivers/i8042/keyboard.c 1 file changed, 5 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved
diff --git a/payloads/libpayload/drivers/i8042/keyboard.c b/payloads/libpayload/drivers/i8042/keyboard.c index dcba031..114e18b 100644 --- a/payloads/libpayload/drivers/i8042/keyboard.c +++ b/payloads/libpayload/drivers/i8042/keyboard.c @@ -245,6 +245,7 @@ } keyboard_state;
static uint64_t keyboard_time; +static uint64_t state_time;
static void keyboard_poll(void) { @@ -334,8 +335,10 @@ break; }
- if (keyboard_state != next_state) + if (keyboard_state != next_state) { keyboard_state = next_state; + state_time = timer_us(0); + } }
bool keyboard_havechar(void) @@ -489,7 +492,7 @@ i8042_cmd(I8042_CMD_EN_KB);
keyboard_state = STATE_DISABLE_SCAN; - keyboard_time = timer_us(0); + keyboard_time = state_time = timer_us(0);
console_add_input_driver(&cons); }