[coreboot-gerrit] Patch set updated for coreboot: 7d8762c libpayload: Remove PC Keyboard from ARM build

Marc Jones (marc.jones@se-eng.com) gerrit at coreboot.org
Tue Dec 30 06:08:26 CET 2014


Marc Jones (marc.jones at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7922

-gerrit

commit 7d8762cbc7a5e1fdac9162688907f666af3b4bbd
Author: Marc Jones <marc.jones at se-eng.com>
Date:   Tue Dec 23 15:22:30 2014 -0700

    libpayload: Remove PC Keyboard from ARM build
    
    The keyboard.c uses IO cycles to access the legacy PC keyboard device.
    ARM can't do IO cycles, so remove the option for ARM configs.
    
    Change-Id: Ifc6c2368563f27867f4babad5afdde0e78f4cf78
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
---
 payloads/libpayload/Config.in             | 3 ++-
 payloads/libpayload/configs/defconfig-arm | 8 ++------
 2 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/payloads/libpayload/Config.in b/payloads/libpayload/Config.in
index f317c18..bef0e05 100644
--- a/payloads/libpayload/Config.in
+++ b/payloads/libpayload/Config.in
@@ -237,7 +237,8 @@ config COREBOOT_VIDEO_CONSOLE
 
 config PC_KEYBOARD
 	bool "Allow input from a PC keyboard"
-	default y
+	default y if ARCH_X86 # uses IO
+	default n
 
 config PC_KEYBOARD_LAYOUT_US
 	bool "English (US) keyboard layout"
diff --git a/payloads/libpayload/configs/defconfig-arm b/payloads/libpayload/configs/defconfig-arm
index 12ad423..fe47d0c 100644
--- a/payloads/libpayload/configs/defconfig-arm
+++ b/payloads/libpayload/configs/defconfig-arm
@@ -1,7 +1,7 @@
 #
 # Automatically generated make config: don't edit
 # libpayload version: 0.2.0
-# Wed Nov 12 14:57:03 2014
+# Tue Dec 23 15:19:30 2014
 #
 
 #
@@ -42,9 +42,7 @@ CONFIG_LP_SERIAL_CONSOLE=y
 # CONFIG_LP_SERIAL_ACS_FALLBACK is not set
 CONFIG_LP_VIDEO_CONSOLE=y
 # CONFIG_LP_COREBOOT_VIDEO_CONSOLE is not set
-CONFIG_LP_PC_KEYBOARD=y
-CONFIG_LP_PC_KEYBOARD_LAYOUT_US=y
-# CONFIG_LP_PC_KEYBOARD_LAYOUT_DE is not set
+# CONFIG_LP_PC_KEYBOARD is not set
 
 #
 # Drivers
@@ -70,5 +68,3 @@ CONFIG_LP_USB_GEN_HUB=y
 CONFIG_LP_LITTLE_ENDIAN=y
 # CONFIG_LP_IO_ADDRESS_SPACE is not set
 CONFIG_LP_ARCH_SPECIFIC_OPTIONS=y
-CONFIG_LP_COREBOOT_INFO_RANGE_BASE=0x10
-CONFIG_LP_COREBOOT_INFO_RANGE_SIZE=0x4000000



More information about the coreboot-gerrit mailing list