summaryrefslogtreecommitdiff
path: root/board/guybrush
diff options
context:
space:
mode:
authorDiana Z <dzigterman@chromium.org>2021-04-09 15:27:39 -0600
committerCommit Bot <commit-bot@chromium.org>2021-04-13 06:36:17 +0000
commit76cd38a0b214bdbfd45ffa7c3633329a8eab65ab (patch)
treebb822a8d761ffd5494d66e6bc1e9d691794f7c92 /board/guybrush
parente764f94a625fb4aeac3ed92f42a2b5085fb19a14 (diff)
downloadchrome-ec-76cd38a0b214bdbfd45ffa7c3633329a8eab65ab.tar.gz
Guybrush: Add LED behavior
Use the common LED on/off states for LED behavior. Set it up for our two colors: amber and white. BRANCH=None BUG=b:184845299 TEST=on guybrush, confirm charging shows Amber, discharging is white, shutdown shows no LED on Signed-off-by: Diana Z <dzigterman@chromium.org> Change-Id: Idc57de843fd2addaaa3006d4abd8566d76b2de6b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2818151 Reviewed-by: Rob Barnes <robbarnes@google.com> Commit-Queue: Rob Barnes <robbarnes@google.com>
Diffstat (limited to 'board/guybrush')
-rw-r--r--board/guybrush/board.h4
-rw-r--r--board/guybrush/build.mk2
-rw-r--r--board/guybrush/led.c90
3 files changed, 95 insertions, 1 deletions
diff --git a/board/guybrush/board.h b/board/guybrush/board.h
index 93b718e198..ae6d9fd1cd 100644
--- a/board/guybrush/board.h
+++ b/board/guybrush/board.h
@@ -37,6 +37,10 @@
/* Fan features */
+/* LED features */
+#define CONFIG_LED_COMMON
+#define CONFIG_LED_ONOFF_STATES
+
#ifndef __ASSEMBLER__
#include "gpio_signal.h"
diff --git a/board/guybrush/build.mk b/board/guybrush/build.mk
index 1e79b1895e..4324e2b034 100644
--- a/board/guybrush/build.mk
+++ b/board/guybrush/build.mk
@@ -9,4 +9,4 @@
BASEBOARD:=guybrush
board-y=board.o
-board-y+=board_fw_config.o
+board-y+=board_fw_config.o led.o
diff --git a/board/guybrush/led.c b/board/guybrush/led.c
new file mode 100644
index 0000000000..d82070cdaa
--- /dev/null
+++ b/board/guybrush/led.c
@@ -0,0 +1,90 @@
+/* Copyright 2021 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.
+ *
+ * Guybrush specific PWM LED settings.
+ */
+
+#include "common.h"
+#include "led_onoff_states.h"
+#include "led_common.h"
+#include "gpio.h"
+#include "pwm.h"
+
+/* Note PWM LEDs are active low */
+#define LED_OFF_LVL 1
+#define LED_ON_LVL 0
+
+#define CPRINTS(format, args...) cprints(CC_PWM, format, ## args)
+
+const int led_charge_lvl_1 = 5;
+
+const int led_charge_lvl_2 = 97;
+
+struct led_descriptor led_bat_state_table[LED_NUM_STATES][LED_NUM_PHASES] = {
+ [STATE_CHARGING_LVL_1] = {{EC_LED_COLOR_AMBER, LED_INDEFINITE} },
+ [STATE_CHARGING_LVL_2] = {{EC_LED_COLOR_AMBER, LED_INDEFINITE} },
+ [STATE_CHARGING_FULL_CHARGE] = {{EC_LED_COLOR_WHITE, LED_INDEFINITE} },
+ [STATE_DISCHARGE_S0] = {{EC_LED_COLOR_WHITE, LED_INDEFINITE} },
+ [STATE_DISCHARGE_S3] = {{EC_LED_COLOR_WHITE, 1 * LED_ONE_SEC},
+ {LED_OFF, 1 * LED_ONE_SEC} },
+ [STATE_DISCHARGE_S5] = {{LED_OFF, LED_INDEFINITE} },
+ [STATE_BATTERY_ERROR] = {{EC_LED_COLOR_AMBER, 1 * LED_ONE_SEC},
+ {LED_OFF, 1 * LED_ONE_SEC} },
+ [STATE_FACTORY_TEST] = {{EC_LED_COLOR_AMBER, 2 * LED_ONE_SEC},
+ {EC_LED_COLOR_WHITE, 2 * LED_ONE_SEC} },
+};
+
+const enum ec_led_id supported_led_ids[] = {
+ EC_LED_ID_BATTERY_LED,
+};
+
+const int supported_led_ids_count = ARRAY_SIZE(supported_led_ids);
+
+void led_set_color_battery(enum ec_led_colors color)
+{
+ switch (color) {
+ case EC_LED_COLOR_AMBER:
+ pwm_enable(PWM_CH_LED_CHRG, LED_ON_LVL);
+ pwm_enable(PWM_CH_LED_FULL, LED_OFF_LVL);
+ break;
+ case EC_LED_COLOR_WHITE:
+ pwm_enable(PWM_CH_LED_CHRG, LED_OFF_LVL);
+ pwm_enable(PWM_CH_LED_FULL, LED_ON_LVL);
+ break;
+ case LED_OFF:
+ pwm_enable(PWM_CH_LED_CHRG, LED_OFF_LVL);
+ pwm_enable(PWM_CH_LED_FULL, LED_OFF_LVL);
+ break;
+ default: /* Unsupported colors */
+ CPRINTS("Unsupported LED color: %d", color);
+ pwm_enable(PWM_CH_LED_CHRG, LED_OFF_LVL);
+ pwm_enable(PWM_CH_LED_FULL, LED_OFF_LVL);
+ break;
+ }
+}
+
+void led_get_brightness_range(enum ec_led_id led_id, uint8_t *brightness_range)
+{
+ if (led_id == EC_LED_ID_BATTERY_LED) {
+ brightness_range[EC_LED_COLOR_AMBER] = 1;
+ brightness_range[EC_LED_COLOR_WHITE] = 1;
+ }
+}
+
+int led_set_brightness(enum ec_led_id led_id, const uint8_t *brightness)
+{
+ if (led_id == EC_LED_ID_BATTERY_LED) {
+ if (brightness[EC_LED_COLOR_WHITE] != 0)
+ led_set_color_battery(EC_LED_COLOR_WHITE);
+ else if (brightness[EC_LED_COLOR_AMBER] != 0)
+ led_set_color_battery(EC_LED_COLOR_AMBER);
+ else
+ led_set_color_battery(LED_OFF);
+ } else {
+ CPRINTS("Unsupported LED set: %d", led_id);
+ return EC_ERROR_INVAL;
+ }
+
+ return EC_SUCCESS;
+}