I wrote this whole post and lost it so re-writting it again, sigh...
What I find it people having issues with HAL_UART_Receive_xx() and HAL_UART_Transmit_xx() don't take into consideration of the HAL return status.
You'll have to re-write your code so that you can re-try receiving and transmitting if HAL returns busy.
Below code snippets should show you how I'm using error flags so I can re-try to enable interrupts or re-transmit again.
bool UART_Receive_IT_errorFlag = false;
bool UART_Transmit_IT_errorFlag = false;
void HAL_UART_RxCpltCallback(UART_HandleTypeDef *huart)
{
Battery_Address = RX_Buff[0];
Charge_Start = RX_Buff[1];
// toggle pin
RS485_Counter++;
EnableUART_Receive_IT();
}
void EnableUART_Receive_IT(void)
{
if(HAL_UART_Receive_IT(&huart1, RX_Buff, 2) != HAL_OK)
{
UART_Receive_IT_errorFlag = true; // failed to enable interrupt so need to re-enable in polling routine.
}
}
void UART_Transmit(void)
{
if(HAL_UART_Transmit_IT(&huart, TX_Buff, 1) != HAL_OK)
{
UART_Transmit_IT_errorFlag = true;
}
}
// main loop, usually called from main while loop
void PollingRoutine(void)
{
CheckBatteryStatus();
CheckUART_Receive_IT_errorFlag();
CheckUART_Transmit_IT_ErrorFlag();
}
// polling for battery message
void CheckBatteryStatus(void)
{
if(Battery_Address == BATTERY_BUS_ADDRESS)
{
Battery_Address == 0; // do we need to clear?
TX_Buff[0] = RS485_Counter;
UART_Transmit();
}
}
void CheckUART_Receive_IT_errorFlag(void)
{
if(UART_Receive_IT_errorFlag)
{
UART_Receive_IT_errorFlag = 0;
EnableUART_Receive_IT(); // try again to enable interrupt
}
}
void CheckUART_Transmit_IT_ErrorFlag(void)
{
if(UART_Transmit_IT_errorFlag)
{
UART_Transmit_IT_errorFlag = 0;
UART_Transmit(); // previous transmit failed, so re-transmit again
}
}
If you find my answers useful, click the accept button so that way others can see the solution.
