diff options
| author | Aseda Aboagye <aaboagye@google.com> | 2019-07-02 10:38:56 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2019-07-02 23:12:48 +0000 |
| commit | 4509620f1e7873ee239b558675a47c69f614a732 (patch) | |
| tree | 62fd72e93bb2661389a09e1b25d6ff118da65dba /board | |
| parent | 9ba43859c8ed86768e95ea01b7fc5a495af683a8 (diff) | |
| download | chrome-ec-4509620f1e7873ee239b558675a47c69f614a732.tar.gz | |
cleanup: Rename CONFIG_MKBP_WAKEUP_MASK for clarity.
CONFIG_MKBP_WAKEUP_MASK is a bit confusing and is wrongly named. The
comment stated that "With this option, we can define the MKBP wakeup
events in this mask (as a white list) in board level, those evets allow
to interrupt AP during S3.". However, these events are NOT MKBP events
at all but are instead host events. This commit tries to clear things
up by renaming CONFIG_MKBP_WAKEUP_MASK to
CONFIG_MKBP_HOST_EVENT_WAKEUP_MASK to better show that these events are
in fact host events.
BUG=b:136282898
BRANCH=None
TEST=`make -j buildall`
Change-Id: I42beadec8217435fd30e679ccf52d784a8ef99a0
Signed-off-by: Aseda Aboagye <aaboagye@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1685784
Tested-by: Aseda Aboagye <aaboagye@chromium.org>
Reviewed-by: Alexandru M Stan <amstan@chromium.org>
Commit-Queue: Aseda Aboagye <aaboagye@chromium.org>
Diffstat (limited to 'board')
| -rw-r--r-- | board/elm/board.h | 4 | ||||
| -rw-r--r-- | board/flapjack/board.h | 4 | ||||
| -rw-r--r-- | board/kukui/board.h | 4 | ||||
| -rw-r--r-- | board/oak/board.h | 4 | ||||
| -rw-r--r-- | board/rainier/board.h | 4 | ||||
| -rw-r--r-- | board/scarlet/board.h | 4 |
6 files changed, 12 insertions, 12 deletions
diff --git a/board/elm/board.h b/board/elm/board.h index 0de9730454..ffd54aacc6 100644 --- a/board/elm/board.h +++ b/board/elm/board.h @@ -172,8 +172,8 @@ #define TIM_CLOCK32 2 #define TIM_WATCHDOG 4 -/* Define the MKBP events which are allowed to wakeup AP in S3. */ -#define CONFIG_MKBP_WAKEUP_MASK \ +/* Define the host events which are allowed to wakeup AP in S3. */ +#define CONFIG_MKBP_HOST_EVENT_WAKEUP_MASK \ (EC_HOST_EVENT_MASK(EC_HOST_EVENT_LID_OPEN) |\ EC_HOST_EVENT_MASK(EC_HOST_EVENT_POWER_BUTTON) |\ EC_HOST_EVENT_MASK(EC_HOST_EVENT_KEY_PRESSED) |\ diff --git a/board/flapjack/board.h b/board/flapjack/board.h index 57e398694c..693a1bb8a9 100644 --- a/board/flapjack/board.h +++ b/board/flapjack/board.h @@ -236,8 +236,8 @@ #define CONFIG_KEYBOARD_PROTOCOL_MKBP #define CONFIG_MKBP_EVENT #define CONFIG_MKBP_USE_GPIO -/* Define the MKBP events which are allowed to wakeup AP in S3. */ -#define CONFIG_MKBP_WAKEUP_MASK \ +/* Define the host events which are allowed to wakeup AP in S3. */ +#define CONFIG_MKBP_HOST_EVENT_WAKEUP_MASK \ (EC_HOST_EVENT_MASK(EC_HOST_EVENT_POWER_BUTTON) |\ EC_HOST_EVENT_MASK(EC_HOST_EVENT_RTC)) diff --git a/board/kukui/board.h b/board/kukui/board.h index 39e1260324..5680e1b2ec 100644 --- a/board/kukui/board.h +++ b/board/kukui/board.h @@ -228,8 +228,8 @@ #define CONFIG_KEYBOARD_PROTOCOL_MKBP #define CONFIG_MKBP_EVENT #define CONFIG_MKBP_USE_GPIO -/* Define the MKBP events which are allowed to wakeup AP in S3. */ -#define CONFIG_MKBP_WAKEUP_MASK \ +/* Define the host events which are allowed to wakeup AP in S3. */ +#define CONFIG_MKBP_HOST_EVENT_WAKEUP_MASK \ (EC_HOST_EVENT_MASK(EC_HOST_EVENT_LID_OPEN) |\ EC_HOST_EVENT_MASK(EC_HOST_EVENT_POWER_BUTTON)) diff --git a/board/oak/board.h b/board/oak/board.h index e4ae567b29..b8ebe7c8e6 100644 --- a/board/oak/board.h +++ b/board/oak/board.h @@ -166,8 +166,8 @@ #define TIM_CLOCK32 2 #define TIM_WATCHDOG 4 -/* Define the MKBP events which are allowed to wakeup AP in S3. */ -#define CONFIG_MKBP_WAKEUP_MASK \ +/* Define the host events which are allowed to wakeup AP in S3. */ +#define CONFIG_MKBP_HOST_EVENT_WAKEUP_MASK \ (EC_HOST_EVENT_MASK(EC_HOST_EVENT_LID_OPEN) |\ EC_HOST_EVENT_MASK(EC_HOST_EVENT_POWER_BUTTON) |\ EC_HOST_EVENT_MASK(EC_HOST_EVENT_KEY_PRESSED) |\ diff --git a/board/rainier/board.h b/board/rainier/board.h index 4903de557d..3827f97ea6 100644 --- a/board/rainier/board.h +++ b/board/rainier/board.h @@ -131,8 +131,8 @@ #define CONFIG_KEYBOARD_PROTOCOL_MKBP #define CONFIG_MKBP_EVENT #define CONFIG_MKBP_USE_GPIO -/* Define the MKBP events which are allowed to wakeup AP in S3. */ -#define CONFIG_MKBP_WAKEUP_MASK \ +/* Define the host events which are allowed to wakeup AP in S3. */ +#define CONFIG_MKBP_HOST_EVENT_WAKEUP_MASK \ (EC_HOST_EVENT_MASK(EC_HOST_EVENT_POWER_BUTTON) |\ EC_HOST_EVENT_MASK(EC_HOST_EVENT_RTC)) diff --git a/board/scarlet/board.h b/board/scarlet/board.h index c13b45029c..5cd82e573c 100644 --- a/board/scarlet/board.h +++ b/board/scarlet/board.h @@ -182,8 +182,8 @@ #define CONFIG_KEYBOARD_PROTOCOL_MKBP #define CONFIG_MKBP_EVENT #define CONFIG_MKBP_USE_GPIO -/* Define the MKBP events which are allowed to wakeup AP in S3. */ -#define CONFIG_MKBP_WAKEUP_MASK \ +/* Define the host events which are allowed to wakeup AP in S3. */ +#define CONFIG_MKBP_HOST_EVENT_WAKEUP_MASK \ (EC_HOST_EVENT_MASK(EC_HOST_EVENT_POWER_BUTTON) |\ EC_HOST_EVENT_MASK(EC_HOST_EVENT_RTC)) |