New example IR2Keyboard

Author: ArminJo

.github/workflows/LibraryBuild.yml

@@ -60,6 +60,9 @@ jobs:
         # With sketches-exclude you may exclude specific examples for a board. Use a comma separated list.
         #############################################################################################################
         include:
+          - arduino-boards-fqbn: arduino:avr:uno
+            sketches-exclude: IR2Keyboard
+
           - arduino-boards-fqbn: arduino:avr:uno|USE_NEC_STANDARD
             sketches-exclude: IR2Keyboard
             build-properties: # the flags were put in compiler.cpp.extra_flags
@@ -70,6 +73,9 @@ jobs:
             build-properties: # the flags were put in compiler.cpp.extra_flags
               All: -DUSE_NO_SEND_PWM
 
+          - arduino-boards-fqbn: arduino:megaavr:nona4809:mode=off
+            sketches-exclude: IR2Keyboard
+
           - arduino-boards-fqbn: esp32:esp32:featheresp32:FlashFreq=80
             platform-url: https://dl.espressif.com/dl/package_esp32_index.json
             sketches-exclude: IR2Keyboard,LGACSendDemo # undefined reference to `TwoWire::onReceive(void (*)(int))'

changelog.md

@@ -4,6 +4,8 @@
 - Changed wrong return code signature of decodePulseDistanceData() and its handling.
 - Removed Mitsubishi protocol, since the implementation is in contradiction with all documentation I could found and therefore supposed to be wrong.
 - Removed AIWA implementation, since it is only for 1 device and at least sending implemented wrong.
+- Added Lego_PF decode.
+- Added new example IR2Keyboard.
 
 ## 2.7.0 2020/09
 - Added ATmega328PB support.

examples/IR2Keyboard/IR2Keyboard.ino

@@ -3,7 +3,7 @@ version=2.7.0
 author=shirriff, z3t0 <[email protected]>
 maintainer=Armin Joachimsmeyer <[email protected]>
 sentence=Send and receive infrared signals with multiple protocols
-paragraph=Currently included protocols: Aiwa, BoseWave, Denon, Dish, JVC, Lego, LG, MagiQuest, NEC, Panasonic, RC5, RC6, Samsung, Sanyo, Sharp, Sony, Whynter, (Pronto).<br/><br/><b>New: </b>Major refactoring, new functions, repeat handling, NRF5 support.<br/>
+paragraph=Currently included protocols: BoseWave, Denon, Dish, JVC, Lego, LG, MagiQuest, NEC, Panasonic, RC5, RC6, Samsung, Sanyo, Sharp, Sony, Whynter, (Pronto).<br/><br/><b>New: </b>Major refactoring, new functions, repeat handling, NRF5 support.<br/>
 category=Communication
 url=https://github.com/z3t0/Arduino-IRremote
 architectures=avr,megaavr,samd,esp32,mbed

library.properties

@@ -34,7 +34,7 @@ struct irparams_struct irparams; // the irparams instance
 //   in a hope of finding out what is going on, but for now they will remain as
 //   functions even in non-DEBUG mode
 //
