diff options
author | eddylu <eddylu@ami.corp-partner.google.com> | 2018-07-16 16:04:25 +0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-07-26 04:07:46 -0700 |
commit | 524de25a4d879d8a27f630383f25ab28481cf8f4 (patch) | |
tree | 3f8d8bec11a1509fd536d882575a8595e305dd2f /board/phaser | |
parent | a59038668c9bde21225563c8cd994dc961d15128 (diff) | |
download | chrome-ec-524de25a4d879d8a27f630383f25ab28481cf8f4.tar.gz |
octopus: Add phaser LED behavior support
Reference LED behavior spec and implement it.
Add Power LED common code.
Yorp and bip also do some necessary charges.
BUG=b:80501031,b:110086152
BRANCH=none
TEST=Verify LED behavior at different power state.
Change-Id: I88dbad30101e7983304c15f88b52b31457607749
Signed-off-by: eddylu <eddylu@ami.corp-partner.google.com>
Reviewed-on: https://chromium-review.googlesource.com/1116628
Commit-Ready: Eddy Lu <eddylu@ami.corp-partner.google.com>
Tested-by: Eddy Lu <eddylu@ami.corp-partner.google.com>
Reviewed-by: Jett Rink <jettrink@chromium.org>
Diffstat (limited to 'board/phaser')
-rw-r--r-- | board/phaser/board.h | 2 | ||||
-rw-r--r-- | board/phaser/build.mk | 2 | ||||
-rw-r--r-- | board/phaser/gpio.inc | 4 | ||||
-rw-r--r-- | board/phaser/led.c | 94 |
4 files changed, 99 insertions, 3 deletions
diff --git a/board/phaser/board.h b/board/phaser/board.h index 62b282460f..367704df9b 100644 --- a/board/phaser/board.h +++ b/board/phaser/board.h @@ -23,6 +23,8 @@ /* Optional features */ #define CONFIG_SYSTEM_UNLOCKED /* Allow dangerous commands while in dev. */ +#define CONFIG_LED_COMMON +#define OCTOPUS_POWER_LED #define CONFIG_TEMP_SENSOR #define CONFIG_THERMISTOR #define CONFIG_STEINHART_HART_3V3_13K7_47K_4050B diff --git a/board/phaser/build.mk b/board/phaser/build.mk index e25c6bb88d..7e806f4667 100644 --- a/board/phaser/build.mk +++ b/board/phaser/build.mk @@ -11,6 +11,6 @@ CHIP_FAMILY:=npcx7 CHIP_VARIANT:=npcx7m6fb BASEBOARD:=octopus -board-y=board.o +board-y=board.o led.o board-$(CONFIG_BATTERY_SMART)+=battery.o board-$(CONFIG_USB_POWER_DELIVERY)+=usb_pd_policy.o diff --git a/board/phaser/gpio.inc b/board/phaser/gpio.inc index 73ba0f728d..d8c9db8c15 100644 --- a/board/phaser/gpio.inc +++ b/board/phaser/gpio.inc @@ -136,8 +136,8 @@ GPIO(USB_C1_HPD_1V8_ODL, PIN(C, 6), GPIO_INPUT | /* C1 DP Hotplug Detect */ GPIO(USB2_OTG_ID, PIN(8, 3), GPIO_ODR_LOW) /* OTG ID */ /* LED */ -GPIO(BAT_LED_ORANGE_L, PIN(C, 3), GPIO_OUT_HIGH) /* LED_1_L */ -GPIO(BAT_LED_BLUE_L, PIN(C, 4), GPIO_OUT_HIGH) /* LED_2_L */ +GPIO(BAT_LED_RED_L, PIN(C, 3), GPIO_OUT_HIGH) /* LED_1_L */ +GPIO(BAT_LED_GREEN_L, PIN(C, 4), GPIO_OUT_HIGH) /* LED_2_L */ GPIO(LED_3_L, PIN(D, 7), GPIO_OUT_HIGH) /* Keyboard Backlight */ diff --git a/board/phaser/led.c b/board/phaser/led.c new file mode 100644 index 0000000000..e9a8912111 --- /dev/null +++ b/board/phaser/led.c @@ -0,0 +1,94 @@ +/* Copyright 2018 The Chromium OS Authors. All rights reserved. + * Use of this source code is governed by a BSD-style license that can be + * found in the LICENSE file. + * + * Power and battery LED control for Phaser + */ + +#include "ec_commands.h" +#include "gpio.h" +#include "led_common.h" +#include "led_states.h" +#include "chipset.h" + +#define LED_ON_LVL 0 +#define LED_OFF_LVL 1 + +const int led_charge_lvl_1 = 5; + +const int led_charge_lvl_2 = 97; + +const int led_power_blink_on_msec = 3000; + +const int led_power_blink_off_msec = 500; + +const struct led_descriptor + led_bat_state_table[LED_NUM_STATES][LED_NUM_PHASES] = { + [STATE_CHARGING_LVL_1] = {{EC_LED_COLOR_RED, LED_INDEFINITE} }, + [STATE_CHARGING_LVL_2] = {{EC_LED_COLOR_AMBER, LED_INDEFINITE} }, + [STATE_CHARGING_FULL_CHARGE] = {{EC_LED_COLOR_GREEN, LED_INDEFINITE} }, + [STATE_DISCHARGE_S0] = {{LED_OFF, LED_INDEFINITE} }, + [STATE_DISCHARGE_S3] = {{LED_OFF, LED_INDEFINITE} }, + [STATE_DISCHARGE_S5] = {{LED_OFF, LED_INDEFINITE} }, + [STATE_BATTERY_ERROR] = {{EC_LED_COLOR_RED, 1 * LED_ONE_SEC}, {LED_OFF, 1 * LED_ONE_SEC} }, + [STATE_FACTORY_TEST] = {{EC_LED_COLOR_RED, 2 * LED_ONE_SEC}, {EC_LED_COLOR_GREEN, 2 * LED_ONE_SEC} }, +}; + +const enum ec_led_id supported_led_ids[] = { + EC_LED_ID_POWER_LED, + EC_LED_ID_BATTERY_LED +}; + +const int supported_led_ids_count = ARRAY_SIZE(supported_led_ids); + +void led_set_color_power(int enable) +{ + if (enable) + gpio_set_level(GPIO_LED_3_L, LED_ON_LVL); + else + gpio_set_level(GPIO_LED_3_L, LED_OFF_LVL); +} + +void led_set_color_battery(enum ec_led_colors color) +{ + switch (color) { + case EC_LED_COLOR_RED: + gpio_set_level(GPIO_BAT_LED_RED_L, LED_ON_LVL); + gpio_set_level(GPIO_BAT_LED_GREEN_L, LED_OFF_LVL); + break; + case EC_LED_COLOR_AMBER: + gpio_set_level(GPIO_BAT_LED_RED_L, LED_ON_LVL); + gpio_set_level(GPIO_BAT_LED_GREEN_L, LED_ON_LVL); + break; + case EC_LED_COLOR_GREEN: + gpio_set_level(GPIO_BAT_LED_RED_L, LED_OFF_LVL); + gpio_set_level(GPIO_BAT_LED_GREEN_L, LED_ON_LVL); + break; + default: /* LED_OFF and other unsupported colors */ + gpio_set_level(GPIO_BAT_LED_RED_L, LED_OFF_LVL); + gpio_set_level(GPIO_BAT_LED_GREEN_L, LED_OFF_LVL); + break; + } +} + +void led_get_brightness_range(enum ec_led_id led_id, uint8_t *brightness_range) +{ + brightness_range[EC_LED_COLOR_RED] = 1; + brightness_range[EC_LED_COLOR_AMBER] = 1; + brightness_range[EC_LED_COLOR_GREEN] = 1; +} + +int led_set_brightness(enum ec_led_id led_id, const uint8_t *brightness) +{ + if (brightness[EC_LED_COLOR_RED] != 0) + led_set_color_battery(EC_LED_COLOR_RED); + else if (brightness[EC_LED_COLOR_AMBER] != 0) + led_set_color_battery(EC_LED_COLOR_AMBER); + else if (brightness[EC_LED_COLOR_GREEN] != 0) + led_set_color_battery(EC_LED_COLOR_GREEN); + else + led_set_color_battery(LED_OFF); + + return EC_SUCCESS; +} + |