[BUG] ESP32 MCPWM remove unnecessary interrupt yields

[HamzaYslmn]

Describe the bug

In SimpleFOC 2.4.0, the ESP32 MCPWM current-sense callbacks return true in three places.

In ESP-IDF, the callback return value indicates whether a higher-priority task was woken and whether a context switch may be required after the ISR.

These callbacks only sample ADC data and update internal buffers; they do not appear to wake any FreeRTOS task. Therefore these returns should be false.

Proposed change

diff --git a/src/current_sense/hardware_specific/esp32/esp32_mcpwm_mcu.cpp b/src/current_sense/hardware_specific/esp32/esp32_mcpwm_mcu.cpp
--- a/src/current_sense/hardware_specific/esp32/esp32_mcpwm_mcu.cpp
+++ b/src/current_sense/hardware_specific/esp32/esp32_mcpwm_mcu.cpp
@@ -112,7 +112,7 @@
 #ifdef SIMPLEFOC_ESP32_USE_ADC_DEBUG
   gpio_set_level((gpio_num_t)debug_pin, 0);
 #endif
-  return true;
+  return false;
 }
 
@@ -120,7 +120,7 @@
   if(edata->direction != MCPWM_TIMER_DIRECTION_UP){
-    return true;
+    return false;
   }
 
@@ -140,7 +140,7 @@
 #ifdef SIMPLEFOC_ESP32_USE_ADC_DEBUG
   gpio_set_level((gpio_num_t)debug_pin, 0);
 #endif
-  return true;
+  return false;
 }

This should not change ADC sampling behavior; it only avoids reporting an unnecessary ISR task wakeup/yield.

Hardware

• ESP32-S3
• DRV8353
• 3-phase BLDC
• Sensorless FOC
• Low-side current sensing
• PlatformIO
• SimpleFOC 2.4.0

Sources

• [SimpleFOC v2.4.0](https://github.com/simplefoc/Arduino-FOC/releases/tag/v2.4.0)
• [SimpleFOC ESP32 MCPWM current-sense source](https://github.com/simplefoc/Arduino-FOC/blob/v2.4.0/src/current_sense/hardware_specific/esp32/esp32_mcpwm_mcu.cpp)
• [ESP-IDF ESP32-S3 MCPWM documentation](https://docs.espressif.com/projects/esp-idf/en/latest/esp32s3/api-reference/peripherals/mcpwm.html)

[HamzaYslmn]

@samguns @zjor @tschundler @geekuillaume

~3.4 us per interrupt, which is 172 ms of core 1 per second