From 5d66f23d216aad16b4b908cd10b206cc2f230eb0 Mon Sep 17 00:00:00 2001 From: Vic Yang Date: Mon, 27 May 2013 07:26:02 +0800 Subject: [PATCH] spring: Keep system power on sysjump If the AP is already on, the kernel should handle low-power event. We shouldn't power off the system on sysjump. BUG=chrome-os-partner:18318 TEST=None BRANCH=spring Change-Id: I4e80c61a25d2fa503d0c97e22dc2f4ad9c44f716 Signed-off-by: Vic Yang Reviewed-on: https://gerrit.chromium.org/gerrit/56706 Reviewed-by: Vincent Palatin --- common/gaia_power.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/common/gaia_power.c b/common/gaia_power.c index 856210fff2..b0589a84f9 100644 --- a/common/gaia_power.c +++ b/common/gaia_power.c @@ -546,10 +546,13 @@ static int wait_for_power_on(void) } #ifdef HAS_TASK_CHARGER - if (charge_keep_power_off()) { + /* + * If the system is already on (value == 1), the kernel + * would handle low power condition and we should not + * shutdown the system from EC. + */ + if (value != 1 && charge_keep_power_off()) { CPRINTF("[%T power on ignored due to low battery]\n"); - if (value == 1) /* System already on */ - power_off(); continue; } #endif