diff options
author | Duncan Laurie <dlaurie@google.com> | 2017-02-24 18:38:54 -0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2017-03-01 16:36:24 -0800 |
commit | 6e157818c18e090d43200b6f6dcfbd1082b60df6 (patch) | |
tree | f23ef45672eecde5ec8c166aad4908d07ae74510 /chip/lm4/gpio.c | |
parent | 850a227aea334bbb82119cf7811d4a53b72de23e (diff) | |
download | chrome-ec-6e157818c18e090d43200b6f6dcfbd1082b60df6.tar.gz |
gpio: Add function to clear pending interrupt
Currently if an interrupt is pending before it is enabled the interrupt
will fire immediately. In most cases this is fine, but if we want to
use the interrupt to trigger something like waking the AP it should be
sure that it won't immediately fire once enabled.
For example: on the Eve board we have the trackpad interrupt run to the
AP and the EC in order to support wake from Deep S3 (magic AP state that
only the EC can wake it from). This interrupt is used in S0 by the AP
while ignored by the EC, and then enabled on the transition to S3 in
order to be able to wake. Since it has been active the interrupt may
be pending in the EC (depending on the chip), which can result in the
interrupt firing immediately and waking the AP.
BUG=chrome-os-partner:62224
BRANCH=none
TEST=This has been functionally tested on npcx only as that is what I
have a use case and system for, the others compile and look right but
have not been directly tested.
Change-Id: I9e0877d99e7f09f4c30bf9861fbad81c12c059ad
Signed-off-by: Duncan Laurie <dlaurie@google.com>
Reviewed-on: https://chromium-review.googlesource.com/446962
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Diffstat (limited to 'chip/lm4/gpio.c')
-rw-r--r-- | chip/lm4/gpio.c | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/chip/lm4/gpio.c b/chip/lm4/gpio.c index 1b1e0a80c6..e90eb2bf4f 100644 --- a/chip/lm4/gpio.c +++ b/chip/lm4/gpio.c @@ -170,6 +170,18 @@ int gpio_disable_interrupt(enum gpio_signal signal) return EC_SUCCESS; } +int gpio_clear_pending_interrupt(enum gpio_signal signal) +{ + const struct gpio_info *g = gpio_list + signal; + + /* Fail if no interrupt handler */ + if (signal >= GPIO_IH_COUNT) + return EC_ERROR_INVAL; + + LM4_GPIO_ICR(g->port) |= g->mask; + return EC_SUCCESS; +} + #ifdef CONFIG_LOW_POWER_IDLE /** * Convert GPIO port to a mask that can be used to set the |