From ce315216119d5df972ab43eb23438682a49e5f3a Mon Sep 17 00:00:00 2001 From: Matt DeVillier Date: Tue, 24 Jul 2018 14:09:51 -0500 Subject: [PATCH] google/caroline: Fix I2C2 ACPI device definition Remove duplicate FMCN package for I2C2, since already generated by the devicetree-linked generic i2c driver. This fixes an ACPI parsing error which resulted in the touchpad and touchscreen being non-functional. Test: build/boot Caroline with GalliumOS 3.0a2, verify touchpad and touchscreen functioning properly, no ACPI errors in dmesg. Change-Id: I68315daf087aef0fc51411605b054e6322d5d7f8 Signed-off-by: Matt DeVillier Reviewed-on: https://review.coreboot.org/27624 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth Reviewed-by: Paul Menzel --- .../glados/variants/caroline/include/variant/acpi/mainboard.asl | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/mainboard/google/glados/variants/caroline/include/variant/acpi/mainboard.asl b/src/mainboard/google/glados/variants/caroline/include/variant/acpi/mainboard.asl index 19c883483b..d0324cf187 100644 --- a/src/mainboard/google/glados/variants/caroline/include/variant/acpi/mainboard.asl +++ b/src/mainboard/google/glados/variants/caroline/include/variant/acpi/mainboard.asl @@ -66,8 +66,6 @@ Scope (\_SB) Scope (\_SB.PCI0.I2C2) { - Name (FMCN, Package () { 87, 197, 26 }) - Device (DIGI) { Name (_HID, "ACPI0C50")