Version 2.0.6 : fix tryToSend interrupt disabling (thanks to Fergus D…

…uncan)
pierremolinaro · Oct 10, 2020 · 50a03e1 · 50a03e1
1 parent 2a04087
commit 50a03e1
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/extras/acan2515.pdf b/extras/acan2515.pdf
diff --git a/library.properties b/library.properties
@@ -1,5 +1,5 @@
 name=ACAN2515
-version=2.0.5
+version=2.0.6
 author=Pierre Molinaro
 maintainer=Pierre Molinaro <[email protected]>
 sentence=Driver for MCP2515 CAN Controller

diff --git a/src/ACAN2515.cpp b/src/ACAN2515.cpp
@@ -868,14 +868,13 @@ uint8_t ACAN2515::receiveErrorCounter (void) {
 //----------------------------------------------------------------------------------------------------------------------
 
 bool ACAN2515::tryToSend (const CANMessage & inMessage) {
-//--- Find send buffer index
+//--- Fix send buffer index
   uint8_t idx = inMessage.idx ;
   if (idx > 2) {
     idx = 0 ;
   }
-//--- Workaround: the Teensy 3.5 / 3.6 "SPI.usingInterrupt" bug
-//    https://github.com/PaulStoffregen/SPI/issues/35
-  #if (defined (__MK64FX512__) || defined (__MK66FX1M0__))
+//--- Bug fix in 2.0.6 (thanks to Fergus Duncan): interrupts were only disabled for Teensy boards
+  #ifndef ARDUINO_ARCH_ESP32
     noInterrupts () ;
   #endif
    //---
@@ -888,7 +887,7 @@ bool ACAN2515::tryToSend (const CANMessage & inMessage) {
         ok = mTransmitBuffer [idx].append (inMessage) ;
       }
     mSPI.endTransaction () ;
-  #if (defined (__MK64FX512__) || defined (__MK66FX1M0__))
+  #ifndef ARDUINO_ARCH_ESP32
     interrupts () ;
   #endif
   return ok ;