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/scarlet/board.c | |
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/scarlet/board.c')
-rw-r--r-- | board/scarlet/board.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/board/scarlet/board.c b/board/scarlet/board.c index 7ae9598490..154c72f9bb 100644 --- a/board/scarlet/board.c +++ b/board/scarlet/board.c @@ -85,10 +85,10 @@ const unsigned int i2c_ports_used = ARRAY_SIZE(i2c_ports); /* power signal list. Must match order of enum power_signal. */ const struct power_signal_info power_signal_list[] = { - {GPIO_PP1250_S3_PG, 1, "PP1250_S3_PWR_GOOD"}, - {GPIO_PP900_S0_PG, 1, "PP900_S0_PWR_GOOD"}, - {GPIO_AP_CORE_PG, 1, "AP_PWR_GOOD"}, - {GPIO_AP_EC_S3_S0_L, 0, "SUSPEND_DEASSERTED"}, + {GPIO_PP1250_S3_PG, POWER_SIGNAL_ACTIVE_HIGH, "PP1250_S3_PWR_GOOD"}, + {GPIO_PP900_S0_PG, POWER_SIGNAL_ACTIVE_HIGH, "PP900_S0_PWR_GOOD"}, + {GPIO_AP_CORE_PG, POWER_SIGNAL_ACTIVE_HIGH, "AP_PWR_GOOD"}, + {GPIO_AP_EC_S3_S0_L, POWER_SIGNAL_ACTIVE_LOW, "SUSPEND_DEASSERTED"}, }; BUILD_ASSERT(ARRAY_SIZE(power_signal_list) == POWER_SIGNAL_COUNT); |