Only suspend I2C related polling, but not the whole per10ms()

Author: richardclli

radio/src/boards/rm-h750/bsp_io.cpp

@@ -38,7 +38,7 @@
 #include "debug.h"
 
 extern const stm32_switch_t* boardGetSwitchDef(uint8_t idx);
-extern bool suspendTimerTask;
+extern bool suspendI2CTasks;
 
 struct bsp_io_expander {
     pca95xx_t exp;
@@ -110,7 +110,7 @@ static void _poll_switches(void *param1, uint32_t trigger_source)
   }
 
   // Suspend hardware reads when required
-  if (suspendTimerTask) return;
+  if (suspendI2CTasks) return;
 
   _read_io_expander(&_io_switches);
   _read_io_expander(&_io_fs_switches);

radio/src/edgetx.cpp

@@ -169,7 +169,7 @@ void checkValidMCU(void)
 static bool evalFSok = false;
 #endif
 
-bool suspendTimerTask = false;
+bool suspendI2CTasks = false;
 
 void per10ms()
 {
@@ -178,8 +178,6 @@ void per10ms()
 
   g_tmr10ms++;
 
-  if (suspendTimerTask) return;
-
 #if defined(GUI)
   if (lightOffCounter) lightOffCounter--;
   if (flashCounter) flashCounter--;
@@ -1093,7 +1091,7 @@ void edgeTxClose(uint8_t shutdown)
   TRACE("edgeTxClose");
 
   watchdogSuspend(2000/*20s*/);
-  suspendTimerTask = true;
+  suspendI2CTasks = true;
 
   if (shutdown) {
     pulsesStop();
@@ -1145,7 +1143,7 @@ void edgeTxResume()
 {
   TRACE("edgeTxResume");
 
-  suspendTimerTask = false;
+  suspendI2CTasks = false;
   if (!sdMounted()) sdInit();
 
   luaInitMainState();

radio/src/targets/common/arm/stm32/csd203_sensor.cpp

@@ -171,6 +171,8 @@ typedef struct {
 } CSD_ALERT;
 /*CSD ALERT Configuration struct*/
 
+extern bool suspendI2CTasks;
+
 void CSD203_Init(CSD_CONFIG *CSD203_CFG);            // Initialise CSD203 config
 uint16_t CSD203_ReadVbus(CSD_CONFIG *CSD203_CFG);    // Read Vbus Voltage
 uint16_t CSD203_ReadRshunt(CSD_CONFIG *CSD203_CFG);  // Read Rshunt
@@ -410,6 +412,8 @@ void readCSD203(void)
 {  // 5ms
   static uint16_t GetSenSorStep = 0;
 
+  if (suspendI2CTasks) return;
+
   if (IICReadStatusFlag == true) return;
 
   IICReadStatusFlag = true;

radio/src/targets/pa01/key_driver.cpp

@@ -68,6 +68,8 @@ enum PhysicalKeys
   ENT  = 16
 };
 
+extern bool suspendI2CTasks;
+
 static bool fct_state[4] = {false, false, false, false};
 static uint32_t keyState = 0;
 #if !defined(BOOT)
@@ -86,6 +88,9 @@ void pollKeys()
   }
   nonReadCount = 0;
 #endif
+
+  if (suspendI2CTasks) return;
+  
   // This function avoids concurrent matrix agitation
 
   uint32_t result = 0;

radio/src/targets/st16/key_driver.cpp

@@ -58,9 +58,14 @@ enum PhysicalTrims {
   TR4R
 };
 
+extern bool suspendI2CTasks;
+
 uint32_t readTrims()
 {
   uint32_t result = 0;
+
+  if (suspendI2CTasks) return result;
+
   uint16_t keys = bsp_input_get();
 
 #define _TRIM(t) \

radio/src/targets/taranis/gx12/bsp_io.cpp

@@ -36,7 +36,7 @@
 #define IO_RESET_GPIO GPIO_PIN(GPIOE, 15)
 
 extern const stm32_switch_t* boardGetSwitchDef(uint8_t idx);
-extern bool suspendTimerTask;
+extern bool suspendI2CTasks;
 
 struct bsp_io_expander {
   pca95xx_t exp;
@@ -86,7 +86,7 @@ static void _poll_switches(void *param1, uint32_t trigger_source)
   }
 
   // Suspend hardware reads when required
-  if (suspendTimerTask) return;
+  if (suspendI2CTasks) return;
 
   _read_io_expander(&_io_switches);
   _read_io_expander(&_io_fs_switches);