Revert "Revert "System instable""

This reverts commit 1d9ef7e634.

code/components/jomjol_controlGPIO/Color.cpp

@@ -89,6 +89,15 @@ Rgb& Rgb::blend( Rgb in ) {
     return *this;
 }
 
+uint8_t IRAM_ATTR Rgb::getGrb( int idx ) {
+    switch ( idx ) {
+        case 0: return g;
+        case 1: return r;
+        case 2: return b;
+    }
+    __builtin_unreachable();
+}
+
 Hsv::Hsv( Rgb r ) {
     int min = std::min( r.r, std::min( r.g, r.b ) );
     int max = std::max( r.r, std::max( r.g, r.b ) );

code/components/jomjol_controlGPIO/Color.h

@@ -19,21 +19,13 @@ union Rgb {
     bool operator==( Rgb in ) const { return in.value == value; }
     Rgb& blend( Rgb in );
     void swap( Rgb& o ) {  value = o.value; }
-    constexpr Rgb(const Rgb& other) : r(other.r), g(other.g), b(other.b), a(other.a) { }
     void linearize() {
         r = channelGamma(r);
         g = channelGamma(g);
         b = channelGamma(b);
     }
 
-    uint8_t IRAM_ATTR getGrb( int idx ) {
-	switch ( idx ) {
-	case 0: return g;
-	case 1: return r;
-	case 2: return b;
-	}
-	__builtin_unreachable();
-    }
+    uint8_t IRAM_ATTR getGrb( int idx );
 
     void stretchChannels( uint8_t maxR, uint8_t maxG, uint8_t maxB ) {
         r = stretch( r, maxR );
@@ -62,7 +54,6 @@ private:
     }
 };
 
-
 union Hsv {
     struct __attribute__ ((packed)) {
         uint8_t h, s, v, a;
@@ -75,5 +66,4 @@ union Hsv {
     Hsv& operator=( Rgb rgb );
     bool operator==( Hsv in ) const { return in.value == value; }
     void swap( Hsv& o ) { value = o.value; }
-    constexpr Hsv(const Hsv& other) : h(other.h), s(other.s), v(other.v), a(other.a) { }
 };

code/components/jomjol_controlGPIO/SmartLeds.cpp

@@ -16,16 +16,16 @@ void IRAM_ATTR SmartLed::interruptHandler(void*) {
         if ( RMT.int_st.val & (1 << (24 + channel ) ) ) { // tx_thr_event
             if ( self )
                 self->copyRmtHalfBlock();
-            RMT.int_clr.val = RMT.int_clr.val | 1 << ( 24 + channel );
+            RMT.int_clr.val |= 1 << ( 24 + channel );
         } else if ( RMT.int_st.val & ( 1 << (3 * channel ) ) ) { // tx_end
             if ( self )
                 xSemaphoreGiveFromISR( self->_finishedFlag, nullptr );
-            RMT.int_clr.val = RMT.int_clr.val | 1 << ( 3 * channel );
+            RMT.int_clr.val |= 1 << ( 3 * channel );
         }
     }
 }
 
-void SmartLed::copyRmtHalfBlock() {
+void IRAM_ATTR SmartLed::copyRmtHalfBlock() {
     int offset = detail::MAX_PULSES * _halfIdx;
     _halfIdx = !_halfIdx;
     int len = 3 - _componentPosition + 3 * ( _count - 1 );

code/components/jomjol_controlGPIO/SmartLeds.h

@@ -56,14 +56,10 @@
         #include <freertos/semphr.h>
         #include <soc/dport_reg.h>
         #include <soc/gpio_sig_map.h>
-	#include <soc/gpio_periph.h>
-	#include <esp32/rom/gpio.h>
         #include <soc/rmt_struct.h>
         #include <driver/spi_master.h>
-	#include <driver/rmt.h>
     }
     #include <stdio.h>
-    extern rmt_mem_t RMTMEM;
 #endif
 
 #include "Color.h"
@@ -135,7 +131,8 @@ public:
         initChannel( _channel );
 
         RMT.tx_lim_ch[ _channel ].limit = detail::MAX_PULSES;
-	RMT.int_ena.val = RMT.int_ena.val | (1 << ( 24 + _channel )) | (1 << ( 3 * _channel ));
+        RMT.int_ena.val |= 1 << ( 24 + _channel );
+        RMT.int_ena.val |= 1 << ( 3 * _channel );
 
         _bitToRmt[ 0 ].level0 = 1;
         _bitToRmt[ 0 ].level1 = 0;

code/components/jomjol_controlGPIO/server_GPIO.cpp

@@ -108,7 +108,7 @@ void GpioPin::init()
     gpio_config(&io_conf);
 
 //    if (_interruptType != GPIO_INTR_DISABLE) {                // ohne GPIO_PIN_MODE_EXTERNAL_FLASH_WS281X, wenn das genutzt wird, dann soll auch der Handler hier nicht initialisiert werden, da das dann über SmartLED erfolgt.
-    if ((_interruptType != GPIO_INTR_DISABLE) && (_mode != GPIO_PIN_MODE_EXTERNAL_FLASH_WS281X)) {
+    if ((_interruptType != GPIO_INTR_DISABLE) && (_interruptType != GPIO_PIN_MODE_EXTERNAL_FLASH_WS281X)) {
         //hook isr handler for specific gpio pin
         ESP_LOGD(TAG_SERVERGPIO, "GpioPin::init add isr handler for GPIO %d", _gpio);
         gpio_isr_handler_add(_gpio, gpio_isr_handler, (void*)&_gpio);

code/components/jomjol_controlGPIO/server_GPIO.h

@@ -5,8 +5,6 @@
 
 #include <esp_http_server.h>
 #include <map>
-#include "hal/gpio_ll.h"
-#include "rom/gpio.h"
 #include "driver/gpio.h"
 
 #include "SmartLeds.h"
@@ -111,4 +109,4 @@ GpioHandler* gpio_handler_get();
 
 
 
-#endif //SERVER_GPIO_H
+#endif //SERVER_GPIO_H