[coreboot-gerrit] New patch to review for coreboot: 9d8e09b samus: Make codec interrupt active high

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Thu Apr 2 23:52:35 CEST 2015


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/9268

-gerrit

commit 9d8e09b1885b3ed7d5a4bfcc313fe29974d2fd34
Author: Ben Zhang <benzh at chromium.org>
Date:   Tue Sep 30 12:55:04 2014 -0700

    samus: Make codec interrupt active high
    
    The codec interrupt needs to be active high because multiple
    interrupt sources share this line:
    
    1) Headphone plug detect
    2) Mic present
    3) Hotword detect
    
    These interrupt sources are OR-ed together.
    
    BUG=chrome-os-partner:29649
    BRANCH=samus
    TEST=Jack detection works on samus
    
    Change-Id: If35fe8493ab30d878d9fac2251acee62c776b0eb
    Signed-off-by: Stefan Reinauer <reinauer at chromium.org>
    Original-Commit-Id: 348608fe61f7848db2bfd22502a0c259d24f8980
    Original-Change-Id: Ief0a291d9455f2d03789198153781ff8133aa1ce
    Original-Signed-off-by: Ben Zhang <benzh at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/220588
    Original-Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
---
 src/mainboard/google/samus/acpi/mainboard.asl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mainboard/google/samus/acpi/mainboard.asl b/src/mainboard/google/samus/acpi/mainboard.asl
index b4b4723..128f266 100644
--- a/src/mainboard/google/samus/acpi/mainboard.asl
+++ b/src/mainboard/google/samus/acpi/mainboard.asl
@@ -193,7 +193,7 @@ Scope (\_SB.PCI0.I2C0)
 			)
 
 			/* GPIO46 is PIRQO (use HOTWORD_DET as codec IRQ) */
-			Interrupt (ResourceConsumer, Edge, ActiveLow) { 30 }
+			Interrupt (ResourceConsumer, Edge, ActiveHigh) { 30 }
 
 			/*
 			 * Codec GPIOs are 1-based in the schematic



More information about the coreboot-gerrit mailing list