Documented no support of SAMD51

Author: ArminJo

.github/workflows/LibraryBuild.yml

@@ -51,6 +51,7 @@ jobs:
           - arduino:avr:uno|USE_NO_SEND_PWM
           - arduino:avr:uno|SEND_PWM_BY_TIMER
           - arduino:avr:uno|USE_OPEN_DRAIN_OUTPUT_FOR_SEND_PIN
+          - arduino:avr:mega:cpu=atmega2560
           - arduino:avr:leonardo
           - arduino:megaavr:nona4809:mode=off
           - arduino:samd:arduino_zero_native
@@ -105,6 +106,11 @@ jobs:
               IRremoteExtensionTest: -DRAW_BUFFER_LENGTH=100 -DIR_SEND_PIN=3 -DUSE_OPEN_DRAIN_OUTPUT_FOR_SEND_PIN
               All: -DUSE_OPEN_DRAIN_OUTPUT_FOR_SEND_PIN
 
+          - arduino-boards-fqbn: arduino:avr:mega:cpu=atmega2560
+            sketches-exclude: IR2Keyboard
+            build-properties: # the flags were put in compiler.cpp.extra_flags
+              IRremoteExtensionTest: -DRAW_BUFFER_LENGTH=700
+
           - arduino-boards-fqbn: arduino:avr:leonardo
             build-properties: # the flags were put in compiler.cpp.extra_flags
               IRremoteExtensionTest: -DRAW_BUFFER_LENGTH=100

README.md

