rfb refactoring

- RF_... -> RFB_...
- rework rcswitch integration, support variable length payload
- rework rfbridge parser (not tested)
- rework settings scanning routine when trying to match rf payload
  with the relay ID
- update build tests

code/espurna/board.cpp

@@ -84,8 +84,8 @@ PROGMEM const char espurna_modules[] =
     #if RFM69_SUPPORT
         "RFM69 "
     #endif
-    #if RF_SUPPORT
-        "RF "
+    #if RFB_SUPPORT
+        "RFB "
     #endif
     #if RPN_RULES_SUPPORT
         "RPN_RULES "

code/espurna/config/arduino.h

@@ -207,7 +207,7 @@
 //#define NTP_SUPPORT            0
 //#define OTA_ARDUINOOTA_SUPPORT 1
 //#define RFM69_SUPPORT          1
-//#define RF_SUPPORT             1
+//#define RFB_SUPPORT            1
 //#define RPN_RULES_SUPPORT      0
 //#define SCHEDULER_SUPPORT      0
 //#define SPIFFS_SUPPORT         1

code/espurna/config/dependencies.h

@@ -49,11 +49,6 @@
 #define MQTT_SUPPORT                1
 #endif
 
-#if RF_SUPPORT
-#undef RELAY_SUPPORT
-#define RELAY_SUPPORT               1
-#endif
-
 #if LED_SUPPORT
 #undef BROKER_SUPPORT
 #define BROKER_SUPPORT              1               // If LED is enabled enable BROKER to supply status changes

code/espurna/config/deprecated.h

@@ -108,3 +108,21 @@
 #warning "WIFI_FALLBACK_APMODE is deprecated! Please use WIFI_AP_MODE instead"
 #define WIFI_AP_MODE ((1 == WIFI_FALLBACK_APMODE) ? WiFiApMode::Fallback : WiFiApMode::Disabled)
 #endif
+
+#ifdef RFB_DIRECT
+#warning "RFB_DIRECT is deprecated! Please use RFB_PROVIDER=RFB_PROVIDER_..."
+#undef RFB_PROVIDER
+#if RFB_DIRECT
+#define RFB_PROVIDER RFB_PROVIDER_RCSWITCH
+#else
+#define RFB_PROVIDER RFB_PROVIDER_EFM8BB1
+#endif
+#endif
+
+// TODO: RF_... -> RFB_...
+
+#ifdef RF_SUPPORT
+#warning "RF_SUPPORT is deprecated! Please use RFB_SUPPORT"
+#undef RFB_SUPPORT
+#define RFB_SUPPORT RF_SUPPORT
+#endif

code/espurna/config/general.h

@@ -1621,40 +1621,26 @@
 
 
 // -----------------------------------------------------------------------------
-// MQTT RF BRIDGE
+// RF BRIDGE
 // -----------------------------------------------------------------------------
 
-#ifndef RF_SUPPORT
-#define RF_SUPPORT                  0
+#ifndef RFB_SUPPORT
+#define RFB_SUPPORT                  0
 #endif
 
-#ifndef RF_DEBOUNCE
-#define RF_DEBOUNCE                 500
+#ifndef RFB_SEND_TIMES
+#define RFB_SEND_TIMES               1               // How many times to send the message
 #endif
 
-#ifndef RF_LEARN_TIMEOUT
-#define RF_LEARN_TIMEOUT            60000
-#endif
-
-#ifndef RF_SEND_TIMES
-#define RF_SEND_TIMES               4               // How many times to send the message
-#endif
-
-#ifndef RF_SEND_DELAY
-#define RF_SEND_DELAY               500             // Interval between sendings in ms
-#endif
-
-#ifndef RF_RECEIVE_DELAY
-#define RF_RECEIVE_DELAY            500             // Interval between recieving in ms (avoid debouncing)
-#endif
-
-// Enable RCSwitch support
+// - RFB_PROVIDER_EFM8BB1
+// Default option for the ITEAD_SONOFF_RFBRIDGE or any custom firmware implementing the protocol
+// - RFB_PROVIDER_RCSWITCH
 // Originally implemented for SONOFF BASIC
 // https://tinkerman.cat/adding-rf-to-a-non-rf-itead-sonoff/
 // Also possible to use with SONOFF RF BRIDGE, thanks to @wildwiz
 // https://github.com/xoseperez/espurna/wiki/Hardware-Itead-Sonoff-RF-Bridge---Direct-Hack
