diff options
| author | Ben Lok <ben.lok@mediatek.com> | 2015-09-17 13:31:51 +0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2015-10-06 22:57:15 -0700 |
| commit | 0596b551dfd45dd4af2e8f7a0baad1141d25bfb8 (patch) | |
| tree | 3373a093cba91d4cbd3c6d87f75a4d0d736dc8f3 /common/mkbp_event.c | |
| parent | 1c2bbee5c7a779cb22519e8710213a641b43eeff (diff) | |
| download | chrome-ec-0596b551dfd45dd4af2e8f7a0baad1141d25bfb8.tar.gz | |
mkbp_event: prevent AC power change events to wakeup AP.
It may use MKBP event to send PD power change events to AP via interrupt.
According to the spec, AC power change events do not be allowed to
wake up AP. In order to avoid it, define a white list in board level,
only allow those events to wakeup the AP during S3 power state.
BRANCH=none
BUG=chrome-os-partner:45127
TEST=manual
Plug PD power adapter to oak, if system is in S3/S5 and it should
starts charging, but should not wake up system/AP at all.
Change-Id: I2f86697d5d3bd24d7de840e21064b91e8841f0eb
Signed-off-by: Ben Lok <ben.lok@mediatek.com>
Reviewed-on: https://chromium-review.googlesource.com/300360
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
Diffstat (limited to 'common/mkbp_event.c')
| -rw-r--r-- | common/mkbp_event.c | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/common/mkbp_event.c b/common/mkbp_event.c index c8b8d3bced..4e965a9d05 100644 --- a/common/mkbp_event.c +++ b/common/mkbp_event.c @@ -6,6 +6,7 @@ */ #include "atomic.h" +#include "chipset.h" #include "gpio.h" #include "host_command.h" #include "link_defs.h" @@ -41,6 +42,23 @@ static void set_host_interrupt(int active) void mkbp_send_event(uint8_t event_type) { set_event(event_type); + +#ifdef CONFIG_MKBP_WAKEUP_MASK + /* checking the event if AP suspended */ + if (chipset_in_state(CHIPSET_STATE_SUSPEND)) { + uint32_t events; + events = *(uint32_t *)host_get_memmap(EC_MEMMAP_HOST_EVENTS); + /* + * interrupt the AP if it is a wakeup event + * which is defined in the white list. + */ + if (events & CONFIG_MKBP_WAKEUP_MASK) + set_host_interrupt(1); + + return; + } +#endif + set_host_interrupt(1); } |