[coreboot-gerrit] Patch set updated for coreboot: mainboard/google/reef: Set edge triggered interrupt for GPIO_22

Vaibhav Shankar (vaibhav.shankar@intel.com) gerrit at coreboot.org
Mon Jan 30 23:31:29 CET 2017


Vaibhav Shankar (vaibhav.shankar at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18267

-gerrit

commit 64361d3ebae257618b0c956c60dfbb9a42b258e5
Author: Vaibhav Shankar <vaibhav.shankar at intel.com>
Date:   Mon Jan 30 12:18:42 2017 -0800

    mainboard/google/reef: Set edge triggered interrupt for GPIO_22
    
    EC sets the logic level based on outstanding wake events. When GPIO_22
    is configured as a level triggered interrupt, the events are not
    cleared from the interrupt handler. Hence, we'd just be re-signalling
    over and over causing an interrupt storm upon lid open. So, GPIO_22
    needs to be configured as EDGE_SINGLE instead of LEVEL.
    
    BUG=chrome-os-partner:62458
    TEST=Lid close/open. check CPU usage using top. It should
    not show 70% CPU usage.
    
    Signed-off-by: Vaibhav Shankar <vaibhav.shankar at intel.com>
    
    Change-Id: I710a690578c6e5b63be34b7fbcb21c703ef56e3a
---
 src/mainboard/google/reef/variants/baseboard/gpio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mainboard/google/reef/variants/baseboard/gpio.c b/src/mainboard/google/reef/variants/baseboard/gpio.c
index 3bb9a5f..a9a6248 100644
--- a/src/mainboard/google/reef/variants/baseboard/gpio.c
+++ b/src/mainboard/google/reef/variants/baseboard/gpio.c
@@ -299,7 +299,7 @@ static const struct pad_config gpio_table[] = {
 	PAD_CFG_GPI(GPIO_19, UP_20K, DEEP),	 /* unused */
 	PAD_CFG_GPI_APIC_LOW(GPIO_20, UP_20K, DEEP), /* NFC IRQ */
 	PAD_CFG_GPI_APIC_LOW(GPIO_21, NONE, DEEP), /* Touch IRQ */
-	PAD_CFG_GPI_SCI_LOW(GPIO_22, NONE, DEEP, LEVEL), /* EC wake */
+	PAD_CFG_GPI_SCI_LOW(GPIO_22, NONE, DEEP, EDGE_SINGLE), /* EC wake */
 	PAD_CFG_GPI(GPIO_23, UP_20K, DEEP),	 /* unused */
 	PAD_CFG_GPI(GPIO_24, NONE, DEEP),	 /* PEN_PDCT_ODL */
 	PAD_CFG_GPI(GPIO_25, UP_20K, DEEP),	 /* unused */



More information about the coreboot-gerrit mailing list