-#ifndef RFB_DIRECT
-#define RFB_DIRECT                  0
+#ifndef RFB_PROVIDER
+#define RFB_PROVIDER                RFB_PROVIDER_RCSWITCH
 #endif
 
 #ifndef RFB_RX_PIN
@@ -1665,6 +1651,21 @@
 #define RFB_TX_PIN                  GPIO_NONE
 #endif
 
+#ifndef RFB_LEARN_TIMEOUT
+#define RFB_LEARN_TIMEOUT            15000
+#endif
+
+#ifndef RFB_SEND_DELAY
+#define RFB_SEND_DELAY               500             // Interval between sendings in ms
+#endif
+
+#ifndef RFB_RECEIVE_DELAY
+#define RFB_RECEIVE_DELAY            500             // Interval between recieving in ms (avoid bouncing)
+#endif
+
+#ifndef RFB_TRANSMIT_TIMES
+#define RFB_TRANSMIT_TIMES          5                // How many times RCSwitch will repeat the message
+#endif
 
 // -----------------------------------------------------------------------------
 // IR Bridge

code/espurna/config/hardware.h

@@ -789,16 +789,16 @@
     #define LED1_PIN            13
     #define LED1_PIN_INVERSE    1
 
-    #define RF_SUPPORT          1
+    #define RFB_SUPPORT          1
 
-    // Only used when RFB_DIRECT=1
+    // only used when RFB_PROVIDER is RCSWITCH
     #define RFB_RX_PIN          4
     #define RFB_TX_PIN          5
 
     // When using un-modified harware, ESPurna communicates with the secondary
     // MCU EFM8BB1 via UART at 19200 bps so we need to change the speed of
     // the port and remove UART noise on serial line
-    #if not RFB_DIRECT
+    #if RFB_PROVIDER == RFB_PROVIDER_EFM8BB1
     #define SERIAL_BAUDRATE         19200
     #define DEBUG_SERIAL_SUPPORT    0
     #endif
@@ -1373,9 +1373,9 @@
     #define LIGHT_CH4_PIN       13      // WHITE
 
     // RF
-    #define RF_SUPPORT          1
-    #define RFB_DIRECT          1
-    #define RFB_RX_PIN          4
+    #define RFB_SUPPORT          1
+    #define RFB_PROVIDER         RFB_PROVIDER_RCSWITCH
+    #define RFB_RX_PIN           4
 
 #elif defined(MAGICHOME_ZJ_WFMN_C_11)
 
@@ -2683,9 +2683,9 @@
     #endif
     #define DALLAS_PIN          2
 
-    #define RF_SUPPORT          1
-    #define RFB_DIRECT          1
-    #define RFB_RX_PIN          14
+    #define RFB_SUPPORT          1
+    #define RFB_PROVIDER         RFB_PROVIDER_RCSWITCH
+    #define RFB_RX_PIN           14
 
     #ifndef DIGITAL_SUPPORT
     #define DIGITAL_SUPPORT      1

code/espurna/config/types.h

@@ -103,6 +103,9 @@
 #define RELAY_PROVIDER_STM          4
 #define RELAY_PROVIDER_MCP23S08     5
 
+#define RFB_PROVIDER_RCSWITCH       0
+#define RFB_PROVIDER_EFM8BB1        1
+
 #define RELAY_GROUP_SYNC_NORMAL      0
 #define RELAY_GROUP_SYNC_INVERSE     1
 #define RELAY_GROUP_SYNC_RECEIVEONLY 2

code/espurna/config/webui.h

@@ -32,7 +32,7 @@
     #endif
 #endif
 
-#if RF_SUPPORT == 1
+#if RFB_SUPPORT == 1
     #ifndef WEBUI_IMAGE
         #define WEBUI_IMAGE    WEBUI_IMAGE_RFBRIDGE
     #else

code/espurna/main.cpp

