summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorben chen <ben.chen2@quanta.corp-partner.google.com>2022-10-18 10:50:03 +0800
committerChromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com>2022-10-18 11:06:03 +0000
commit03eb03f2b2ac8d298ba2ea9f8c263a7018375314 (patch)
treef77c3b0c58103e1c59dadacc75cf7184faa9f1e1
parent0d071906c5807082978aaeb3069485a22731e388 (diff)
downloadchrome-ec-factory-corsola-15197.B-main.tar.gz
kuldax: modify power led color green to whitefactory-corsola-15197.B-main
Supports power led color to white from green type BUG=b:254014264 BRANCH=cros/main TEST="ectool led power white=100" can show correct led color. Change-Id: If55c5f480444933949eeb5bd60f88f658ffb83b5 Signed-off-by: Ben Chen <ben.chen2@quanta.corp-partner.google.com> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3960512 Tested-by: Sue Chen <sue.chen@quanta.corp-partner.google.com> Reviewed-by: Zhuohao Lee <zhuohao@chromium.org>
-rw-r--r--board/kuldax/board.h2
-rw-r--r--board/kuldax/led.c34
-rw-r--r--board/kuldax/pwm.c4
3 files changed, 20 insertions, 20 deletions
diff --git a/board/kuldax/board.h b/board/kuldax/board.h
index 64804bf16a..66ce4603b2 100644
--- a/board/kuldax/board.h
+++ b/board/kuldax/board.h
@@ -175,7 +175,7 @@ enum temp_sensor_id {
};
enum pwm_channel {
- PWM_CH_LED_GREEN, /* PWM0 */
+ PWM_CH_LED_WHITE, /* PWM0 */
PWM_CH_FAN, /* PWM5 */
PWM_CH_LED_RED, /* PWM2 */
PWM_CH_COUNT
diff --git a/board/kuldax/led.c b/board/kuldax/led.c
index 842cee0530..8da760193b 100644
--- a/board/kuldax/led.c
+++ b/board/kuldax/led.c
@@ -3,8 +3,8 @@
* found in the LICENSE file.
*
* Power LED control for Brask.
- * Solid green - active power
- * Green flashing - suspended
+ * Solid white - active power
+ * White flashing - suspended
* Red flashing - alert
* Solid red - critical
*/
@@ -34,7 +34,7 @@ const int supported_led_ids_count = ARRAY_SIZE(supported_led_ids);
enum led_color {
LED_OFF = 0,
LED_RED,
- LED_GREEN,
+ LED_WHITE,
/* Number of colors, not a color itself */
LED_COLOR_COUNT
@@ -42,7 +42,7 @@ enum led_color {
static int set_color_power(enum led_color color, int duty)
{
- int green = 0;
+ int white = 0;
int red = 0;
if (duty < 0 || 100 < duty)
@@ -51,8 +51,8 @@ static int set_color_power(enum led_color color, int duty)
switch (color) {
case LED_OFF:
break;
- case LED_GREEN:
- green = 1;
+ case LED_WHITE:
+ white = 1;
break;
case LED_RED:
red = 1;
@@ -66,10 +66,10 @@ static int set_color_power(enum led_color color, int duty)
else
pwm_set_duty(PWM_CH_LED_RED, 0);
- if (green)
- pwm_set_duty(PWM_CH_LED_GREEN, duty);
+ if (white)
+ pwm_set_duty(PWM_CH_LED_WHITE, duty);
else
- pwm_set_duty(PWM_CH_LED_GREEN, 0);
+ pwm_set_duty(PWM_CH_LED_WHITE, 0);
return EC_SUCCESS;
}
@@ -138,7 +138,7 @@ static void led_tick(void)
static void led_suspend(void)
{
- CONFIG_TICK(LED_PULSE_TICK_US, LED_GREEN);
+ CONFIG_TICK(LED_PULSE_TICK_US, LED_WHITE);
led_tick();
}
DECLARE_DEFERRED(led_suspend);
@@ -178,7 +178,7 @@ static void led_resume(void)
hook_call_deferred(&led_suspend_data, -1);
hook_call_deferred(&led_shutdown_data, -1);
if (led_auto_control_is_enabled(EC_LED_ID_POWER_LED))
- set_color(EC_LED_ID_POWER_LED, LED_GREEN, 100);
+ set_color(EC_LED_ID_POWER_LED, LED_WHITE, 100);
}
DECLARE_HOOK(HOOK_CHIPSET_RESUME, led_resume, HOOK_PRIO_DEFAULT);
@@ -220,8 +220,8 @@ static int command_led(int argc, const char **argv)
set_color(id, LED_OFF, 0);
} else if (!strcasecmp(argv[1], "red")) {
set_color(id, LED_RED, 100);
- } else if (!strcasecmp(argv[1], "green")) {
- set_color(id, LED_GREEN, 100);
+ } else if (!strcasecmp(argv[1], "white")) {
+ set_color(id, LED_WHITE, 100);
} else if (!strcasecmp(argv[1], "alert")) {
led_alert(1);
} else if (!strcasecmp(argv[1], "crit")) {
@@ -231,21 +231,21 @@ static int command_led(int argc, const char **argv)
}
return EC_SUCCESS;
}
-DECLARE_CONSOLE_COMMAND(led, command_led, "[debug|red|green|off|alert|crit]",
+DECLARE_CONSOLE_COMMAND(led, command_led, "[debug|red|white|off|alert|crit]",
"Turn on/off LED.");
void led_get_brightness_range(enum ec_led_id led_id, uint8_t *brightness_range)
{
brightness_range[EC_LED_COLOR_RED] = 100;
- brightness_range[EC_LED_COLOR_GREEN] = 100;
+ brightness_range[EC_LED_COLOR_WHITE] = 100;
}
int led_set_brightness(enum ec_led_id id, const uint8_t *brightness)
{
if (brightness[EC_LED_COLOR_RED])
return set_color(id, LED_RED, brightness[EC_LED_COLOR_RED]);
- else if (brightness[EC_LED_COLOR_GREEN])
- return set_color(id, LED_GREEN, brightness[EC_LED_COLOR_GREEN]);
+ else if (brightness[EC_LED_COLOR_WHITE])
+ return set_color(id, LED_WHITE, brightness[EC_LED_COLOR_WHITE]);
else
return set_color(id, LED_OFF, 0);
}
diff --git a/board/kuldax/pwm.c b/board/kuldax/pwm.c
index fe7e82894a..9ad0a686cc 100644
--- a/board/kuldax/pwm.c
+++ b/board/kuldax/pwm.c
@@ -11,7 +11,7 @@
#include "pwm_chip.h"
const struct pwm_t pwm_channels[] = {
- [PWM_CH_LED_GREEN] = { .channel = 0,
+ [PWM_CH_LED_WHITE] = { .channel = 0,
.flags = PWM_CONFIG_ACTIVE_LOW |
PWM_CONFIG_DSLEEP,
.freq = 2000 },
@@ -35,6 +35,6 @@ static void board_pwm_init(void)
pwm_set_duty(PWM_CH_FAN, 100);
pwm_enable(PWM_CH_LED_RED, 1);
- pwm_enable(PWM_CH_LED_GREEN, 1);
+ pwm_enable(PWM_CH_LED_WHITE, 1);
}
DECLARE_HOOK(HOOK_INIT, board_pwm_init, HOOK_PRIO_DEFAULT);