Fixed a race condition on ESP32 (thanks to Nick Kirkby), version 1.1.5

pierremolinaro · Jun 2, 2019 · 45fe4d1 · 45fe4d1
1 parent 26e5fc6
commit 45fe4d1
Show file tree

Hide file tree

Showing 15 changed files with 210 additions and 144 deletions.
diff --git a/examples/LoopBackDemoTeensy3x/LoopBackDemoTeensy3x.ino b/examples/LoopBackDemoTeensy3x/LoopBackDemoTeensy3x.ino
@@ -93,7 +93,7 @@ void setup () {
   }
 }
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 static uint32_t gBlinkLedDate = 0 ;
 static uint32_t gReceivedFrameCount = 0 ;

diff --git a/examples/LoopBackDemoTeensy3xExtendedFilterTest/LoopBackDemoTeensy3xExtendedFilterTest.ino b/examples/LoopBackDemoTeensy3xExtendedFilterTest/LoopBackDemoTeensy3xExtendedFilterTest.ino
@@ -69,7 +69,7 @@ void setup () {
   }
 }
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 static uint32_t gBlinkLedDate = 0 ;
 static uint32_t gPhase = 0 ;

diff --git a/examples/LoopBackDemoTeensy3xNoInt/LoopBackDemoTeensy3xNoInt.ino b/examples/LoopBackDemoTeensy3xNoInt/LoopBackDemoTeensy3xNoInt.ino
@@ -91,7 +91,7 @@ void setup () {
   }
 }
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 static uint32_t gBlinkLedDate = 0 ;
 static uint32_t gReceivedFrameCount = 0 ;

diff --git a/examples/LoopBackDemoTeensy3xStandardFilterTest/LoopBackDemoTeensy3xStandardFilterTest.ino b/examples/LoopBackDemoTeensy3xStandardFilterTest/LoopBackDemoTeensy3xStandardFilterTest.ino
@@ -69,7 +69,7 @@ void setup () {
   }
 }
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 static uint32_t gBlinkLedDate = 0 ;
 static uint32_t gPhase = 0 ;

diff --git a/examples/LoopBackDemoTeensy3xWithFilters/LoopBackDemoTeensy3xWithFilters.ino b/examples/LoopBackDemoTeensy3xWithFilters/LoopBackDemoTeensy3xWithFilters.ino
@@ -94,7 +94,7 @@ void setup () {
   }
 }
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 static uint32_t gBlinkLedDate = 0 ;
 static uint8_t gPhase = 0 ;

diff --git a/examples/LoopBackIntensiveTestTeensy3xUsingTxQ/LoopBackIntensiveTestTeensy3xUsingTxQ.ino b/examples/LoopBackIntensiveTestTeensy3xUsingTxQ/LoopBackIntensiveTestTeensy3xUsingTxQ.ino
@@ -96,7 +96,7 @@ void setup () {
   }
 }
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 static uint32_t gBlinkLedDate = 0 ;
 static uint32_t gFrameCount = 0 ;

diff --git a/extras/acan2517.pdf b/extras/acan2517.pdf
diff --git a/library.properties b/library.properties
@@ -1,5 +1,5 @@
 name=ACAN2517
-version=1.1.4
+version=1.1.5
 author=Pierre Molinaro
 maintainer=Pierre Molinaro <[email protected]>
 sentence=Driver for MCP2517FD CAN Controller (CAN 2.0B mode)

