diff options
| author | Jun Lin <CHLin56@nuvoton.com> | 2021-12-07 14:08:52 +0800 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-12-21 17:52:20 +0000 |
| commit | 60eb772497faa4537c297d554580bdc0ff3a8ad1 (patch) | |
| tree | 743684a061dfafa58bef8cebbfa2ba8bee5fc86e | |
| parent | a10d6f2d3c3824a5c95c8b79f2a038d976850dc8 (diff) | |
| download | chrome-ec-60eb772497faa4537c297d554580bdc0ff3a8ad1.tar.gz | |
Zephyr: chip/npcx: power_policy: remove npcx_power_console_is_in_use
The upstream UART driver uses pm_constraint counter as a reference to
record if the system can enter deep sleep (PM_SUSPEND) state or not,
no matter for Tx or Rx. Remove npcx_power_console_is_in_use in this CL
as it is useless now.
BUG=none
BRANCH=none
TEST=pass "zmake testall"
TEST=build npcx9_evb and volteer with PR:, make sure they can
boot up; The console output messages don't break.
The system can enter deep sleep after the
CONFIG_UART_CONSOLE_INPUT_EXPIRED_TIMEOUT expires.
Signed-off-by: Jun Lin <CHLin56@nuvoton.com>
Change-Id: Ie152ce593c0af2fff683df0d3bdc44879aa0f5c3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3319988
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Tested-by: CH Lin <chlin56@nuvoton.com>
Reviewed-by: Keith Short <keithshort@chromium.org>
Commit-Queue: Keith Short <keithshort@chromium.org>
| -rw-r--r-- | zephyr/shim/chip/npcx/power_policy.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/zephyr/shim/chip/npcx/power_policy.c b/zephyr/shim/chip/npcx/power_policy.c index 5fca3791b3..5c6b4dc309 100644 --- a/zephyr/shim/chip/npcx/power_policy.c +++ b/zephyr/shim/chip/npcx/power_policy.c @@ -20,8 +20,7 @@ struct pm_state_info pm_policy_next_state(uint8_t cpu, int32_t ticks) { ARG_UNUSED(cpu); - /* Deep sleep is allowed and console is not in use. */ - if (DEEP_SLEEP_ALLOWED != 0 && !npcx_power_console_is_in_use()) { + if (DEEP_SLEEP_ALLOWED) { for (int i = ARRAY_SIZE(residency_info) - 1; i >= 0; i--) { if (!pm_constraint_get(residency_info[i].state)) { continue; |