@@ -436,7 +436,7 @@ ATtiny and Digispark boards are only tested with the recommended [ATTinyCore](ht
 - ATmega4809 (Nano every)
 - ATtiny3217 (Tiny Core 32 Dev Board)
 - ATtiny84, 85, 167 (Digispark + Digispark Pro)
-- SAMD (Zero, MKR*, **but not DUE, which is SAM architecture**)
+- SAMD21 (Zero, MKR*, **but not SAMD51 and not DUE, the latter is SAM architecture**)
 - ESP32 (ESP32 C3 since board package 2.0.2 from Espressif)
 - ESP8266 [This fork](https://github.com/crankyoldgit/IRremoteESP8266) supports an [impressive set of protocols and a lot of air conditioners](https://github.com/crankyoldgit/IRremoteESP8266/blob/master/SupportedProtocols.md)
 - Sparkfun Pro Micro

src/IRReceive.hpp

@@ -869,6 +869,10 @@ void CheckForRecordGapsMicros(Print *aSerial, IRData *aIRDataPtr) {
  * Print functions
  * Since a library should not allocate the "Serial" object, all functions require a pointer to a Print object.
  **********************************************************************************************************************/
+void IRrecv::printActiveIRProtocols(Print *aSerial) {
+    // call no class function with same name
+    ::printActiveIRProtocols(aSerial);
+}
 void printActiveIRProtocols(Print *aSerial) {
 #if defined(DECODE_NEC)
     aSerial->print(F("NEC, "));
@@ -934,6 +938,7 @@ void printActiveIRProtocols(Print *aSerial) {
  * @param aSerial The Print object on which to write, for Arduino you can use &Serial.
  */
 void IRrecv::printIRResultShort(Print *aSerial) {
+    // call no class function with same name
     ::printIRResultShort(aSerial, &decodedIRData, true);
 }
 
@@ -1030,6 +1035,7 @@ void printIRResultShort(Print *aSerial, IRData *aIRDataPtr, bool aPrintGap) {
  * @param aSerial The Print object on which to write, for Arduino you can use &Serial.
  */
 void IRrecv::printIRSendUsage(Print *aSerial) {
+    // call no class function with same name
     ::printIRSendUsage(aSerial, &decodedIRData);
 }
 
@@ -1339,6 +1345,11 @@ void IRrecv::printIRResultAsCVariables(Print *aSerial) {
     }
 }
 
+const __FlashStringHelper* IRrecv::getProtocolString() {
+    // call no class function with same name
+    return ::getProtocolString(decodedIRData.protocol);
+}
+
 const __FlashStringHelper* getProtocolString(decode_type_t aProtocol) {
     switch (aProtocol) {
     default:

src/IRremoteInt.h

@@ -218,7 +218,13 @@ class IRrecv {
     void printIRResultMinimal(Print *aSerial);
     void printIRResultRawFormatted(Print *aSerial, bool aOutputMicrosecondsInsteadOfTicks = true);
     void printIRResultAsCVariables(Print *aSerial);
+
+    /*
+     * Next 3 functions are also available as non member functions
+     */
     void printIRSendUsage(Print *aSerial);
+    void printActiveIRProtocols(Print *aSerial);
+    const __FlashStringHelper* getProtocolString();
 
     void compensateAndPrintIRResultAsCArray(Print *aSerial, bool aOutputMicrosecondsInsteadOfTicks = true);
     void compensateAndPrintIRResultAsPronto(Print *aSerial, unsigned int frequency = 38000U);
@@ -322,6 +328,9 @@ bool MATCH_MARK(unsigned int measured_ticks, unsigned int desired_us);
 bool MATCH_SPACE(unsigned int measured_ticks, unsigned int desired_us);
 
 int getMarkExcessMicros();
+/*
+ * Next 3 functions are also available as member functions
+ */
 void printActiveIRProtocols(Print *aSerial);
 void printIRResultShort(Print *aSerial, IRData *aIRDataPtr, bool aPrintGap);
 void printIRSendUsage(Print *aSerial, IRData *aIRDataPtr);

src/TinyIRReceiver.hpp

@@ -56,8 +56,17 @@
  * @{
  */
 
-//#define DEBUG // to see if attachInterrupt used
-//#define TRACE // to see the state of the ISR state machine
+#if defined(DEBUG)
+#define LOCAL_DEBUG_ATTACH_INTERRUPT
+#else
+//#define LOCAL_DEBUG_ATTACH_INTERRUPT  // to see if attachInterrupt() or static interrupt (by register tweaking) is used
+#endif
+#if defined(TRACE)
+#define LOCAL_TRACE_STATE_MACHINE
+#else
+//#define LOCAL_TRACE_STATE_MACHINE  // to see the state of the ISR state machine
+#endif
+
 //#define _IR_MEASURE_TIMING        // Activate this if you want to enable internal hardware timing measurement.
 //#define _IR_TIMING_TEST_PIN 7
 TinyIRReceiverStruct TinyIRReceiverControl;
@@ -136,14 +145,13 @@ void IRPinChangeInterruptHandler(void)
 
     uint8_t tState = TinyIRReceiverControl.IRReceiverState;
 
-#if defined(TRACE)
+#if defined(LOCAL_TRACE_STATE_MACHINE)
     Serial.print(tState);
-    Serial.print(' ');
+    Serial.print(F(" D="));
+    Serial.print(tMicrosOfMarkOrSpace);
 //    Serial.print(F(" I="));
 //    Serial.print(tIRLevel);
-//    Serial.print(F(" D="));
-//    Serial.print(tDeltaMicros);
-//    Serial.println();
+    Serial.print('|');
 #endif
 
     if (tIRLevel == LOW) {
@@ -294,7 +302,7 @@ void initPCIInterruptForTinyReceiver() {
     enablePCIInterruptForTinyReceiver();
 }
 
-#if defined (DEBUG) && !defined(STR)
+#if defined (LOCAL_DEBUG_ATTACH_INTERRUPT) && !defined(STR)
 // Helper macro for getting a macro definition as string
 #define STR_HELPER(x) #x
 #define STR(x) STR_HELPER(x)
@@ -313,11 +321,11 @@ void enablePCIInterruptForTinyReceiver() {
 #elif !defined(__AVR__) || defined(TINY_RECEIVER_USE_ARDUINO_ATTACH_INTERRUPT)
     // costs 112 bytes program memory + 4 bytes RAM
     attachInterrupt(digitalPinToInterrupt(IR_INPUT_PIN), IRPinChangeInterruptHandler, CHANGE);
-#  if defined(DEBUG)
+#  if defined(LOCAL_DEBUG_ATTACH_INTERRUPT)
     Serial.println(F("Use attachInterrupt for pin=" STR(IR_INPUT_PIN)));
 #  endif
 #else
-#  if defined(DEBUG)
+#  if defined(LOCAL_DEBUG_ATTACH_INTERRUPT)
     Serial.println(F("Use static interrupt for pin=" STR(IR_INPUT_PIN)));
 #  endif
 #  if defined(__AVR_ATtiny25__) || defined(__AVR_ATtiny45__) || defined(__AVR_ATtiny85__)
@@ -527,4 +535,10 @@ ISR(PCINT1_vect)
 
 /** @}*/
 
+#if defined(LOCAL_DEBUG_ATTACH_INTERRUPT)
+#undef LOCAL_DEBUG_ATTACH_INTERRUPT
+#endif
+#if defined(LOCAL_TRACE_STATE_MACHINE)
+#undef LOCAL_TRACE_STATE_MACHINE
+#endif
 #endif // _TINY_IR_RECEIVER_HPP