Revert "fix(Examples): Fix RF Test RTOS Scheduling Bug (#1011)"

This reverts commit 323668c.

Author: EricB-ADI

Examples/MAX32655/Bluetooth/RF_Test/main.c

@@ -243,8 +243,8 @@ uint8_t processEscSequence(uint8_t *seq)
  *
  *  \brief  adds latest command to command history buffer
  *
- *  \param  q pointer to the circular buffer holding command history
- *  \param  cmd pointer to the command string to be added
+ *  \param  q pointer to the circular buffer holding command history 
+ *  \param  cmd pointer to the command string to be added 
  *
  *  \return None.
  */
@@ -271,9 +271,9 @@ void cmdHistoryAdd(queue_t *q, const uint8_t *cmd)
  *  \fn     updateQueuePointer.
  *
  *  \brief  Updates an internal marker pointing to historical command to be printed, based on up/down arrow
- *
- *  \param  q pointer to the circular buffer holding command history
- *
+ * 
+ *  \param  q pointer to the circular buffer holding command history 
+ * 
  *  \param  upArrow flag used to upated the queuePoniter delimiting which command to print
  *
  *  \return None.
@@ -393,9 +393,9 @@ void prompt(void)
  *  \fn     printHint
  *
  *  \brief  Prints the help string of any command matching the current inputbuffer
- *
+ * 
  *  \param  buff pointer to the inputbuffer
- *
+ * 
  *  \return None.
  */
 /*************************************************************************************************/
@@ -694,25 +694,17 @@ void txTestTask(void *pvParameters)
         snprintf(str, sizeof(str), "%s%s", str, (const char *)getPhyStr(phy));
         APP_TRACE_INFO1("%s", str);
 
-        //Prevent FreeRTOS from context switching until the LL is finished
-        vTaskSuspendAll();
-
         /* stat test */
         if (testConfig.testType == BLE_TX_TEST) {
             res = LlEnhancedTxTest(testConfig.channel, packetLen, packetType, phy, 0);
         } else {
             res = LlEnhancedRxTest(testConfig.channel, phy, 0, 0);
         }
-        xTaskResumeAll(); //Restore scheduler
-
         APP_TRACE_INFO2("result = %u %s", res, res == LL_SUCCESS ? "(SUCCESS)" : "(FAIL)");
         /* if duration value was given then let the test run that amount of time and end */
         if (testConfig.duration_ms) {
             vTaskDelay(testConfig.duration_ms);
-            //Prevent FreeRTOS from context switching until the LL is finished
-            vTaskSuspendAll();
             LlEndTest(NULL);
-            xTaskResumeAll(); //Restore scheduler
             xSemaphoreGive(rfTestMutex);
         }
         pausePrompt = false;

Examples/MAX32665/Bluetooth/RF_Test/main.c

@@ -246,8 +246,8 @@ uint8_t processEscSequence(uint8_t *seq)
  *
  *  \brief  adds latest command to command history buffer
  *
- *  \param  q pointer to the circular buffer holding command history
- *  \param  cmd pointer to the command string to be added
+ *  \param  q pointer to the circular buffer holding command history 
+ *  \param  cmd pointer to the command string to be added 
  *
  *  \return None.
  */
@@ -274,9 +274,9 @@ void cmdHistoryAdd(queue_t *q, const uint8_t *cmd)
  *  \fn     updateQueuePointer.
  *
  *  \brief  Updates an internal marker pointing to historical command to be printed, based on up/down arrow
- *
- *  \param  q pointer to the circular buffer holding command history
- *
+ * 
+ *  \param  q pointer to the circular buffer holding command history 
+ * 
  *  \param  upArrow flag used to upated the queuePoniter delimiting which command to print
  *
  *  \return None.
@@ -396,9 +396,9 @@ void prompt(void)
  *  \fn     printHint
  *
  *  \brief  Prints the help string of any command matching the current inputbuffer
- *
+ * 
  *  \param  buff pointer to the inputbuffer
