Hello Felix Singer, Matt Delco, Patrick Georgi, Furquan Shaikh, Julius Werner, Angel Pons, Patrick Rudolph, EricR Lai,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/47087
to review the following change.
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 --- M payloads/libpayload/drivers/i8042/keyboard.c 1 file changed, 6 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/87/47087/1
diff --git a/payloads/libpayload/drivers/i8042/keyboard.c b/payloads/libpayload/drivers/i8042/keyboard.c index 27abd95..a30bdaf 100644 --- a/payloads/libpayload/drivers/i8042/keyboard.c +++ b/payloads/libpayload/drivers/i8042/keyboard.c @@ -249,6 +249,8 @@ STATE_RUNNING, } keyboard_state;
+uint64_t state_time; + static void keyboard_poll(void) { enum keyboard_state next_state = keyboard_state; @@ -289,8 +291,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) @@ -448,6 +452,7 @@ keyboard_drain_input();
keyboard_state = STATE_CONFIGURE; + state_time = timer_us(0);
console_add_input_driver(&cons); }