@@ -244,7 +244,7 @@ void setup() {
     #if I2C_SUPPORT
         i2cSetup();
     #endif
-    #if RF_SUPPORT
+    #if RFB_SUPPORT
         rfbSetup();
     #endif
     #if ALEXA_SUPPORT

code/espurna/rfbridge.h

@@ -6,12 +6,18 @@ Copyright (C) 2016-2019 by Xose Pérez <xose dot perez at gmail dot com>
 
 */
 
+#pragma once
+
 #include "espurna.h"
 
-#if RF_SUPPORT
+#if RFB_SUPPORT
+
+#include "broker.h"
 
-#if RFB_DIRECT
-#include <RCSwitch.h>
+#if RFB_PROVIDER == RFB_PROVIDER_EFM8BB1
+BrokerDeclare(RfbridgeBroker, void(const char* code));
+#elif RFB_PROVIDER == RFB_PROVIDER_RCSWITCH
+BrokerDeclare(RfbridgeBroker, void(unsigned char protocol, const char* code));
 #endif
 
 void rfbStatus(unsigned char id, bool status);
@@ -23,4 +29,4 @@ void rfbStore(unsigned char id, bool status, const char * code);
 void rfbForget(unsigned char id, bool status);
 void rfbSetup();
 
-#endif // RF_SUPPORT == 1
+#endif // RFB_SUPPORT == 1

code/espurna/system.cpp

@@ -251,7 +251,7 @@ void _systemSetupSpecificHardware() {
 
     // These devices use the hardware UART
     // to communicate to secondary microcontrollers
-    #if (RF_SUPPORT && !RFB_DIRECT) || (RELAY_PROVIDER == RELAY_PROVIDER_DUAL) || (RELAY_PROVIDER == RELAY_PROVIDER_STM)
+    #if (RFB_SUPPORT && (RFB_PROVIDER == RFB_PROVIDER_EFM8BB1)) || (RELAY_PROVIDER == RELAY_PROVIDER_DUAL) || (RELAY_PROVIDER == RELAY_PROVIDER_STM)
         Serial.begin(SERIAL_BAUDRATE);
     #endif
 

code/espurna/utils.cpp

@@ -792,7 +792,7 @@ char* strnstr(const char* buffer, const char* token, size_t n) {
 }
 
 // From a byte array to an hexa char array ("A220EE...", double the size)
-size_t hexEncode(uint8_t * in, size_t in_size, char * out, size_t out_size) {
+size_t hexEncode(const uint8_t * in, size_t in_size, char * out, size_t out_size) {
     if ((2 * in_size + 1) > (out_size)) return 0;
 
     static const char base16[] = "0123456789ABCDEF";

code/espurna/utils.h

@@ -69,5 +69,5 @@ void nice_delay(unsigned long ms);
 
 double roundTo(double num, unsigned char positions);
 
-size_t hexEncode(uint8_t* in, size_t in_size, char* out, size_t out_size);
+size_t hexEncode(const uint8_t* in, size_t in_size, char* out, size_t out_size);
 size_t hexDecode(const char* in, size_t in_size, uint8_t* out, size_t out_size);

code/test/build/nondefault.h

@@ -5,11 +5,11 @@
 #define NETBIOS_SUPPORT 1
 #define NOFUSS_SUPPORT 1
 #define OTA_MQTT_SUPPORT 1
-#define RFB_DIRECT 1
 #define RFM69_SUPPORT 1
-#define RF_SUPPORT 1
 #define RPN_RULES_SUPPORT 1
 #define SSDP_SUPPORT 1
 #define UART_MQTT_SUPPORT 1
 #define TERMINAL_WEB_API_SUPPORT 1
 #define TERMINAL_MQTT_SUPPORT 1
+#define RFB_SUPPORT 1
+#define RFB_PROVIDER RFB_PROVIDER_RCSWITCH

code/test/build/rfbridge.h

@@ -1,4 +1,5 @@
-#define DUMMY_RELAY_COUNT       8
-#define RF_SUPPORT              1
-#define SERIAL_BAUDRATE         19200
-#define DEBUG_SERIAL_SUPPORT    0
+#define DUMMY_RELAY_COUNT 8
+#define SERIAL_BAUDRATE 19200
+#define DEBUG_SERIAL_SUPPORT 0
+#define RFB_SUPPORT 1
+#define RFB_PROVIDER RFB_PROVIDER_EFM8BB1