- *
+ * 
  *  \return None.
  */
 /*************************************************************************************************/
@@ -695,25 +695,17 @@ void txTestTask(void *pvParameters)
         snprintf(str, sizeof(str), "%s%s", str, (const char *)getPhyStr(phy));
         APP_TRACE_INFO1("%s", str);
 
-        //Prevent FreeRTOS from context switching until the LL is finished
-        vTaskSuspendAll();
-
         /* stat test */
         if (testConfig.testType == BLE_TX_TEST) {
             res = LlEnhancedTxTest(testConfig.channel, packetLen, packetType, phy, 0);
         } else {
             res = LlEnhancedRxTest(testConfig.channel, phy, 0, 0);
         }
-        xTaskResumeAll(); //Restore scheduler
-
         APP_TRACE_INFO2("result = %u %s", res, res == LL_SUCCESS ? "(SUCCESS)" : "(FAIL)");
         /* if duration value was given then let the test run that amount of time and end */
         if (testConfig.duration_ms) {
             vTaskDelay(testConfig.duration_ms);
-            //Prevent FreeRTOS from context switching until the LL is finished
-            vTaskSuspendAll();
             LlEndTest(NULL);
-            xTaskResumeAll(); //Restore scheduler
             xSemaphoreGive(rfTestMutex);
         }
         pausePrompt = false;

Examples/MAX32690/Bluetooth/RF_Test/main.c

@@ -246,8 +246,8 @@ uint8_t processEscSequence(uint8_t *seq)
  *
  *  \brief  adds latest command to command history buffer
  *
- *  \param  q pointer to the circular buffer holding command history
- *  \param  cmd pointer to the command string to be added
+ *  \param  q pointer to the circular buffer holding command history 
+ *  \param  cmd pointer to the command string to be added 
  *
  *  \return None.
  */
@@ -274,9 +274,9 @@ void cmdHistoryAdd(queue_t *q, const uint8_t *cmd)
  *  \fn     updateQueuePointer.
  *
  *  \brief  Updates an internal marker pointing to historical command to be printed, based on up/down arrow
- *
- *  \param  q pointer to the circular buffer holding command history
- *
+ * 
+ *  \param  q pointer to the circular buffer holding command history 
+ * 
  *  \param  upArrow flag used to upated the queuePoniter delimiting which command to print
  *
  *  \return None.
@@ -396,9 +396,9 @@ void prompt(void)
  *  \fn     printHint
  *
  *  \brief  Prints the help string of any command matching the current inputbuffer
- *
+ * 
  *  \param  buff pointer to the inputbuffer
- *
+ * 
  *  \return None.
  */
 /*************************************************************************************************/
@@ -695,25 +695,17 @@ void txTestTask(void *pvParameters)
         snprintf(str, sizeof(str), "%s%s", str, (const char *)getPhyStr(phy));
         APP_TRACE_INFO1("%s", str);
 
-        //Prevent FreeRTOS from context switching until the LL is finished
-        vTaskSuspendAll();
-
         /* stat test */
         if (testConfig.testType == BLE_TX_TEST) {
             res = LlEnhancedTxTest(testConfig.channel, packetLen, packetType, phy, 0);
         } else {
             res = LlEnhancedRxTest(testConfig.channel, phy, 0, 0);
         }
-        xTaskResumeAll(); //Restore scheduler
-
         APP_TRACE_INFO2("result = %u %s", res, res == LL_SUCCESS ? "(SUCCESS)" : "(FAIL)");
         /* if duration value was given then let the test run that amount of time and end */
         if (testConfig.duration_ms) {
             vTaskDelay(testConfig.duration_ms);
-            //Prevent FreeRTOS from context switching until the LL is finished
-            vTaskSuspendAll();
             LlEndTest(NULL);
-            xTaskResumeAll(); //Restore scheduler
             xSemaphoreGive(rfTestMutex);
         }
         pausePrompt = false;