-int MATCH(int measured, int desired) {
+int MATCH(unsigned int measured, unsigned int desired) {
 #if DEBUG
     Serial.print(F("Testing: "));
     Serial.print(TICKS_LOW(desired), DEC);
@@ -57,7 +57,7 @@ int MATCH(int measured, int desired) {
 //+========================================================
 // Due to sensor lag, when received, Marks tend to be 100us too long
 //
-int MATCH_MARK(int measured_ticks, int desired_us) {
+int MATCH_MARK(uint16_t measured_ticks, unsigned int desired_us) {
 #if DEBUG
     Serial.print(F("Testing mark (actual vs desired): "));
     Serial.print(measured_ticks * MICROS_PER_TICK, DEC);
@@ -86,7 +86,7 @@ int MATCH_MARK(int measured_ticks, int desired_us) {
 //+========================================================
 // Due to sensor lag, when received, Spaces tend to be 100us too short
 //
-int MATCH_SPACE(int measured_ticks, int desired_us) {
+int MATCH_SPACE(uint16_t measured_ticks, unsigned int desired_us) {
 #if DEBUG
     Serial.print(F("Testing space (actual vs desired): "));
     Serial.print(measured_ticks * MICROS_PER_TICK, DEC);

src/IRremote.cpp

@@ -162,9 +162,9 @@ typedef enum {
 //------------------------------------------------------------------------------
 // Mark & Space matching functions
 //
-int MATCH(int measured, int desired);
-int MATCH_MARK(int measured_ticks, int desired_us);
-int MATCH_SPACE(int measured_ticks, int desired_us);
+int MATCH(unsigned int measured, unsigned int desired);
+int MATCH_MARK(uint16_t measured_ticks, unsigned int desired_us);
+int MATCH_SPACE(uint16_t measured_ticks, unsigned int desired_us);
 
 /****************************************************
  *                     RECEIVING
@@ -174,15 +174,15 @@ int MATCH_SPACE(int measured_ticks, int desired_us);
  */
 struct decode_results {
     decode_type_t decode_type;  ///< UNKNOWN, NEC, SONY, RC5, ...
-    unsigned int address;       ///< Used by Panasonic & Sharp6 NEC_standard [16-bits]
-    unsigned long value;        ///< Decoded value / command [max 32-bits]
-    int bits;                   ///< Number of bits in decoded value
-    unsigned int magnitude;     ///< Used by MagiQuest [16-bits]
+    uint16_t address;           ///< Used by Panasonic & Sharp6 NEC_standard [16-bits]
+    uint32_t value;             ///< Decoded value / command [max 32-bits]
+    uint16_t bits;              ///< Number of bits in decoded value
+    uint16_t magnitude;         ///< Used by MagiQuest [16-bits]
     bool isRepeat;              ///< True if repeat of value is detected
 
     // next 3 values are copies of irparams values - see IRremoteint.h
-    unsigned int *rawbuf;       ///< Raw intervals in 50uS ticks
-    unsigned int rawlen;        ///< Number of records in rawbuf
+    uint16_t *rawbuf;           ///< Raw intervals in 50uS ticks
+    uint16_t rawlen;            ///< Number of records in rawbuf
     bool overflow;              ///< true if IR raw code too long
 };
 

src/IRremote.h

@@ -132,7 +132,7 @@ static void dumpDuration(Stream& stream, uint16_t duration, uint16_t timebase) {
     dumpNumber(stream, (duration * MICROS_PER_TICK + timebase / 2) / timebase);
 }
 
-static void dumpSequence(Stream& stream, const volatile unsigned int *data, size_t length, uint16_t timebase) {
+static void dumpSequence(Stream& stream, const volatile uint16_t *data, size_t length, uint16_t timebase) {
     for (unsigned int i = 0; i < length; i++)
         dumpDuration(stream, data[i], timebase);
 

src/irPronto.cpp

@@ -48,9 +48,9 @@ struct irparams_struct {
     uint8_t recvpin;                ///< Pin connected to IR data from detector
     uint8_t blinkpin;
     uint8_t blinkflag;              ///< true -> enable blinking of pin on IR processing
-    unsigned int rawlen;            ///< counter of entries in rawbuf
-    unsigned int timer;             ///< State timer, counts 50uS ticks.
-    unsigned int rawbuf[RAW_BUFFER_LENGTH]; ///< raw data, first entry is the length of the gap between previous and current command
+    uint16_t rawlen;                ///< counter of entries in rawbuf
+    uint16_t timer;             ///< State timer, counts 50uS ticks.
+    uint16_t rawbuf[RAW_BUFFER_LENGTH]; ///< raw data, first entry is the length of the gap between previous and current command
     uint8_t overflow;               ///< Raw buffer overflow occurred
 };
 
@@ -101,11 +101,11 @@ extern struct irparams_struct irparams;
 //#define TICKS_LOW(us)   ((int)(((us)*LTOL/MICROS_PER_TICK)))
 //#define TICKS_HIGH(us)  ((int)(((us)*UTOL/MICROS_PER_TICK + 1)))
 #if MICROS_PER_TICK == 50 && TOLERANCE == 25           // Defaults
-#define TICKS_LOW(us)   ((int) ((us)/67 ))     // (us) / ((MICROS_PER_TICK:50 / LTOL:75 ) * 100)
-#define TICKS_HIGH(us)  ((int) ((us)/40 + 1))  // (us) / ((MICROS_PER_TICK:50 / UTOL:125) * 100) + 1
+#define TICKS_LOW(us)   ((us)/67 )     // (us) / ((MICROS_PER_TICK:50 / LTOL:75 ) * 100)
+#define TICKS_HIGH(us)  ((us)/40 + 1)  // (us) / ((MICROS_PER_TICK:50 / UTOL:125) * 100) + 1
 #else
-    #define TICKS_LOW(us)   ((int) ((long) (us) * LTOL / (MICROS_PER_TICK * 100) ))
-    #define TICKS_HIGH(us)  ((int) ((long) (us) * UTOL / (MICROS_PER_TICK * 100) + 1))
+    #define TICKS_LOW(us)   ((uint16_t) ((long) (us) * LTOL / (MICROS_PER_TICK * 100) ))
+    #define TICKS_HIGH(us)  ((uint16_t) ((long) (us) * UTOL / (MICROS_PER_TICK * 100) + 1))
 #endif
 
 //------------------------------------------------------------------------------