led_strip: fix stability for WS2812B

UncleRus · Apr 30, 2021 · 65a2433 · 65a2433
1 parent 933114d
commit 65a2433
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -25,6 +25,7 @@
 - (led_strip) White component for RGBW strips now calculated automatically
 - (led_strip) Fixed bug in led_strip_set_pixels()
 - (led_strip) Added support for global brightness for ESP-IDF >= v4.4
+- (led_strip) Improved stability for WS2812B
 - (refactoring) Updated component makefiles and CMakeLists
 - (i2cdev) Added option to disable all mutexes
 

diff --git a/components/led_strip/led_strip.c b/components/led_strip/led_strip.c
@@ -64,10 +64,10 @@ static const char *TAG = "led_strip";
 
 #define LED_STRIP_RMT_CLK_DIV 2
 
-#define WS2812_T0H_NS   350
+#define WS2812_T0H_NS   400
 #define WS2812_T0L_NS   1000
 #define WS2812_T1H_NS   1000
-#define WS2812_T1L_NS   350
+#define WS2812_T1L_NS   400
 
 #define SK6812_T0H_NS   300
 #define SK6812_T0L_NS   900
@@ -113,7 +113,7 @@ static void IRAM_ATTR _rmt_adapter(const void *src, rmt_item32_t *dest, size_t s
     while (size < src_size && num < wanted_num)
     {
 #ifdef LED_STIRP_BRIGNTNESS
-        uint8_t b = scale8_video(*psrc, brightness);
+        uint8_t b = brightness != 255 ? scale8_video(*psrc, brightness) : *psrc;
 #else
         uint8_t b = *psrc;
 #endif