Missing out byte while receiving data on UART

[newbie_stm32]

Hi Community,

I am working in a freeRTOS environment where one of the threads parses the serial data from the UART. A protocol is defined to decode the hex stream received from another device. HAL_UART_RxCpltCallback is configured to receive one byte at a time and is filled in an array that houses a sufficient amount of data (no overflow). But sometimes a byte is getting missed out which is a mid byte.

To troubleshoot this issue set the RX FIFO threshold to generate an interrupt if the FIFO is full. However, whenever a byte is missed it does not trigger a HAL_UARTEx_RxFifoFullCallback (still figuring out how to get this work no interrupt is being generated even if I set the threshold to UART_RXFIFO_THRESHOLD_1_8).

Core: STM32G491

Clock: 170MHz (HSE + PLL)

Here is the program flow:

 

uint8_t rx_data;

for (;;)
{
 /* This is added for the synchronization to disable ISR being called */
 HAL_UART_AbortReceive(&huart1);

 /* 1. Check for the stream ending \r\n
 * 2. Decode and process the byte stream
 * 3. Reset index (index = 0)
 */

 /* This is added for the synchronization */
 HAL_UART_Receive_IT(&huart1, &rx_data, sizeof(rx_data));
 
 osDelay(1);
}

 

ISR

 

void HAL_UART_RxCpltCallback (UART_HandleTypeDef * huart)
{
 rx_buffer[index++] = rx_data;
 HAL_UART_Receive_IT(&huart1, &rx_data, sizeof(rx_data));
}

 

[TDK]

> HAL_UART_AbortReceive

This shouldn't be called from the main loop. If you abort something already in progress, it makes sense that bytes are being lost.

Also, calling HAL_UART_Receive_IT repeatedly is not proper.

 

What you should be doing is the following:

• Call HAL_UART_Receive_IT once within the main thread to start reception.
• Call HAL_UART_Receive_IT once within HAL_UART_RxCpltCallback (you're doing this).

[newbie_stm32]

Hi @TDK, thanks for pointing out misconception.