From 2b19de99a2c4acd8041a50593ff3a7586fe06a27 Mon Sep 17 00:00:00 2001 From: Selene ToyKeeper Date: Fri, 4 Jul 2025 04:57:34 -0600 Subject: made new settings apply to all button LEDs on lights bigger than 8K ROM This replaces "USE_CONFIGURABLE_RGB_VOLTAGE_LEVELS" with "USE_AUX_THRESHOLD_CONFIG", which controls the brightness of button LEDs while the main LEDs are on, and during post-off voltage display. Same basic concept, but works on single-color LEDs too, and lets the user finally configure POVD thresholds. The code for this is a bit messy, but the aux LED code as a whole is pretty messy since it wasn't designed for the things it does now. The entire thing needs a refactor or rewrite someday. But not today. For now, this is just enough to make the pull request cover more use cases before merging into trunk. I've tested it on a variety of lights, but am not yet entirely comfortable with it. However, it worked on at least these: - 1-color button LED, no RGB - front RGB, 1-color button LED - front RGB, hardwired also to RGB button - RGB button, no other aux These may need extra changes, and may have extra config options which do nothing... - front RGB, no button LED - 1-color front aux, no button LED - no aux at all - attiny85 lights (some could theoretically support the new options, but none even try) --- hw/wurkkos/fc13/anduril.h | 1 - hw/wurkkos/ts11/anduril.h | 1 - 2 files changed, 2 deletions(-) (limited to 'hw/wurkkos') diff --git a/hw/wurkkos/fc13/anduril.h b/hw/wurkkos/fc13/anduril.h index dd23d4a..db766c7 100644 --- a/hw/wurkkos/fc13/anduril.h +++ b/hw/wurkkos/fc13/anduril.h @@ -10,6 +10,5 @@ // turn on the aux LEDs while main LEDs are on #define USE_AUX_RGB_LEDS_WHILE_ON 20 -#define USE_CONFIGURABLE_RGB_VOLTAGE_LEVELS #define USE_INDICATOR_LED_WHILE_RAMPING diff --git a/hw/wurkkos/ts11/anduril.h b/hw/wurkkos/ts11/anduril.h index e329c38..169c848 100644 --- a/hw/wurkkos/ts11/anduril.h +++ b/hw/wurkkos/ts11/anduril.h @@ -12,6 +12,5 @@ // (but not until the main LEDs are bright enough to overpower the aux) // (setting this lower makes an annoying effect on some levels) #define USE_AUX_RGB_LEDS_WHILE_ON 50 -#define USE_CONFIGURABLE_RGB_VOLTAGE_LEVELS #define USE_INDICATOR_LED_WHILE_RAMPING -- cgit v1.2.3