Jan/WLED_MM_Infinity
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

strip wait function: increase idle wait time to 120ms

Browse Source 
Updated strip wait function to use longer idle wait time (similar to the timeout used by upstream WLED 0.16)
This commit is contained in:
Frank
2025-11-18 23:13:47 +01:00
committed by GitHub
parent 872c38f3c2
commit bc7a96d7d4
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
wled00/FX_fcn.cpp
View File

@@ -56,13 +56,13 @@ static portMUX_TYPE s_wled_strip_mux = portMUX_INITIALIZER_UNLOCKED; // to prote
#error "Max segments must be at least max number of busses!" #error "Max segments must be at least max number of busses!"
#endif #endif
// WLEDMM experimental . this is a "C style" wrapper for strip.waitUntilIdle() // WLEDMM experimental . this is a "C style" wrapper for strip.()
// This workaround is just needed for the segment class, that does't know about "strip" // This workaround is just needed for the segment class, that does't know about "strip"
void strip_wait_until_idle(String whoCalledMe) { void strip_wait_until_idle(String whoCalledMe) {
#if defined(ARDUINO_ARCH_ESP32) && defined(WLEDMM_PROTECT_SERVICE) // WLEDMM experimental #if defined(ARDUINO_ARCH_ESP32) && defined(WLEDMM_PROTECT_SERVICE) // WLEDMM experimental
if (strip.isServicing() && (strncmp(pcTaskGetTaskName(NULL), "loopTask", 8) != 0)) { // if we are in looptask (arduino loop), its safe to proceed without waiting if (strip.isServicing() && (strncmp(pcTaskGetTaskName(NULL), "loopTask", 8) != 0)) { // if we are in looptask (arduino loop), its safe to proceed without waiting
USER_PRINTLN(whoCalledMe + String(": strip is still drawing effects.")); USER_PRINTLN(whoCalledMe + String(": strip is still drawing effects."));
strip.waitUntilIdle(); strip.();
} }
#endif #endif
} }
@@ -1893,7 +1893,8 @@ void WS2812FX::finalizeInit(void)
// WLEDMM wait until strip is idle (=not servicing). // WLEDMM wait until strip is idle (=not servicing).
// on 8266 this function does nothing, because we can only do "busy waiting" on ESP32 // on 8266 this function does nothing, because we can only do "busy waiting" on ESP32
#define MAX_IDLE_WAIT_MS 50 // seems to work in most cases //#define MAX_IDLE_WAIT_MS 50 // seems to work in most cases
#define MAX_IDLE_WAIT_MS 120 // better safe than sorry - similar to the timeout used by upstream WLED
void WS2812FX::waitUntilIdle(void) { void WS2812FX::waitUntilIdle(void) {
#if defined(ARDUINO_ARCH_ESP32) && defined(WLEDMM_PROTECT_SERVICE) #if defined(ARDUINO_ARCH_ESP32) && defined(WLEDMM_PROTECT_SERVICE)
if (isServicing()) { if (isServicing()) {