[coreboot-gerrit] Patch set updated for coreboot: google/parrot: Fix keyboard interrupts, DSDT

Matt DeVillier (matt.devillier@gmail.com) gerrit at coreboot.org
Fri Nov 25 01:12:17 CET 2016


Matt DeVillier (matt.devillier at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17017

-gerrit

commit c01b090f662ff194c1c393cc55d8aac127c3725c
Author: Prabal Saha <coolstarorganization at gmail.com>
Date:   Thu Oct 13 07:52:23 2016 -0700

    google/parrot: Fix keyboard interrupts, DSDT
    
    Commit 967cd9a [ChromeOS: fix Kconfig dependencies] broke keyboard
    interrupts on parrot by making SERIRQ_CONTINUOUS_MODE conditional on
    CONFIG_CHROMEOS, which it should not be; fix by moving back under main
    board specific options config.
    
    Additionally, Windows [8/8.1/10] fails to enumerate the keyboard when
    its ACPI entry is located under the SIO device since it is missing an
    _HID entry, so add the appropriate value per ACPI spec 5 ch. 9.7
    
    Change-Id: Ia69e9b326001d2026b15b4ec03c94f7d03c8a700
    Signed-off-by: Prabal Saha <coolstarorganization at gmail.com>
    Signed-off-by: Matt DeVillier <matt.devillier at gmail.com>
---
 src/ec/compal/ene932/acpi/superio.asl | 3 ++-
 src/mainboard/google/parrot/Kconfig   | 5 ++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/ec/compal/ene932/acpi/superio.asl b/src/ec/compal/ene932/acpi/superio.asl
index 8eba623..498f2e0 100644
--- a/src/ec/compal/ene932/acpi/superio.asl
+++ b/src/ec/compal/ene932/acpi/superio.asl
@@ -16,6 +16,7 @@
 // Scope is \_SB.PCI0.LPCB
 
 Device (SIO) {
+	Name (_HID, EisaId("PNP0A05"))
 	Name (_UID, 0)
 	Name (_ADR, 0)
 
@@ -23,4 +24,4 @@ Device (SIO) {
 #ifdef SIO_EC_ENABLE_PS2K
 	#include <drivers/pc80/pc/ps2_controller.asl>
 #endif
-}
+}
\ No newline at end of file
diff --git a/src/mainboard/google/parrot/Kconfig b/src/mainboard/google/parrot/Kconfig
index 974c58f..7b6b49b 100644
--- a/src/mainboard/google/parrot/Kconfig
+++ b/src/mainboard/google/parrot/Kconfig
@@ -14,13 +14,12 @@ config BOARD_SPECIFIC_OPTIONS # dummy
 	select MAINBOARD_HAS_CHROMEOS
 	select MAINBOARD_HAS_LPC_TPM
 	select INTEL_INT15
+	# Workaround for EC/KBC IRQ1.
+	select SERIRQ_CONTINUOUS_MODE
 
 config CHROMEOS
 	select VBOOT_VBNV_CMOS
 
-	# Workaround for EC/KBC IRQ1.
-	select SERIRQ_CONTINUOUS_MODE
-
 config MAINBOARD_DIR
 	string
 	default google/parrot



More information about the coreboot-gerrit mailing list