diff options
author | Furquan Shaikh <furquan@chromium.org> | 2017-09-22 12:54:28 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2017-10-03 14:26:09 -0700 |
commit | a27f1049b6d92d46de9a648ba64f451387a598d4 (patch) | |
tree | d7396d9183f565c2fbbecf2b9d4da6bd2880c5e3 /board/elm | |
parent | 29dc221f98fa85efd3b351f0816c037562952753 (diff) | |
download | chrome-ec-a27f1049b6d92d46de9a648ba64f451387a598d4.tar.gz |
power: Add flags parameter to power_signal_info
Replace structure member "level" in power_signal_info with "flags".
"level" has been used on all boards to indicate active-high or
active-low levels. Addition of "flags" allows easy extension of
power_signal_info structure to define various flags that might be
applicable to power signals (e.g. "level"). Going forward, additional
flag will be added in follow-up CLs.
Also, provide a helper function power_signal_is_asserted that checks
the actual level of a signal and compares it to the flags level to
identify if a power signal is asserted.
BUG=b:65421825
BRANCH=None
TEST=make -j buildall
Change-Id: Iacaabd1185b347c17b5159f05520731505b824b8
Signed-off-by: Furquan Shaikh <furquan@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/679979
Reviewed-by: Nicolas Boichat <drinkcat@chromium.org>
Diffstat (limited to 'board/elm')
-rw-r--r-- | board/elm/board.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/board/elm/board.c b/board/elm/board.c index 78989c7348..775960d424 100644 --- a/board/elm/board.c +++ b/board/elm/board.c @@ -76,8 +76,8 @@ void usb_evt(enum gpio_signal signal) /* power signal list. Must match order of enum power_signal. */ const struct power_signal_info power_signal_list[] = { - {GPIO_SOC_POWER_GOOD, 1, "POWER_GOOD"}, /* Active high */ - {GPIO_SUSPEND_L, 0, "SUSPEND#_ASSERTED"}, /* Active low */ + {GPIO_SOC_POWER_GOOD, POWER_SIGNAL_ACTIVE_HIGH, "POWER_GOOD"}, + {GPIO_SUSPEND_L, POWER_SIGNAL_ACTIVE_LOW, "SUSPEND#_ASSERTED"}, }; BUILD_ASSERT(ARRAY_SIZE(power_signal_list) == POWER_SIGNAL_COUNT); |