fwk: led: runtime to update the led duration

In the commit a30cccd, we only modified the
LED driver to reduce the tick from 200ms to 10ms when the system enter suspend.

We did not modify the LED driver to change the duration at runtime.
It causes the period-ms value not to meet the actual behavior.

This change modified the LED driver to change the color duration
at runtime.

BRANCH=fwk-main
BUG=None
TEST=When the system enter suspend, the led breathing can meet
     the period-ms value.

Signed-off-by: Josh Tsai <[email protected]>
(cherry picked from commit 5c04251c8ce2b246075a4d744d05432da730cf84)

Author: Josh-Tsai

zephyr/program/framework/include/led.h

@@ -141,21 +141,21 @@ struct led_pins_node_t {
 
 struct pattern_color_node_t {
 	struct led_pins_node_t *led_color_node;
-	uint8_t duration;
+	int period_ms;
 };
 
 struct led_pattern_node_t {
 	uint8_t cur_color;
-	uint8_t ticks;
+	uint16_t ticks;
 	enum led_transition transition;
 	struct pattern_color_node_t *pattern_color;
 	uint8_t pattern_len;
 };
 
 #define GET_COLOR(pattern_element, color_index) \
 	pattern_element.pattern_color[color_index].led_color
-#define GET_DURATION(pattern_element, color_index) \
-	pattern_element.pattern_color[color_index].duration
+#define GET_DURATION(pattern_element, color_index, curr_tick) \
+	(pattern_element.pattern_color[color_index].period_ms / curr_tick)
 
 /**
  * Set LED color using color enum
@@ -188,6 +188,13 @@ int fp_led_auto_is_enable(void);
 
 int kbbl_auto_dim_is_enable(void);
 
+/**
+ * Return the current tick time from led driver
+ *
+ * @return current led tick time
+ */
+int led_get_current_tick_time(void);
+
 #ifdef TEST_BUILD
 const struct led_pins_node_t *led_get_node(enum led_color color,
 					   enum ec_led_id led_id);

zephyr/program/framework/src/led.c

@@ -62,8 +62,7 @@ DT_FOREACH_CHILD_STATUS_OKAY_VARGS(
 		.led_color_node = &PINS_NODE_FROM_POLICY(                     \
 			GET_PROP(DT_PARENT(id), led_id),                      \
 			GET_PROP(id, led_color)),                             \
-		.duration =                                                   \
-			DT_PROP_OR(id, period_ms, 0) / HOOK_TICK_INTERVAL_MS, \
+		.period_ms = DT_PROP_OR(id, period_ms, 0)                     \
 	},
 
 #define PATTERN_COLOR_ARRAY(id) DT_CAT(PATTERN_COLOR_, id)
@@ -156,6 +155,11 @@ static bool last_kbbl_led_brightness;
 #endif
 static int prev_als_lux;
 
+int led_get_current_tick_time(void)
+{
+	return led_tick_time;
+}
+
 /*
  * return auto dim status, it decided by BIOS setup menu
  * option(Power Button LED Brightness level).
@@ -322,11 +326,11 @@ static void set_color(int node_idx)
 
 		led_set_color_with_pattern(&patterns[i]);
 
-		if (GET_DURATION(patterns[i], patterns[i].cur_color) != 0)
+		if (GET_DURATION(patterns[i], patterns[i].cur_color, led_tick_time) != 0)
 			patterns[i].ticks++;
 
 		if (patterns[i].ticks >=
-		    GET_DURATION(patterns[i], patterns[i].cur_color)) {
+		    GET_DURATION(patterns[i], patterns[i].cur_color, led_tick_time)) {
 			patterns[i].cur_color++;
 			patterns[i].ticks = 0;
 		}

zephyr/program/framework/src/led_pwm.c

@@ -131,7 +131,8 @@ void led_set_color_with_pattern(const struct led_pattern_node_t *pattern)
 {
 	uint8_t pins_count = pattern->pattern_color[pattern->cur_color]
 				     .led_color_node->pins_count;
-	uint8_t duration = pattern->pattern_color[pattern->cur_color].duration;
+	int curr_tick = led_get_current_tick_time();
+	int duration = pattern->pattern_color[pattern->cur_color].period_ms / curr_tick;
 	struct pwm_pin_t *next_color =
 		pattern->pattern_color[pattern->cur_color]
 			.led_color_node->pwm_pins;