Merge pull request #412 from fabik111/fix-critical-section-for-opta

sebromero · web-flow · commit 9dfa52b51511 · 2025-10-29T11:30:54.000+01:00
Enter critical section only for Arduino Nano 33 BLE
diff --git a/src/utility/HCICordioTransport.cpp b/src/utility/HCICordioTransport.cpp
@@ -311,7 +311,9 @@ size_t HCICordioTransportClass::write(const uint8_t* data, size_t length)
 void HCICordioTransportClass::handleRxData(uint8_t* data, uint8_t len)
 {
   {
+    #if defined(ARDUINO_ARCH_NRF52840)
     mbed::CriticalSectionLock critical_section;
+    #endif
     if (_rxBuf.availableForStore() < len) {
       // drop!
       return;
@@ -321,7 +323,6 @@ void HCICordioTransportClass::handleRxData(uint8_t* data, uint8_t len)
       _rxBuf.store_char(data[i]);
     }
   }
-
   bleEventFlags.set(0x01);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -311,7 +311,9 @@ size_t HCICordioTransportClass::write(const uint8_t* data, size_t length)`
`311`	`311`	`void HCICordioTransportClass::handleRxData(uint8_t* data, uint8_t len)`
`312`	`312`	`{`
`313`	`313`	`{`
	`314`	`+ #if defined(ARDUINO_ARCH_NRF52840)`
`314`	`315`	`mbed::CriticalSectionLock critical_section;`
	`316`	`+ #endif`
`315`	`317`	`if (_rxBuf.availableForStore() < len) {`
`316`	`318`	`// drop!`
`317`	`319`	`return;`
`@@ -321,7 +323,6 @@ void HCICordioTransportClass::handleRxData(uint8_t* data, uint8_t len)`
`321`	`323`	`_rxBuf.store_char(data[i]);`
`322`	`324`	`}`
`323`	`325`	`}`
`324`		`-`
`325`	`326`	`bleEventFlags.set(0x01);`
`326`	`327`	`}`
`327`	`328`