Hi Simon,
On Wed, Sep 9, 2015 at 12:32 PM, Simon Glass sjg@chromium.org wrote:
Some boards have an i8042 device. Enable the driver for all x86 boards, and add a device tree node for those which may have this keyboard.
Also adjust the configuration so that i8042 is always separate from the VGA, and rename the stdin driver accordingly. With this commit the keyboard will not work, but it is fixed in the next commit.
Signed-off-by: Simon Glass sjg@chromium.org
arch/x86/Kconfig | 6 ++++++ arch/x86/dts/bayleybay.dts | 1 + arch/x86/dts/chromebook_link.dts | 1 + arch/x86/dts/keyboard.dtsi | 5 +++++ include/configs/x86-chromebook.h | 2 +- include/configs/x86-common.h | 2 +- 6 files changed, 15 insertions(+), 2 deletions(-) create mode 100644 arch/x86/dts/keyboard.dtsi
diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index 5e42d7d..72a66ea 100644 --- a/arch/x86/Kconfig +++ b/arch/x86/Kconfig @@ -401,6 +401,12 @@ config PCIE_ECAM_SIZE so a default 0x10000000 size covers all of the 256 buses which is the maximum number of PCI buses as defined by the PCI specification.
+config I8042_KEYB
default y+config DM_KEYBOARD
default y
I am not in favor of adding driver options in the arch Kconfig. We specify other drivers in the boards' defconfig files. Can we move these two to defconfig too?
source "arch/x86/lib/efi/Kconfig"
endmenu diff --git a/arch/x86/dts/bayleybay.dts b/arch/x86/dts/bayleybay.dts index d646987..58d97c8 100644 --- a/arch/x86/dts/bayleybay.dts +++ b/arch/x86/dts/bayleybay.dts @@ -10,6 +10,7 @@ #include <dt-bindings/interrupt-router/intel-irq.h>
/include/ "skeleton.dtsi" +/include/ "keyboard.dtsi" /include/ "serial.dtsi" /include/ "rtc.dtsi"
diff --git a/arch/x86/dts/chromebook_link.dts b/arch/x86/dts/chromebook_link.dts index 4291141..a52c84f 100644 --- a/arch/x86/dts/chromebook_link.dts +++ b/arch/x86/dts/chromebook_link.dts @@ -1,6 +1,7 @@ /dts-v1/;
/include/ "skeleton.dtsi" +/include/ "keyboard.dtsi" /include/ "serial.dtsi" /include/ "rtc.dtsi"
diff --git a/arch/x86/dts/keyboard.dtsi b/arch/x86/dts/keyboard.dtsi new file mode 100644 index 0000000..000751b --- /dev/null +++ b/arch/x86/dts/keyboard.dtsi @@ -0,0 +1,5 @@ +/ {
keyboard {compatible = "intel,i8042-keyboard";};+}; diff --git a/include/configs/x86-chromebook.h b/include/configs/x86-chromebook.h index 2be8850..4ff8b94 100644 --- a/include/configs/x86-chromebook.h +++ b/include/configs/x86-chromebook.h @@ -51,7 +51,7 @@ #define CONFIG_ENV_IS_IN_SPI_FLASH #define CONFIG_ENV_OFFSET 0x003f8000
-#define CONFIG_STD_DEVICES_SETTINGS "stdin=usbkbd,vga,serial\0" \ +#define CONFIG_STD_DEVICES_SETTINGS "stdin=usbkbd,i8042-kbd,serial\0" \ "stdout=vga,serial\0" \ "stderr=vga,serial\0"
diff --git a/include/configs/x86-common.h b/include/configs/x86-common.h index 3f153f2..ff8fe0f 100644 --- a/include/configs/x86-common.h +++ b/include/configs/x86-common.h @@ -147,7 +147,7 @@ #define CONFIG_VIDEO #define CONFIG_VIDEO_SW_CURSOR #define VIDEO_FB_16BPP_WORD_SWAP -#define CONFIG_I8042_KBD +#define CONFIG_VGA_AS_SINGLE_DEVICE
Can we remove CONFIG_VGA_AS_SINGLE_DEVICE from all x86 board config.h files to avoid duplication?
#define CONFIG_CFB_CONSOLE #define CONFIG_CONSOLE_SCROLL_LINES 5
--
Regards, Bin