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/samus | |
| 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/samus')
| -rw-r--r-- | board/samus/board.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/board/samus/board.c b/board/samus/board.c index 37ecbfeee3..01ad05e7a2 100644 --- a/board/samus/board.c +++ b/board/samus/board.c @@ -57,15 +57,15 @@ static void pd_mcu_interrupt(enum gpio_signal signal) /* power signal list. Must match order of enum power_signal. */ const struct power_signal_info power_signal_list[] = { - {GPIO_PP1050_PGOOD, 1, "PGOOD_PP1050"}, - {GPIO_PP1200_PGOOD, 1, "PGOOD_PP1200"}, - {GPIO_PP1800_PGOOD, 1, "PGOOD_PP1800"}, - {GPIO_VCORE_PGOOD, 1, "PGOOD_VCORE"}, - {GPIO_PCH_SLP_S0_L, 1, "SLP_S0_DEASSERTED"}, - {GPIO_PCH_SLP_S3_L, 1, "SLP_S3_DEASSERTED"}, - {GPIO_PCH_SLP_S5_L, 1, "SLP_S5_DEASSERTED"}, - {GPIO_PCH_SLP_SUS_L, 1, "SLP_SUS_DEASSERTED"}, - {GPIO_PCH_SUSWARN_L, 1, "SUSWARN_DEASSERTED"}, + {GPIO_PP1050_PGOOD, POWER_SIGNAL_ACTIVE_HIGH, "PGOOD_PP1050"}, + {GPIO_PP1200_PGOOD, POWER_SIGNAL_ACTIVE_HIGH, "PGOOD_PP1200"}, + {GPIO_PP1800_PGOOD, POWER_SIGNAL_ACTIVE_HIGH, "PGOOD_PP1800"}, + {GPIO_VCORE_PGOOD, POWER_SIGNAL_ACTIVE_HIGH, "PGOOD_VCORE"}, + {GPIO_PCH_SLP_S0_L, POWER_SIGNAL_ACTIVE_HIGH, "SLP_S0_DEASSERTED"}, + {GPIO_PCH_SLP_S3_L, POWER_SIGNAL_ACTIVE_HIGH, "SLP_S3_DEASSERTED"}, + {GPIO_PCH_SLP_S5_L, POWER_SIGNAL_ACTIVE_HIGH, "SLP_S5_DEASSERTED"}, + {GPIO_PCH_SLP_SUS_L, POWER_SIGNAL_ACTIVE_HIGH, "SLP_SUS_DEASSERTED"}, + {GPIO_PCH_SUSWARN_L, POWER_SIGNAL_ACTIVE_HIGH, "SUSWARN_DEASSERTED"}, }; BUILD_ASSERT(ARRAY_SIZE(power_signal_list) == POWER_SIGNAL_COUNT); |