diff options
Diffstat (limited to 'board/spring')
-rw-r--r-- | board/spring/board.c | 51 | ||||
-rw-r--r-- | board/spring/board.h | 3 | ||||
-rw-r--r-- | board/spring/usb_charging.c | 4 |
3 files changed, 37 insertions, 21 deletions
diff --git a/board/spring/board.c b/board/spring/board.c index 227fc10f10..1dc0707203 100644 --- a/board/spring/board.c +++ b/board/spring/board.c @@ -330,12 +330,12 @@ int board_led_breathing(int enabled) ret |= lp5562_engine_load(LP5562_ENG_SEL_1, breathing_prog, sizeof(breathing_prog)); - ret |= lp5562_set_engine(LP5562_ENG_SEL_NONE, - LP5562_ENG_SEL_NONE, - LP5562_ENG_SEL_1); ret |= lp5562_engine_control(LP5562_ENG_RUN, LP5562_ENG_HOLD, LP5562_ENG_HOLD); + ret |= lp5562_set_engine(LP5562_ENG_SEL_NONE, + LP5562_ENG_SEL_NONE, + LP5562_ENG_SEL_1); } else { ret |= lp5562_engine_control(LP5562_ENG_HOLD, LP5562_ENG_HOLD, @@ -348,26 +348,49 @@ int board_led_breathing(int enabled) return ret; } -int board_battery_led(enum charging_state state) +static void board_battery_led_update(void) { int current; int desired_current; + + /* Current states and next states */ static uint32_t color = LED_COLOR_RED; static int breathing; + static int led_power; int new_color = LED_COLOR_RED; int new_breathing = 0; - int ret = 0; + int new_led_power; + + /* Determine LED power */ + new_led_power = board_get_ac(); + if (new_led_power != led_power) { + led_power = new_led_power; + if (new_led_power) { + lp5562_poweron(); + } else { + color = LED_COLOR_NONE; + if (breathing) { + board_led_breathing(0); + breathing = 0; + } + lp5562_poweroff(); + } + } + if (!new_led_power) + return; /* * LED power is controlled by accessory detection. We only * set color here. */ - switch (state) { + switch (charge_get_state()) { case ST_IDLE: new_color = LED_COLOR_GREEN; break; case ST_DISCHARGING: - new_color = LED_COLOR_NONE; + /* Discharging with AC, must be battery assist */ + new_color = LED_COLOR_YELLOW; + new_breathing = 1; break; case ST_PRE_CHARGING: new_color = LED_COLOR_YELLOW; @@ -382,7 +405,7 @@ int board_battery_led(enum charging_state state) if (current < 0 && desired_current > 0) { /* Battery assist */ new_breathing = 1; - new_color = LED_COLOR_NONE; + new_color = LED_COLOR_YELLOW; break; } @@ -396,16 +419,16 @@ int board_battery_led(enum charging_state state) break; } - if (new_breathing != breathing) { - ret |= board_led_breathing(new_breathing); - breathing = new_breathing; - } if (new_color != color) { - ret |= lp5562_set_color(new_color); + lp5562_set_color(new_color); color = new_color; } - return ret; + if (new_breathing != breathing) { + board_led_breathing(new_breathing); + breathing = new_breathing; + } } +DECLARE_HOOK(HOOK_SECOND, board_battery_led_update, HOOK_PRIO_DEFAULT); /*****************************************************************************/ /* Host commands */ diff --git a/board/spring/board.h b/board/spring/board.h index 7a6d4c8f05..2b0f62e54a 100644 --- a/board/spring/board.h +++ b/board/spring/board.h @@ -164,9 +164,6 @@ void board_pwm_duty_cycle(int percent); /* Update USB port status */ void board_usb_charge_update(int force_update); -/* Update battery LED color */ -int board_battery_led(enum charging_state state); - /* Get USB port device type */ int board_get_usb_dev_type(void); diff --git a/board/spring/usb_charging.c b/board/spring/usb_charging.c index 5c29af3e69..ba60e4dbaf 100644 --- a/board/spring/usb_charging.c +++ b/board/spring/usb_charging.c @@ -386,12 +386,8 @@ static void usb_device_change(int dev_type) current_limit = I_LIMIT_1500MA; board_pwm_nominal_duty_cycle(current_limit); - - /* Turns on battery LED */ - lp5562_poweron(); } else { board_ilim_config(ILIM_CONFIG_MANUAL_ON); - lp5562_poweroff(); } /* Log to console */ |