From 423eac3ac1dc2e01aac290f7799205a114107171 Mon Sep 17 00:00:00 2001 From: Mary Ruthven Date: Wed, 18 Jan 2017 14:51:05 -0800 Subject: cr50: decrease sleep delay to 20 seconds after init and resume 3 minutes is too long to delay sleep after init and resume from wake pin. This change decreases the delay to 20 seconds BUG=none BRANCH=none TEST=manual Use the cr50 power consumption to verify the sleep state active 50mW sleep 7mW deep sleep 1mW make sure suzyq is disconnected use uart to reboot cr50 run 'reboot ap-off' on the EC console make sure cr50 enters deep sleep at second 20 use uart to wakeup cr50 make sure it stays awake for 20 seconds and then enters deep sleep. wake it up again using uart and run 'idle s' verify it enters regular sleep after 20 seconds use uart to wake it up, make sure it does a regular sleep resume and then goes back into regular sleep after 20 seconds Change-Id: I65791bd3d915ceda11dc29b74e150ba589f2fa9e Signed-off-by: Mary Ruthven Reviewed-on: https://chromium-review.googlesource.com/430388 Reviewed-by: Vadim Bendebury --- board/cr50/board.c | 2 +- chip/g/idle.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/board/cr50/board.c b/board/cr50/board.c index fe23db81de..4c56281dbb 100644 --- a/board/cr50/board.c +++ b/board/cr50/board.c @@ -290,7 +290,7 @@ void pmu_wakeup_interrupt(void) * Delay sleep long enough for a SPI slave transaction to start * or for the system to be reset. */ - delay_sleep_by(3 * MINUTE); + delay_sleep_by(20 * SECOND); /* * If sys_rst_l or plt_rst_l (if signal is present) is diff --git a/chip/g/idle.c b/chip/g/idle.c index b376f14e82..0d4656a8c6 100644 --- a/chip/g/idle.c +++ b/chip/g/idle.c @@ -211,8 +211,8 @@ void __idle(void) if (!idle_action) idle_action = idle_default; - /* Disable sleep until 3 minutes after init */ - delay_sleep_by(3 * MINUTE); + /* Disable sleep for 20 seconds after init */ + delay_sleep_by(20 * SECOND); while (1) { -- cgit v1.2.1