From f2225767a2bb6cc7d643ee7f13aa957542c6471a Mon Sep 17 00:00:00 2001 From: WillyJL Date: Sat, 23 Aug 2025 01:59:31 +0200 Subject: [PATCH] Give useful names to ledblink*() functions --- firmware/application/src/app_main.c | 32 +++---- firmware/application/src/rfid_main.c | 2 +- firmware/application/src/rgb_marquee.c | 116 ++++++++++++------------- firmware/application/src/rgb_marquee.h | 14 +-- 4 files changed, 82 insertions(+), 82 deletions(-) diff --git a/firmware/application/src/app_main.c b/firmware/application/src/app_main.c index 958638a..8d91bac 100644 --- a/firmware/application/src/app_main.c +++ b/firmware/application/src/app_main.c @@ -314,11 +314,11 @@ static void system_off_enter(void) { color = 2; } } - if (m_system_off_processing) ledblink5(color, slot, dir ? 7 : 0); - if (m_system_off_processing) ledblink4(color, dir, 7, 99, 75); - if (m_system_off_processing) ledblink4(color, !dir, 7, 75, 50); - if (m_system_off_processing) ledblink4(color, dir, 7, 50, 25); - if (m_system_off_processing) ledblink4(color, !dir, 7, 25, 0); + if (m_system_off_processing) rgb_marquee_sweep_from_to(color, slot, dir ? 7 : 0); + if (m_system_off_processing) rgb_marquee_sweep_fade(color, dir, 7, 99, 75); + if (m_system_off_processing) rgb_marquee_sweep_fade(color, !dir, 7, 75, 50); + if (m_system_off_processing) rgb_marquee_sweep_fade(color, dir, 7, 50, 25); + if (m_system_off_processing) rgb_marquee_sweep_fade(color, !dir, 7, 25, 0); } rgb_marquee_stop(); if (!m_system_off_processing) { @@ -460,11 +460,11 @@ static void check_wakeup_src(void) { // Button wake-up boot animation uint8_t animation_config = settings_get_animation_config(); if (animation_config == SettingsAnimationModeFull) { - ledblink2(color, !dir, 11); - ledblink2(color, dir, 11); - ledblink2(color, !dir, dir ? slot : 7 - slot); + rgb_marquee_sweep_to(color, !dir, 11); + rgb_marquee_sweep_to(color, dir, 11); + rgb_marquee_sweep_to(color, !dir, dir ? slot : 7 - slot); } else if (animation_config == SettingsAnimationModeMinimal) { - ledblink2(color, !dir, dir ? slot : 7 - slot); + rgb_marquee_sweep_to(color, !dir, dir ? slot : 7 - slot); } else { set_slot_light_color(color); } @@ -497,7 +497,7 @@ static void check_wakeup_src(void) { uint8_t animation_config = settings_get_animation_config(); if (animation_config == SettingsAnimationModeFull) { // In the case of field wake-up, only one round of RGB is swept as the power-on animation - ledblink2(color, !dir, dir ? slot : 7 - slot); + rgb_marquee_sweep_to(color, !dir, dir ? slot : 7 - slot); } set_slot_light_color(color); light_up_by_slot(); @@ -526,9 +526,9 @@ static void check_wakeup_src(void) { tag_emulation_factory_init(); // RGB - ledblink2(0, !dir, 11); - ledblink2(1, dir, 11); - ledblink2(2, !dir, 11); + rgb_marquee_sweep_to(0, !dir, 11); + rgb_marquee_sweep_to(1, dir, 11); + rgb_marquee_sweep_to(2, !dir, 11); // Show RGB for slot. set_slot_light_color(color); @@ -924,12 +924,12 @@ static void blink_usb_led_status(void) { } } else { // The light effect is enabled and can be displayed - if (is_rgb_marquee_enable()) { + if (rgb_marquee_is_enabled()) { is_working = true; if (g_usb_port_opened) { - ledblink1(color, dir); + rgb_marquee_usb_open_sweep(color, dir); } else { - ledblink6(); + rgb_marquee_usb_idle(); } } else { if (is_working) { diff --git a/firmware/application/src/rfid_main.c b/firmware/application/src/rfid_main.c index f93d448..deb72e7 100644 --- a/firmware/application/src/rfid_main.c +++ b/firmware/application/src/rfid_main.c @@ -94,7 +94,7 @@ void light_up_by_slot(void) { void apply_slot_change(uint8_t slot_now, uint8_t slot_new) { uint8_t color_now = get_color_by_slot(slot_now); uint8_t color_new = get_color_by_slot(slot_new); - ledblink3(slot_now, color_now, slot_new, color_new); + rgb_marquee_slot_switch(slot_now, color_now, slot_new, color_new); } /** diff --git a/firmware/application/src/rgb_marquee.c b/firmware/application/src/rgb_marquee.c index 4241668..341406c 100644 --- a/firmware/application/src/rgb_marquee.c +++ b/firmware/application/src/rgb_marquee.c @@ -32,9 +32,9 @@ nrf_drv_pwm_config_t pwm_config = {//PWM configuration structure .step_mode = NRF_PWM_STEP_AUTO }; static autotimer *timer; -static uint8_t ledblink6_step = 0; -static uint8_t ledblink6_color = RGB_RED; -static uint8_t ledblink1_step = 0; +static uint8_t rgb_marquee_usb_idle_step = 0; +static uint8_t rgb_marquee_usb_idle_color = RGB_RED; +static uint8_t rgb_marquee_usb_open_sweep_step = 0; extern bool g_usb_led_marquee_enable; @@ -45,14 +45,14 @@ void rgb_marquee_init(void) { void rgb_marquee_stop(void) { nrfx_pwm_stop(&pwm0_ins, true); nrfx_pwm_uninit(&pwm0_ins);//turn off pwm output - ledblink6_step = 0; - ledblink1_step = 0; + rgb_marquee_usb_idle_step = 0; + rgb_marquee_usb_open_sweep_step = 0; } // reset RGB state machines to force a refresh of the LED color void rgb_marquee_reset(void) { - ledblink6_step = 0; - ledblink1_step = 0; + rgb_marquee_usb_idle_step = 0; + rgb_marquee_usb_open_sweep_step = 0; } // Brightness to PWM value @@ -62,12 +62,12 @@ uint16_t get_pwmduty(uint8_t light_level) { // 4 Lights and the level of brightness levels (no return) //COLOR 0-R,1-G,2-B -void ledblink1(uint8_t color, uint8_t dir) { +void rgb_marquee_usb_open_sweep(uint8_t color, uint8_t dir) { static uint8_t startled = 0; static uint8_t setled = 0; uint32_t *led_pins_arr; - if (!g_usb_led_marquee_enable && ledblink1_step != 0) { + if (!g_usb_led_marquee_enable && rgb_marquee_usb_open_sweep_step != 0) { startled = 0; setled = 0; rgb_marquee_stop(); @@ -81,7 +81,7 @@ void ledblink1(uint8_t color, uint8_t dir) { led_pins_arr = hw_get_led_reversal_array(); } - if (ledblink1_step == 0) { + if (rgb_marquee_usb_open_sweep_step == 0) { //Adjust the color set_slot_light_color(color); pwm_sequ_val.channel_0 = 1; @@ -89,13 +89,13 @@ void ledblink1(uint8_t color, uint8_t dir) { pwm_sequ_val.channel_2 = 1; pwm_sequ_val.channel_3 = 1; bsp_set_timer(timer, 0); - ledblink1_step = 1; + rgb_marquee_usb_open_sweep_step = 1; // Reset the state of the light when the USB is turned on to open the communication - ledblink6_step = 0; + rgb_marquee_usb_idle_step = 0; } - if (ledblink1_step == 1) { + if (rgb_marquee_usb_open_sweep_step == 1) { setled = startled; for (uint8_t i = 0; i < 4; i++) { pwm_config.output_pins[i] = led_pins_arr[setled]; @@ -109,12 +109,12 @@ void ledblink1(uint8_t color, uint8_t dir) { nrf_drv_pwm_simple_playback(&pwm0_ins, &seq, 1, NRF_DRV_PWM_FLAG_LOOP); bsp_set_timer(timer, 0); - ledblink1_step = 2; + rgb_marquee_usb_open_sweep_step = 2; } - if (ledblink1_step == 2) { + if (rgb_marquee_usb_open_sweep_step == 2) { if (!(NO_TIMEOUT_1MS(timer, 80))) { - ledblink1_step = 1; + rgb_marquee_usb_open_sweep_step = 1; } } } @@ -122,7 +122,7 @@ void ledblink1(uint8_t color, uint8_t dir) { // 4 Lights Dragon Tail horizontal movement cycle (not returning), including the disappearance of the tail and the head of the head slowly //dir 0-from 1 card slot to 8 card slot, 1-from 8 card slot to 1 card slot (Direction, the end point is determined by the END parameter) //end To scan the number of lamps, decide the final animation area with the direction -void ledblink2(uint8_t color, uint8_t dir, uint8_t end) { +void rgb_marquee_sweep_to(uint8_t color, uint8_t dir, uint8_t end) { uint8_t startled = 0; uint8_t setled = 0; uint8_t leds2turnon = 0; @@ -203,12 +203,12 @@ void ledblink2(uint8_t color, uint8_t dir, uint8_t end) { //led_down LED to be extinguished //color_led_down The color of the LED to be extinguished 0-R,1-G,2-B volatile bool callback_waiting = 0; -static void ledblink3_pwm_callback(nrfx_pwm_evt_type_t event_type) { +static void rgb_marquee_slot_switch_pwm_callback(nrfx_pwm_evt_type_t event_type) { if (event_type == NRF_DRV_PWM_EVT_FINISHED) { callback_waiting = 1; } } -void ledblink3(uint8_t led_down, uint8_t color_led_down, uint8_t led_up, uint8_t color_led_up) { +void rgb_marquee_slot_switch(uint8_t led_down, uint8_t color_led_down, uint8_t led_up, uint8_t color_led_up) { int16_t light_level = 99; //ledBrightnessValue uint32_t *led_pins = hw_get_led_array(); if (led_down >= 0 && led_down <= 7) { @@ -229,7 +229,7 @@ void ledblink3(uint8_t led_down, uint8_t color_led_down, uint8_t led_up, uint8_t set_slot_light_color(color_led_down); - nrf_drv_pwm_init(&pwm0_ins, &pwm_config, ledblink3_pwm_callback); + nrf_drv_pwm_init(&pwm0_ins, &pwm_config, rgb_marquee_slot_switch_pwm_callback); nrf_drv_pwm_simple_playback(&pwm0_ins, &seq, 1, NRF_DRV_PWM_FLAG_LOOP); while (callback_waiting == 0); //Waiting for the output of the PWM module to complete @@ -257,7 +257,7 @@ void ledblink3(uint8_t led_down, uint8_t color_led_down, uint8_t led_up, uint8_t set_slot_light_color(color_led_up); - nrf_drv_pwm_init(&pwm0_ins, &pwm_config, ledblink3_pwm_callback); + nrf_drv_pwm_init(&pwm0_ins, &pwm_config, rgb_marquee_slot_switch_pwm_callback); nrf_drv_pwm_simple_playback(&pwm0_ins, &seq, 1, NRF_DRV_PWM_FLAG_LOOP); while (callback_waiting == 0); //Waiting for the output of the PWM module to complete @@ -272,7 +272,7 @@ void ledblink3(uint8_t led_down, uint8_t color_led_down, uint8_t led_up, uint8_t //dir 0-from 1 card slot to 8 card slot, 1-from 8 card slot to 1 card slot (Direction, the end point is determined by the END parameter) //end To scan the number of lamps, decide the final animation area with the direction //start_light stop_light 0-99 Indicate gradient brightness -void ledblink4(uint8_t color, uint8_t dir, uint8_t end, uint8_t start_light, uint8_t stop_light) { +void rgb_marquee_sweep_fade(uint8_t color, uint8_t dir, uint8_t end, uint8_t start_light, uint8_t stop_light) { uint8_t startled = 0; uint8_t setled = 0; uint8_t leds2turnon = 0; @@ -346,7 +346,7 @@ void ledblink4(uint8_t color, uint8_t dir, uint8_t end, uint8_t start_light, uin //color The color of the lit LED 0-R,1-G,2-B //start Start the lamp position //stop Stop lamp position -void ledblink5(uint8_t color, uint8_t start, uint8_t stop) { +void rgb_marquee_sweep_from_to(uint8_t color, uint8_t start, uint8_t stop) { uint8_t setled = start; uint32_t *led_pins = hw_get_led_array(); //Set the brightness @@ -375,25 +375,25 @@ void ledblink5(uint8_t color, uint8_t start, uint8_t stop) { // Charging animation // the current percentage of the battery 0-4 4 represents full electric breathing light volatile bool callback_waiting6 = 0; -void ledblink6_pwm_callback(nrfx_pwm_evt_type_t event_type) { +void rgb_marquee_usb_idle_pwm_callback(nrfx_pwm_evt_type_t event_type) { if (event_type == NRF_DRV_PWM_EVT_FINISHED) { callback_waiting6 = 1; } } -void ledblink6(void) { +void rgb_marquee_usb_idle(void) { uint32_t *led_array = hw_get_led_array(); const uint16_t delay_time = 25; static int16_t light_level = 99; //LED brightness value - if (!g_usb_led_marquee_enable && ledblink6_step != 0) { + if (!g_usb_led_marquee_enable && rgb_marquee_usb_idle_step != 0) { light_level = 99; callback_waiting6 = 0; rgb_marquee_stop(); return; } - if (ledblink6_step == 0) { - set_slot_light_color(ledblink6_color); + if (rgb_marquee_usb_idle_step == 0) { + set_slot_light_color(rgb_marquee_usb_idle_color); for (uint8_t i = 0; i < RGB_LIST_NUM; i++) { nrf_gpio_pin_clear(led_array[i]); } @@ -401,87 +401,87 @@ void ledblink6(void) { pwm_config.output_pins[1] = led_array[3]; pwm_config.output_pins[2] = led_array[4]; pwm_config.output_pins[3] = led_array[5]; - ledblink6_step = 1; + rgb_marquee_usb_idle_step = 1; // Reset the state of the lamp when the USB is not turned on - ledblink1_step = 0; + rgb_marquee_usb_open_sweep_step = 0; } - if (ledblink6_step == 1) { + if (rgb_marquee_usb_idle_step == 1) { light_level = 0; - ledblink6_step = 2; + rgb_marquee_usb_idle_step = 2; } - if (ledblink6_step == 2 || ledblink6_step == 3 || ledblink6_step == 4) { + if (rgb_marquee_usb_idle_step == 2 || rgb_marquee_usb_idle_step == 3 || rgb_marquee_usb_idle_step == 4) { if (light_level <= 99) { - if (ledblink6_step == 2) { + if (rgb_marquee_usb_idle_step == 2) { //Treatment brightness pwm_sequ_val.channel_0 = get_pwmduty(light_level); pwm_sequ_val.channel_1 = pwm_sequ_val.channel_0; pwm_sequ_val.channel_2 = pwm_sequ_val.channel_0; pwm_sequ_val.channel_3 = pwm_sequ_val.channel_0; nrfx_pwm_uninit(&pwm0_ins); //Close PWM output - set_slot_light_color(ledblink6_color); - nrf_drv_pwm_init(&pwm0_ins, &pwm_config, ledblink6_pwm_callback); + set_slot_light_color(rgb_marquee_usb_idle_color); + nrf_drv_pwm_init(&pwm0_ins, &pwm_config, rgb_marquee_usb_idle_pwm_callback); nrf_drv_pwm_simple_playback(&pwm0_ins, &seq, 1, NRF_DRV_PWM_FLAG_LOOP); - ledblink6_step = 3; + rgb_marquee_usb_idle_step = 3; } - if (ledblink6_step == 3) { //Waiting for the output of the PWM module to complete + if (rgb_marquee_usb_idle_step == 3) { //Waiting for the output of the PWM module to complete if (callback_waiting6 != 0) { - ledblink6_step = 4; + rgb_marquee_usb_idle_step = 4; bsp_set_timer(timer, 0); } } - if (ledblink6_step == 4) { + if (rgb_marquee_usb_idle_step == 4) { if (!NO_TIMEOUT_1MS(timer, delay_time)) { callback_waiting = 0; light_level++; - ledblink6_step = 2; + rgb_marquee_usb_idle_step = 2; } } } else { - ledblink6_step = 5; + rgb_marquee_usb_idle_step = 5; } } - if (ledblink6_step == 5) { + if (rgb_marquee_usb_idle_step == 5) { light_level = 99; - ledblink6_step = 6; + rgb_marquee_usb_idle_step = 6; } - if (ledblink6_step == 6 || ledblink6_step == 7 || ledblink6_step == 8) { + if (rgb_marquee_usb_idle_step == 6 || rgb_marquee_usb_idle_step == 7 || rgb_marquee_usb_idle_step == 8) { if (light_level >= 0) { - if (ledblink6_step == 6) { + if (rgb_marquee_usb_idle_step == 6) { //Treatment brightness pwm_sequ_val.channel_0 = get_pwmduty(light_level); pwm_sequ_val.channel_1 = pwm_sequ_val.channel_0; pwm_sequ_val.channel_2 = pwm_sequ_val.channel_0; pwm_sequ_val.channel_3 = pwm_sequ_val.channel_0; nrfx_pwm_uninit(&pwm0_ins); //Close PWM output - set_slot_light_color(ledblink6_color); - nrf_drv_pwm_init(&pwm0_ins, &pwm_config, ledblink6_pwm_callback); + set_slot_light_color(rgb_marquee_usb_idle_color); + nrf_drv_pwm_init(&pwm0_ins, &pwm_config, rgb_marquee_usb_idle_pwm_callback); nrf_drv_pwm_simple_playback(&pwm0_ins, &seq, 1, NRF_DRV_PWM_FLAG_LOOP); - ledblink6_step = 7; + rgb_marquee_usb_idle_step = 7; } - if (ledblink6_step == 7) { //Waiting for the output of the PWM module to complete + if (rgb_marquee_usb_idle_step == 7) { //Waiting for the output of the PWM module to complete if (callback_waiting6 != 0) { - ledblink6_step = 8; + rgb_marquee_usb_idle_step = 8; bsp_set_timer(timer, 0); } } - if (ledblink6_step == 8) { + if (rgb_marquee_usb_idle_step == 8) { if (!NO_TIMEOUT_1MS(timer, delay_time)) { callback_waiting = 0; light_level--; - ledblink6_step = 6; + rgb_marquee_usb_idle_step = 6; } } } else { - ledblink6_step = 0; - //if (++ledblink6_color == RGB_WHITE) ledblink6_color = RGB_RED; + rgb_marquee_usb_idle_step = 0; + //if (++rgb_marquee_usb_idle_color == RGB_WHITE) rgb_marquee_usb_idle_color = RGB_RED; uint8_t new_color = rand() % 6; - for (; new_color == ledblink6_color; new_color = rand() % 6); - ledblink6_color = new_color; + for (; new_color == rgb_marquee_usb_idle_color; new_color = rand() % 6); + rgb_marquee_usb_idle_color = new_color; } } } @@ -492,6 +492,6 @@ void ledblink6(void) { * @return true Make the state, flickering in the lighting effect * @return false The state is prohibited, in the state of ordinary card slot indicator */ -bool is_rgb_marquee_enable(void) { +bool rgb_marquee_is_enabled(void) { return g_usb_led_marquee_enable; } diff --git a/firmware/application/src/rgb_marquee.h b/firmware/application/src/rgb_marquee.h index edfa70e..b0fab9e 100644 --- a/firmware/application/src/rgb_marquee.h +++ b/firmware/application/src/rgb_marquee.h @@ -8,12 +8,12 @@ void rgb_marquee_init(void); void rgb_marquee_stop(void); void rgb_marquee_reset(void); -bool is_rgb_marquee_enable(void); -void ledblink1(uint8_t color, uint8_t dir); -void ledblink2(uint8_t color, uint8_t dir, uint8_t end); -void ledblink3(uint8_t led_down, uint8_t color_led_down, uint8_t led_up, uint8_t color_led_up); -void ledblink4(uint8_t color, uint8_t dir, uint8_t end, uint8_t start_light, uint8_t stop_light); -void ledblink5(uint8_t color, uint8_t start, uint8_t stop); -void ledblink6(void); +bool rgb_marquee_is_enabled(void); +void rgb_marquee_usb_open_sweep(uint8_t color, uint8_t dir); +void rgb_marquee_sweep_to(uint8_t color, uint8_t dir, uint8_t end); +void rgb_marquee_slot_switch(uint8_t led_down, uint8_t color_led_down, uint8_t led_up, uint8_t color_led_up); +void rgb_marquee_sweep_fade(uint8_t color, uint8_t dir, uint8_t end, uint8_t start_light, uint8_t stop_light); +void rgb_marquee_sweep_from_to(uint8_t color, uint8_t start, uint8_t stop); +void rgb_marquee_usb_idle(void); #endif