aboutsummaryrefslogtreecommitdiff
path: root/quantum/led_matrix.c
diff options
context:
space:
mode:
authorXScorpion2 <rcalt2vt@gmail.com>2021-06-19 21:03:06 -0500
committerGitHub <noreply@github.com>2021-06-19 19:03:06 -0700
commit5b7cf9fdc86bf92b465565b2be5d501ea9529f37 (patch)
tree96195d31e1fb0a220024526f787f98a3ece810a7 /quantum/led_matrix.c
parent3c790123ce23638cacc9cade86df25bcf6a981d5 (diff)
downloadqmk_firmware-5b7cf9fdc86bf92b465565b2be5d501ea9529f37.tar.gz
qmk_firmware-5b7cf9fdc86bf92b465565b2be5d501ea9529f37.zip
RGB Matrix eeprom write limiting (#13238)
Diffstat (limited to 'quantum/led_matrix.c')
-rw-r--r--quantum/led_matrix.c30
1 files changed, 16 insertions, 14 deletions
diff --git a/quantum/led_matrix.c b/quantum/led_matrix.c
index 942e6d7dc..e7e933e1d 100644
--- a/quantum/led_matrix.c
+++ b/quantum/led_matrix.c
@@ -33,6 +33,14 @@ const led_point_t k_led_matrix_center = {112, 32};
33const led_point_t k_led_matrix_center = LED_MATRIX_CENTER; 33const led_point_t k_led_matrix_center = LED_MATRIX_CENTER;
34#endif 34#endif
35 35
36// clang-format off
37#ifndef LED_MATRIX_IMMEDIATE_EEPROM
38# define led_eeconfig_update(v) led_update_eeprom |= v
39#else
40# define led_eeconfig_update(v) if (v) eeconfig_update_led_matrix()
41#endif
42// clang-format on
43
36// Generic effect runners 44// Generic effect runners
37#include "led_matrix_runners/effect_runner_dx_dy_dist.h" 45#include "led_matrix_runners/effect_runner_dx_dy_dist.h"
38#include "led_matrix_runners/effect_runner_dx_dy.h" 46#include "led_matrix_runners/effect_runner_dx_dy.h"
@@ -104,6 +112,7 @@ last_hit_t g_last_hit_tracker;
104 112
105// internals 113// internals
106static bool suspend_state = false; 114static bool suspend_state = false;
115static bool led_update_eeprom = false;
107static uint8_t led_last_enable = UINT8_MAX; 116static uint8_t led_last_enable = UINT8_MAX;
108static uint8_t led_last_effect = UINT8_MAX; 117static uint8_t led_last_effect = UINT8_MAX;
109static effect_params_t led_effect_params = {0, LED_FLAG_ALL, false}; 118static effect_params_t led_effect_params = {0, LED_FLAG_ALL, false};
@@ -276,6 +285,7 @@ static void led_task_timers(void) {
276 285
277static void led_task_sync(void) { 286static void led_task_sync(void) {
278 // next task 287 // next task
288 if (led_update_eeprom) eeconfig_update_led_matrix();
279 if (sync_timer_elapsed32(g_led_timer) >= LED_MATRIX_LED_FLUSH_LIMIT) led_task_state = STARTING; 289 if (sync_timer_elapsed32(g_led_timer) >= LED_MATRIX_LED_FLUSH_LIMIT) led_task_state = STARTING;
280} 290}
281 291
@@ -465,9 +475,7 @@ bool led_matrix_get_suspend_state(void) { return suspend_state; }
465void led_matrix_toggle_eeprom_helper(bool write_to_eeprom) { 475void led_matrix_toggle_eeprom_helper(bool write_to_eeprom) {
466 led_matrix_eeconfig.enable ^= 1; 476 led_matrix_eeconfig.enable ^= 1;
467 led_task_state = STARTING; 477 led_task_state = STARTING;
468 if (write_to_eeprom) { 478 led_eeconfig_update(write_to_eeprom);
469 eeconfig_update_led_matrix();
470 }
471 dprintf("led matrix toggle [%s]: led_matrix_eeconfig.enable = %u\n", (write_to_eeprom) ? "EEPROM" : "NOEEPROM", led_matrix_eeconfig.enable); 479 dprintf("led matrix toggle [%s]: led_matrix_eeconfig.enable = %u\n", (write_to_eeprom) ? "EEPROM" : "NOEEPROM", led_matrix_eeconfig.enable);
472} 480}
473void led_matrix_toggle_noeeprom(void) { led_matrix_toggle_eeprom_helper(false); } 481void led_matrix_toggle_noeeprom(void) { led_matrix_toggle_eeprom_helper(false); }
@@ -475,7 +483,7 @@ void led_matrix_toggle(void) { led_matrix_toggle_eeprom_helper(true); }
475 483
476void led_matrix_enable(void) { 484void led_matrix_enable(void) {
477 led_matrix_enable_noeeprom(); 485 led_matrix_enable_noeeprom();
478 eeconfig_update_led_matrix(); 486 led_eeconfig_update(true);
479} 487}
480 488
481void led_matrix_enable_noeeprom(void) { 489void led_matrix_enable_noeeprom(void) {
@@ -485,7 +493,7 @@ void led_matrix_enable_noeeprom(void) {
485 493
486void led_matrix_disable(void) { 494void led_matrix_disable(void) {
487 led_matrix_disable_noeeprom(); 495 led_matrix_disable_noeeprom();
488 eeconfig_update_led_matrix(); 496 led_eeconfig_update(true);
489} 497}
490 498
491void led_matrix_disable_noeeprom(void) { 499void led_matrix_disable_noeeprom(void) {
@@ -507,9 +515,7 @@ void led_matrix_mode_eeprom_helper(uint8_t mode, bool write_to_eeprom) {
507 led_matrix_eeconfig.mode = mode; 515 led_matrix_eeconfig.mode = mode;
508 } 516 }
509 led_task_state = STARTING; 517 led_task_state = STARTING;
510 if (write_to_eeprom) { 518 led_eeconfig_update(write_to_eeprom);
511 eeconfig_update_led_matrix();
512 }
513 dprintf("led matrix mode [%s]: %u\n", (write_to_eeprom) ? "EEPROM" : "NOEEPROM", led_matrix_eeconfig.mode); 519 dprintf("led matrix mode [%s]: %u\n", (write_to_eeprom) ? "EEPROM" : "NOEEPROM", led_matrix_eeconfig.mode);
514} 520}
515void led_matrix_mode_noeeprom(uint8_t mode) { led_matrix_mode_eeprom_helper(mode, false); } 521void led_matrix_mode_noeeprom(uint8_t mode) { led_matrix_mode_eeprom_helper(mode, false); }
@@ -536,9 +542,7 @@ void led_matrix_set_val_eeprom_helper(uint8_t val, bool write_to_eeprom) {
536 return; 542 return;
537 } 543 }
538 led_matrix_eeconfig.val = (val > LED_MATRIX_MAXIMUM_BRIGHTNESS) ? LED_MATRIX_MAXIMUM_BRIGHTNESS : val; 544 led_matrix_eeconfig.val = (val > LED_MATRIX_MAXIMUM_BRIGHTNESS) ? LED_MATRIX_MAXIMUM_BRIGHTNESS : val;
539 if (write_to_eeprom) { 545 led_eeconfig_update(write_to_eeprom);
540 eeconfig_update_led_matrix();
541 }
542 dprintf("led matrix set val [%s]: %u\n", (write_to_eeprom) ? "EEPROM" : "NOEEPROM", led_matrix_eeconfig.val); 546 dprintf("led matrix set val [%s]: %u\n", (write_to_eeprom) ? "EEPROM" : "NOEEPROM", led_matrix_eeconfig.val);
543} 547}
544void led_matrix_set_val_noeeprom(uint8_t val) { led_matrix_set_val_eeprom_helper(val, false); } 548void led_matrix_set_val_noeeprom(uint8_t val) { led_matrix_set_val_eeprom_helper(val, false); }
@@ -556,9 +560,7 @@ void led_matrix_decrease_val(void) { led_matrix_decrease_val_helper(true); }
556 560
557void led_matrix_set_speed_eeprom_helper(uint8_t speed, bool write_to_eeprom) { 561void led_matrix_set_speed_eeprom_helper(uint8_t speed, bool write_to_eeprom) {
558 led_matrix_eeconfig.speed = speed; 562 led_matrix_eeconfig.speed = speed;
559 if (write_to_eeprom) { 563 led_eeconfig_update(write_to_eeprom);
560 eeconfig_update_led_matrix();
561 }
562 dprintf("led matrix set speed [%s]: %u\n", (write_to_eeprom) ? "EEPROM" : "NOEEPROM", led_matrix_eeconfig.speed); 564 dprintf("led matrix set speed [%s]: %u\n", (write_to_eeprom) ? "EEPROM" : "NOEEPROM", led_matrix_eeconfig.speed);
563} 565}
564void led_matrix_set_speed_noeeprom(uint8_t speed) { led_matrix_set_speed_eeprom_helper(speed, false); } 566void led_matrix_set_speed_noeeprom(uint8_t speed) { led_matrix_set_speed_eeprom_helper(speed, false); }