diff options
author | Namyoon Woo <namyoon@chromium.org> | 2018-09-07 16:39:26 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-09-13 18:58:53 -0700 |
commit | 04c593b8175b7b73204b6ce800573d769d83cd7a (patch) | |
tree | 51137b1669038c4e4e2f4ee7147f382e420d827b /board/eve | |
parent | 10b223dd62ca5178290a201a353e21bcd040b24d (diff) | |
download | chrome-ec-04c593b8175b7b73204b6ce800573d769d83cd7a.tar.gz |
keyboard: display keycap label in key status debug messages
The current debug message in keyboard_8042.c displays a tuple of
row, column, and press status. Additionally, this CL displays a
keycap label for better readability.
For keycap label mapping table can be adjustable under
"CONFIG_KEYBOARD_SCANCODE_MUTABLE" condition as scancode_set2[] is.
For coral board, Enabling CONFIG_KEYBOARD_DEBUG (w/o this CL)
occupies 652 bytes in flash, and this CL occupies another 312 bytes.
BUG=b:111060830
TEST=manually tested by pressing keyboards and check the EC console
screen.
BRANCH=kblog
Signed-off-by: Namyoon Woo <namyoon@chromium.org>
Changes to be committed:
modified: board/eve/board.c
modified: common/keyboard_8042.c
modified: common/keyboard_8042_sharedlib.c
modified: include/keyboard_8042_sharedlib.h
Change-Id: Idd71a5475b1ee313f99e087be9143dcfb6f81550
Reviewed-on: https://chromium-review.googlesource.com/1214543
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Namyoon Woo <namyoon@chromium.org>
Reviewed-by: Hung-Te Lin <hungte@chromium.org>
Diffstat (limited to 'board/eve')
-rw-r--r-- | board/eve/board.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/board/eve/board.c b/board/eve/board.c index 9943d4688b..e4ff6ad44a 100644 --- a/board/eve/board.c +++ b/board/eve/board.c @@ -490,6 +490,9 @@ static void board_init(void) /* Set F13 to new defined key on EVT */ CPRINTS("Overriding F13 scan code"); scancode_set2[3][9] = 0xe007; +#ifdef CONFIG_KEYBOARD_DEBUG + keycap_label[3][9] = KLLI_F13; +#endif } #endif } |