diff --git a/src/ACAN2517.cpp b/src/ACAN2517.cpp
diff --git a/src/ACAN2517.h b/src/ACAN2517.h
@@ -1,22 +1,22 @@
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 // A CAN driver for MCP2517FD, CAN 2.0B mode
 // by Pierre Molinaro
 // https://github.com/pierremolinaro/acan2517
 //
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 #pragma once
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 #include <ACAN2517Settings.h>
 #include <ACANBuffer.h>
 #include <ACAN2517Filters.h>
 #include <SPI.h>
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 //   ACAN2517 class
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 class ACAN2517 {
 
@@ -169,5 +169,5 @@ class ACAN2517 {
 
 } ;
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
diff --git a/src/ACAN2517Filters.h b/src/ACAN2517Filters.h
@@ -1,21 +1,21 @@
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 // An utility class for:
 //   - ACAN2517 CAN driver for MCP2517FD (CAN 2.0B mode)
 // by Pierre Molinaro
 // https://github.com/pierremolinaro/acan2517
 //
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 #ifndef ACAN2517_FILTERS_CLASS_DEFINED
 #define ACAN2517_FILTERS_CLASS_DEFINED
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 #include <CANMessage.h>
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 //  ACAN2517Filters class
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 class ACAN2517Filters {
 
@@ -233,6 +233,6 @@ class ACAN2517Filters {
 
 } ;
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 #endif
diff --git a/src/ACAN2517Settings.cpp b/src/ACAN2517Settings.cpp
@@ -1,15 +1,15 @@
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 // A CAN driver for MCP2517 (CAN 2.0B mode)
 // by Pierre Molinaro
 // https://github.com/pierremolinaro/acan2517
 //
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 #include <ACAN2517Settings.h>
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 //    sysClock
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 uint32_t ACAN2517Settings::sysClock (const Oscillator inOscillator) {
   uint32_t sysClock = 40UL * 1000 * 1000 ;
@@ -35,9 +35,9 @@ uint32_t ACAN2517Settings::sysClock (const Oscillator inOscillator) {
   return sysClock ;
 }
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 //   CONSTRUCTOR
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 ACAN2517Settings::ACAN2517Settings (const Oscillator inOscillator,
                                     const uint32_t inDesiredBitRate,
@@ -100,23 +100,23 @@ mOscillator (inOscillator) {
   mBitRateClosedToDesiredRate = (diff * ppm) <= (((uint64_t) W) * inTolerancePPM) ;
 } ;
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 //   ACCESSORS
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 uint32_t ACAN2517Settings::actualBitRate (void) const {
   const uint32_t TQCount = 1 /* Sync Seg */ + mPhaseSegment1 + mPhaseSegment2 ;
   return mSysClock / mBitRatePrescaler / TQCount ;
 }
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 bool ACAN2517Settings::exactBitRate (void) const {
   const uint32_t TQCount = 1 /* Sync Seg */ + mPhaseSegment1 + mPhaseSegment2 ;
   return mSysClock == (mBitRatePrescaler * mDesiredBitRate * TQCount) ;
 }
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 uint32_t ACAN2517Settings::ppmFromDesiredBitRate (void) const {
   const uint32_t TQCount = 1 /* Sync Seg */ + mPhaseSegment1 + mPhaseSegment2 ;
@@ -126,7 +126,7 @@ uint32_t ACAN2517Settings::ppmFromDesiredBitRate (void) const {
   return (uint32_t) ((diff * ppm) / W) ;
 }
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 uint32_t ACAN2517Settings::samplePointFromBitStart (void) const {
   const uint32_t TQCount = 1 /* Sync Seg */ + mPhaseSegment1 + mPhaseSegment2 ;
@@ -135,7 +135,7 @@ uint32_t ACAN2517Settings::samplePointFromBitStart (void) const {
   return (samplePoint * partPerCent) / TQCount ;
 }
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 uint32_t ACAN2517Settings::CANBitSettingConsistency (void) const {
   uint32_t errorCode = 0 ; // Means no error
@@ -168,7 +168,7 @@ uint32_t ACAN2517Settings::CANBitSettingConsistency (void) const {
   return errorCode ;
 }
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------
 
 uint32_t ACAN2517Settings::ramUsage (void) const {
   uint32_t result = 0 ;
@@ -182,5 +182,5 @@ uint32_t ACAN2517Settings::ramUsage (void) const {
   return result ;
 }
 
-//——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
+//----------------------------------------------------------------------------------------------------------------------