[coreboot-gerrit] New patch to review for coreboot: 8042 keyboard: power button processing

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Thu Jan 19 18:39:47 CET 2017


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18177

-gerrit

commit be53688e09e8cf2a087a8c1bf889e388026066f6
Author: Shelley Chen <shchen at chromium.org>
Date:   Fri Jan 13 09:40:39 2017 -0800

    8042 keyboard: power button processing
    
    power button is usually dropped because it's not
    in the keyboard matrix range.  Adding in condition
    to forward it like other keys.
    
    BUG=chrome-os-partner:61275
    BRANCH=None
    TEST=reboot and make sure power button selection
         in detachable menus is processed on reef.
    
    Change-Id: I86897fa8d73a56533ef62bba05458ac3d339237e
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 25654e214f0ab8685d445ced62612a02be851126
    Original-Change-Id: I516a0043bd7730789728d5c5498d0a0f30a2acac
    Original-Signed-off-by: Shelley Chen <shchen at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/428199
    Original-Reviewed-by: Randall Spangler <rspangler at chromium.org>
---
 payloads/libpayload/drivers/keyboard.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/payloads/libpayload/drivers/keyboard.c b/payloads/libpayload/drivers/keyboard.c
index 6431336..23c8780 100644
--- a/payloads/libpayload/drivers/keyboard.c
+++ b/payloads/libpayload/drivers/keyboard.c
@@ -32,6 +32,7 @@
 #include <libpayload.h>
 
 #define I8042_CMD_DIS_KB     0xad
+#define POWER_BUTTON         0x90
 
 struct layout_maps {
 	const char *country;
@@ -255,6 +256,9 @@ int keyboard_getchar(void)
 		}
 	}
 
+	if (ch == 0x5e)
+		ret = POWER_BUTTON;
+
 	return ret;
 }
 



More information about the coreboot-gerrit mailing list