Skip to main content
Anesh
Associate III
May 18, 2025
Solved

UART with DMA, not working with TCIE (Transmission Complete Interrupt Enable)

  • May 18, 2025
  • 2 replies
  • 1319 views

Dear together, 

For the past dew days Im stuck kwith UART communication with DMA.

To start with, I am implementing TX now. Later I will implement RX. I have managed for TX to work without interrupt. As per my application, I need Interrupt to inform the micro controller when transmission is completed.

But when I enabled the DMA_CCR_TCIE bit in  DMA1_Channel2->CCR, micro controller hangs. For your information, I have provided my code here. 

This is the function to Initialize DMA

void Init_DMA(void)
{
	// Enable clock for DMA1 peripheral by setting the DMA1EN bit in RCC AHBENR register
	RCC->AHBENR |= RCC_AHBENR_DMA1EN;

	// Enable DMA1_Channel2_3 interrupt in NVIC
	NVIC_EnableIRQ(DMA1_Channel2_3_IRQn);

	// Set priority for DMA1_Channel2_3 interrupt (Preemption = 0, Sub = 0)
	NVIC_SetPriority(DMA1_Channel2_3_IRQn, 0);
}

 

This is the function to Initialize UART1

void Init_UART1(void)
{
 // 1. Enable clocks for GPIOB, USART1, and DMA1
 RCC->AHBENR |= RCC_AHBENR_GPIOBEN;
 RCC->APB2ENR |= RCC_APB2ENR_USART1EN;

 // 2. Configure PB5 as DE output (push-pull, high speed)
 GPIOB->MODER &= ~(3 << (5 * 2));
 GPIOB->MODER |= (1 << (5 * 2)); // Output mode
 GPIOB->OTYPER &= ~(1 << 5); // Push-pull
 GPIOB->OSPEEDR |= (3 << (5 * 2)); // High speed
 GPIOB->PUPDR &= ~(3 << (5 * 2)); // No pull
 GPIOB->ODR &= ~(1 << 5); // Set DE LOW (receive mode)

 // 3. Configure PB6 as USART1_TX (AF0)
 GPIOB->MODER &= ~(3 << (6 * 2));
 GPIOB->MODER |= (2 << (6 * 2)); // Alternate function
 GPIOB->OTYPER &= ~(1 << 6); // Push-pull
 GPIOB->OSPEEDR |= (3 << (6 * 2)); // High speed
 GPIOB->PUPDR &= ~(3 << (6 * 2));
 GPIOB->PUPDR |= (1 << (6 * 2)); // Pull-up
 GPIOB->AFR[0] &= ~(0xF << (6 * 4));
 GPIOB->AFR[0] |= (0x0 << (6 * 4)); // AF0 = USART1

 // Reset USART1
 USART1->CR1 = 0;
 USART1->CR2 = 0;
 USART1->CR3 = 0;

 USART1->BRR = Set_USART1_BRR; // Set baud rate

 switch (Set_Parity)
 {
 case 0: // 8N2 (No parity, 2 stop bits)
 USART1->CR1 &= ~USART_CR1_PCE;
 USART1->CR1 &= ~USART_CR1_M;
 USART1->CR2 &= ~USART_CR2_STOP;
 USART1->CR2 |= USART_CR2_STOP_1 | USART_CR2_STOP_0; // 2 stop bits
 break;
 case 1: // 8E1 (Even parity, 1 stop bit)
 USART1->CR1 |= USART_CR1_PCE;
 USART1->CR1 &= ~USART_CR1_PS;
 USART1->CR1 |= USART_CR1_M;
 USART1->CR2 &= ~USART_CR2_STOP; // 1 stop bit
 break;
 case 2: // 8O1 (Odd parity, 1 stop bit)
 USART1->CR1 |= USART_CR1_PCE;
 USART1->CR1 |= USART_CR1_PS;
 USART1->CR1 |= USART_CR1_M;
 USART1->CR2 &= ~USART_CR2_STOP; // 1 stop bit
 break;
 }

 // 1 stop bit: CR2 STOP[1:0] = 00 (default, so we can skip)
 // Enable transmitter and receiver
 USART1->CR1 |= USART_CR1_TE | USART_CR1_RE;

 // Enable USART
 USART1->CR1 |= USART_CR1_UE;
}

 

This is the function used to transfet the data throgh DMA

void MB_Frame_Transmit(volatile uint8_t *data, uint16_t len)
{
 uart_tx_busy = 1;
 SET_DE; // Enable RS485 Transmit mode

 // Disable DMA channel before reconfiguring
 DMA1_Channel2->CCR &= ~DMA_CCR_EN;

 // Clear DMA interrupt flags for Channel 2
 DMA1->IFCR = DMA_IFCR_CTCIF2 | DMA_IFCR_CTEIF2;

 // Configure DMA Channel 2 for USART1_TX
 DMA1_Channel2->CNDTR = len;
 DMA1_Channel2->CPAR = (uint32_t)&USART1->TDR;
 DMA1_Channel2->CMAR = (uint32_t)data;

 // Reconfigure DMA Channel 2
 DMA1_Channel2->CCR =
 DMA_CCR_MINC | // Memory increment
 DMA_CCR_DIR | // Read from memory (TX)
 /*DMA_CCR_TCIE | // Enable transfer complete interrupt*/
 DMA_CCR_PL_1; // High priority (optional)

 // Enable USART1 TX DMA mode
 USART1->CR3 |= USART_CR3_DMAT;

 // Enable DMA Channel 2
 DMA1_Channel2->CCR |= DMA_CCR_EN;
}

 

This is the ISR

void DMA1_Channel2_3_IRQHandler(void)
{
 // Check Transfer Complete interrupt for Channel 2
 if (DMA1->ISR & DMA_ISR_TCIF2)
 {
 // Clear transfer complete flag
 DMA1->IFCR = DMA_IFCR_CTCIF2;

 // Disable DMA Channel 2 and USART DMA mode
 DMA1_Channel2->CCR &= ~DMA_CCR_EN;
 USART1->CR3 &= ~USART_CR3_DMAT;

 uart_tx_busy = 0;
 CLEAR_DE; // Set RS485 to Receive mode
 }

 // Optional: handle transfer error (TEIF2)
 if (DMA1->ISR & DMA_ISR_TEIF2)
 {
 DMA1->IFCR = DMA_IFCR_CTEIF2;
 // You can set an error flag or retry here

 Switch_Reset_LED_ON;
 }
}

 

I call the DMA1_Channel2_3_IRQHandler() function from main once in 600 mS. With this loop delay. I will not re initiate the DMA transfer before previous transfer is completed. delay_ms() is the blocking function implemented by timer 14.

Now the interesting part: If I disable the TCIE bit inise the MB_Frame_Transmit() function, DMA transfer happens perfectly once in 600ms. I probe this in my oscilloscope and confirm.

But if I enable TCIE, transmission happens only for first time when micro controller is restarted. Then it hangs.

 

Few more observations. This is my main function 

int main(void)
{
	Config_System_Clock();
	Init_GPIOs();
	Init_DMA();
	Init_UART1();
	Init_Tim14_1ms();

	 __enable_irq();

	for (uint8_t i = 0; i < 10; ++i) { TX_Array[i] = i; }

	//UART1_Transmit_DMA(TX_Array, 10);

	SET_DE;
	//Switch_Reset_LED_ON;

	while(1)
	{
		MB_Frame_Transmit(TX_Array, 10);
		Switch_Status_LED_ON; delay_ms(300);
		Switch_Status_LED_OFF; delay_ms(300);
	}
}

When the TCIE is enabled, the status LED is turned ON then not turned OFF. When means just for one time Switch_Statsu_LED_ON funtion is executed once then nothing further. Also the Reset LED placed insde Error handler is not turned, ON. which means means that part of ISR is also not executed. 

Please let me know, what is the issue in my code.

Thank you, Anesh S.

 

Best answer by waclawek.jan

> But why ISR is called immediately at time <= (length-2) of transmission?

That's when DMA finished its job.

The UART transmitter is initially empty and requests data from DMA through UART_SR.TXE, which indicates "UART TX holding register empty". DMA transfers that data into UART's holding register. UART transfers that byte rapidly from holding register into its shift register and starts to shift the bits out (i.e. transmit - note that UART baudrate is several orders of magnitude slower than the mcu system clock, so transmitting one single *bit* takes much much longer than transferring a whole byte by DMA). That means, that the UART holding register is empty again, UART_SR.TXE goes up, which triggers another byte to be transferred by DMA to the holding register, where it waits until the first byte gets completely shifted out from the shift register. It means, that DMA already transferred 2 bytes and UART barely started to transmit the first byte's startbit.

For longer arrays, this goes on, DMA is virtually always ahead of UART by 2 bytes, so it finishes its transfer 2 bytes before UART transmits the last byte's stopbit.

JW

2 replies

Tesla DeLorean
Guru
May 18, 2025

Probably not hanging, but stuck in an IRQ storm. If sources of interrupts are not cleared it will keep re-entering the IRQ Handler and no foreground code execution will occur.

Check pending interrupts.

Toggle GPIO at every IRQ Handler entry, scope pin as it will be happening very fast.

Perhaps have a count, and stop and inspect in the debugger. Determine WHERE it is stuck.

Check perhaps also HT (Half Transfer)

Tips, Buy me a coffee, or three.. PayPal VenmoUp vote any posts that you find helpful, it shows what's working..
Anesh
AneshAuthor
Associate III
May 18, 2025

Hello, I modified the ISR code to turn ON an LED and toggle another port pin. But none of these lines are executed. Before that I enabled TCIE.
I tried to debug code by STM32CubeIDE. Debug is not proceeding above the function MB_Frame_Transmit(TX_Array, 10);. Anyhow I am not fluent in debugging process.
Please let me know what is the next step to debug.

 

void DMA1_Channel2_3_IRQHandler(void)
{
	 Switch_Reset_LED_ON;
	 Toggle_Status_LED;

	// Check Transfer Complete interrupt for Channel 2
 if (DMA1->ISR & DMA_ISR_TCIF2)
 {
 // Clear transfer complete flag
 DMA1->IFCR = DMA_IFCR_CTCIF2;

 // Disable DMA Channel 2 and USART DMA mode
 DMA1_Channel2->CCR &= ~DMA_CCR_EN;
 USART1->CR3 &= ~USART_CR3_DMAT;

 uart_tx_busy = 0;
 CLEAR_DE; // Set RS485 to Receive mode
 }

 // Optional: handle transfer error (TEIF2)
 if (DMA1->ISR & DMA_ISR_TEIF2)
 {
 DMA1->IFCR = DMA_IFCR_CTEIF2;
 // You can set an error flag or retry here
 }
}

 

Tesla DeLorean
Guru
May 18, 2025

Like I said, STOP the code in the DEBUGGER, and determine WHERE it is stuck.

Instrument things like Error_Handler() and HardFault_Handler(), or any place you have silent while(1) loops where it can stuck and die.

Add other instrumentation so you know what's going on without the debugger, these things are happening faster than you can react, so provide continuous output rather than dead-stop with break-points and single-stepping.

DMA will only reload / retrigger in CIRCULAR mode.

Tips, Buy me a coffee, or three.. PayPal VenmoUp vote any posts that you find helpful, it shows what's working..
waclawek.jan
waclawek.janBest answer
Super User
May 20, 2025

> But why ISR is called immediately at time <= (length-2) of transmission?

That's when DMA finished its job.

The UART transmitter is initially empty and requests data from DMA through UART_SR.TXE, which indicates "UART TX holding register empty". DMA transfers that data into UART's holding register. UART transfers that byte rapidly from holding register into its shift register and starts to shift the bits out (i.e. transmit - note that UART baudrate is several orders of magnitude slower than the mcu system clock, so transmitting one single *bit* takes much much longer than transferring a whole byte by DMA). That means, that the UART holding register is empty again, UART_SR.TXE goes up, which triggers another byte to be transferred by DMA to the holding register, where it waits until the first byte gets completely shifted out from the shift register. It means, that DMA already transferred 2 bytes and UART barely started to transmit the first byte's startbit.

For longer arrays, this goes on, DMA is virtually always ahead of UART by 2 bytes, so it finishes its transfer 2 bytes before UART transmits the last byte's stopbit.

JW

Anesh
AneshAuthor
Associate III
May 20, 2025

Thank you for detailed information. Now I have implemented using USART_ISR_TC of UART1. I probed all the functions by place port pin on/off commands. Both the ISRs are working fine. I think I can handle even without ISR by DMA.

Anyhow, I left it to be in code, Inside this ISR I enabled USART_ISR_TC. I hope nothing will be hurt.

I post my complete code here of TX with DMA. I request you to have a look for any potential issues. 

#include <stdint.h>
#include <stddef.h>

#include "stm32f030x8.h"

#define Set_USART1_BRR (40000000/9600) // 9600 Baudrate
#define Set_Parity 0x01 // EVen

#define DMA1_Channel2_ISR_TCIF DMA_ISR_TCIF2 // Transfer complete flag for channel 2
#define DMA1_Channel2_IFCR_CTCIF DMA_IFCR_CTCIF2 // Clear transfer complete flag channel 2

#define Switch_Reset_LED_ON (GPIOB->BSRR = GPIO_BSRR_BS_10) // PB10 (Set pin)
#define Switch_Status_LED_ON (GPIOB->BSRR = GPIO_BSRR_BS_4) // PB4 (Set pin)
#define Switch_RXD_LED_ON (GPIOB->BSRR = GPIO_BSRR_BS_3) // PB3 (Set pin)
#define Switch_TXD_LED_ON (GPIOA->BSRR = GPIO_BSRR_BS_15) // PA15 (Set pin)

#define Switch_Reset_LED_OFF (GPIOB->BSRR = GPIO_BSRR_BR_10) // PB10 (Set pin)
#define Switch_Status_LED_OFF (GPIOB->BSRR = GPIO_BSRR_BR_4) // PB4 (Set pin)
#define Switch_RXD_LED_OFF (GPIOB->BSRR = GPIO_BSRR_BR_3) // PB3 (Set pin)
#define Switch_TXD_LED_OFF (GPIOA->BSRR = GPIO_BSRR_BR_15) // PA15 (Set pin)

#define Toggle_Reset_LED (GPIOB->ODR ^= GPIO_ODR_10) // PB10 (Set pin)
#define Toggle_Status_LED (GPIOB->ODR ^= GPIO_ODR_4)

#define CLEAR_DE (GPIOB->ODR &= ~(1 << 5));
#define SET_DE (GPIOB->ODR |= (1 << 5));

void Config_System_Clock(void);
void Init_GPIOs(void);
void Init_DMA(void);
void Init_UART1(void);
void MB_Frame_Transmit(volatile uint8_t *pData, uint16_t size);
void Init_Tim14_1ms(void);
void delay_ms(uint32_t ms);
void Error_Handler(void);
void block_delay_500ms_for_loop(void);

volatile uint32_t Ticks_1ms, Ticks_100ms;
volatile uint8_t Flag_1ms, Flag_100ms;

volatile uint8_t TX_Array[10]={0}, RX_Array[10]={0};

volatile uint8_t uart_tx_busy = 0;

int main(void)
{
	Config_System_Clock();
	Init_GPIOs();
	Init_DMA();
	Init_UART1();
	Init_Tim14_1ms();

	 __enable_irq();

	for (uint8_t i = 0; i < 10; ++i) { TX_Array[i] = i; }

	//UART1_Transmit_DMA(TX_Array, 10);


	//Switch_Reset_LED_ON;

	while(1)
	{
		if(Flag_100ms)
		{
			MB_Frame_Transmit(TX_Array, 10);
			Flag_100ms=0;
		}
		//block_delay_500ms_for_loop();
		//Switch_Status_LED_ON; delay_ms(300);
		//Switch_Status_LED_OFF; delay_ms(300);
		//Toggle_Reset_LED;

		if(Flag_1ms)
		{
			Toggle_Status_LED;
			Flag_1ms=0;
		}
	}
}

void Init_GPIOs(void)
{
	RCC->AHBENR |= RCC_AHBENR_GPIOAEN; // Enable GPIOA clock
	RCC->AHBENR |= RCC_AHBENR_GPIOBEN; // Enable GPIOB clock
	RCC->AHBENR |= RCC_AHBENR_GPIOCEN; // Enable GPIOC clock

	/* Settings for ONB_Reset_LED: PB10 */
	GPIOB->MODER &= ~GPIO_MODER_MODER10; // Clear mode for PB10
	GPIOB->MODER |= GPIO_MODER_MODER10_0; // Set as Output (01)
	GPIOB->OTYPER &= ~GPIO_OTYPER_OT_10; // Push-pull (0)
	GPIOB->OSPEEDR &= ~GPIO_OSPEEDR_OSPEEDR10; // Clear speed for PB10
	GPIOB->OSPEEDR |= GPIO_OSPEEDR_OSPEEDR10_0; // Medium speed (01)
	GPIOB->PUPDR &= ~GPIO_PUPDR_PUPDR10; // No pull-up, no pull-down (00)

	/* Settings for ONB_Status_LED: PB4 */
	GPIOB->MODER &= ~GPIO_MODER_MODER4; // Clear mode for PB4
	GPIOB->MODER |= GPIO_MODER_MODER4_0; // Set as Output (01)
	GPIOB->OTYPER &= ~GPIO_OTYPER_OT_4; // Push-pull (0)
	GPIOB->OSPEEDR &= ~GPIO_OSPEEDR_OSPEEDR4; // Clear speed for PB4
	GPIOB->OSPEEDR |= GPIO_OSPEEDR_OSPEEDR4_0; // Medium speed (01)
	GPIOB->PUPDR &= ~GPIO_PUPDR_PUPDR4; // No pull-up, no pull-down (00)

	/* Settings for ONB_RXD_LED: PB3 */
	GPIOB->MODER &= ~GPIO_MODER_MODER3; // Clear mode for PB3
	GPIOB->MODER |= GPIO_MODER_MODER3_0; // Set as Output (01)
	GPIOB->OTYPER &= ~GPIO_OTYPER_OT_3; // Push-pull (0)
	GPIOB->OSPEEDR &= ~GPIO_OSPEEDR_OSPEEDR3; // Clear speed for PB3
	GPIOB->OSPEEDR |= GPIO_OSPEEDR_OSPEEDR3_0; // Medium speed (01)
	GPIOB->PUPDR &= ~GPIO_PUPDR_PUPDR3; // No pull-up, no pull-down (00)

	/* Settings for ONB_TXD_LED: PA15 */
	GPIOA->MODER &= ~GPIO_MODER_MODER15; // Clear mode for PA15
	GPIOA->MODER |= GPIO_MODER_MODER15_0; // Set as Output (01)
	GPIOA->OTYPER &= ~GPIO_OTYPER_OT_15; // Push-pull (0)
	GPIOA->OSPEEDR &= ~GPIO_OSPEEDR_OSPEEDR15; // Clear speed for PA15
	GPIOA->OSPEEDR |= GPIO_OSPEEDR_OSPEEDR15_0; // Medium speed (01)
	GPIOA->PUPDR &= ~GPIO_PUPDR_PUPDR15; // No pull-up, no pull-down (00)
}

void Init_DMA(void)
{
 // Enable clock for DMA1
 RCC->AHBENR |= RCC_AHBENR_DMA1EN;

 // Optional: Preconfigure TX DMA Channel2 default settings (actual values set during transmit)
 DMA1_Channel2->CCR = 0; // Disable first
 DMA1_Channel2->CNDTR = 0;
 DMA1_Channel2->CPAR = (uint32_t)&USART1->TDR;
 DMA1_Channel2->CMAR = 0;
 DMA1_Channel2->CCR =
 DMA_CCR_MINC | // Memory increment mode
 DMA_CCR_DIR | // Read from memory (TX)
 DMA_CCR_TCIE | // Transfer complete interrupt
 DMA_CCR_PL_1; // High priority

 // Clear any pending DMA interrupt flags
 DMA1->IFCR = DMA_IFCR_CTCIF2 | DMA_IFCR_CTEIF2;

 // Configure NVIC for DMA Channel 2/3 interrupt
 NVIC_ClearPendingIRQ(DMA1_Channel2_3_IRQn);
 NVIC_SetPriority(DMA1_Channel2_3_IRQn, 0);
 NVIC_EnableIRQ(DMA1_Channel2_3_IRQn);
}

void Init_UART1(void)
{
 // 1. Enable clocks
 RCC->AHBENR |= RCC_AHBENR_GPIOBEN;
 RCC->APB2ENR |= RCC_APB2ENR_USART1EN;

 // 2. Configure PB5 as DE (output, push-pull, high speed)
 GPIOB->MODER &= ~(3 << (5 * 2));
 GPIOB->MODER |= (1 << (5 * 2));
 GPIOB->OTYPER &= ~(1 << 5);
 GPIOB->OSPEEDR |= (3 << (5 * 2));
 GPIOB->PUPDR &= ~(3 << (5 * 2));
 GPIOB->ODR &= ~(1 << 5); // Set DE low initially

 // 3. Configure PB6 as USART1_TX (AF0)
 GPIOB->MODER &= ~(3 << (6 * 2));
 GPIOB->MODER |= (2 << (6 * 2)); // Alternate function
 GPIOB->OTYPER &= ~(1 << 6);
 GPIOB->OSPEEDR |= (3 << (6 * 2));
 GPIOB->PUPDR &= ~(3 << (6 * 2));
 GPIOB->PUPDR |= (1 << (6 * 2)); // Pull-up
 GPIOB->AFR[0] &= ~(0xF << (6 * 4));
 GPIOB->AFR[0] |= (0x0 << (6 * 4)); // AF0

 // 4. Reset USART1 registers safely (only if USART is disabled)
 if (!(USART1->CR1 & USART_CR1_UE))
 {
 USART1->CR1 = 0;
 USART1->CR2 = 0;
 USART1->CR3 = 0;
 }

 // 5. Clear status flags
 volatile uint32_t tmp;
 tmp = USART1->ISR;
 tmp = USART1->RDR;
 (void)tmp;

 // 6. Set baud rate
 USART1->BRR = Set_USART1_BRR;

 // 7. Configure parity and stop bits
 switch (Set_Parity)
 {
 case 0: // 8N2
 USART1->CR1 &= ~USART_CR1_PCE;
 USART1->CR1 &= ~USART_CR1_M;
 USART1->CR2 &= ~USART_CR2_STOP;
 USART1->CR2 |= USART_CR2_STOP_1 | USART_CR2_STOP_0;
 break;
 case 1: // 8E1
 USART1->CR1 |= USART_CR1_PCE;
 USART1->CR1 &= ~USART_CR1_PS;
 USART1->CR1 |= USART_CR1_M;
 USART1->CR2 &= ~USART_CR2_STOP;
 break;
 case 2: // 8O1
 USART1->CR1 |= USART_CR1_PCE;
 USART1->CR1 |= USART_CR1_PS;
 USART1->CR1 |= USART_CR1_M;
 USART1->CR2 &= ~USART_CR2_STOP;
 break;
 }

 // 8. Enable TX and RX
 USART1->CR1 |= USART_CR1_TE | USART_CR1_RE;

 // 9. Enable USART1 interrupt (for TCIE used in TX complete)
 NVIC_ClearPendingIRQ(USART1_IRQn);
 NVIC_SetPriority(USART1_IRQn, 1);
 NVIC_EnableIRQ(USART1_IRQn);

 // 10. Enable USART1
 USART1->CR1 |= USART_CR1_UE;
}


void MB_Frame_Transmit(volatile uint8_t *data, uint16_t len)
{
	//Switch_Status_LED_ON;

	uart_tx_busy = 1;
 SET_DE; // Enable RS485 transmit mode

 // Disable DMA channel before reconfiguration
 DMA1_Channel2->CCR &= ~DMA_CCR_EN;

 // Clear DMA Channel 2 interrupt flags
 DMA1->IFCR = DMA_IFCR_CTCIF2 | DMA_IFCR_CTEIF2;

 // Configure DMA for USART1_TX
 DMA1_Channel2->CPAR = (uint32_t)&USART1->TDR;
 DMA1_Channel2->CMAR = (uint32_t)data;
 DMA1_Channel2->CNDTR = len;

 DMA1_Channel2->CCR =
 DMA_CCR_MINC | // Memory increment mode
 DMA_CCR_DIR | // Read from memory
 DMA_CCR_TCIE | // Enable transfer complete interrupt
 DMA_CCR_PL_1; // High priority (optional)

 // Enable USART1 DMA TX
 USART1->CR3 |= USART_CR3_DMAT;

 // Enable DMA channel
 DMA1_Channel2->CCR |= DMA_CCR_EN;

 //Switch_Status_LED_OFF;
}


void DMA1_CH2_3_IRQHandler(void)
{
 uint32_t isr = DMA1->ISR;

 if (isr & DMA_ISR_TCIF2)
 {
 	//Switch_Status_LED_ON;

 	// Clear transfer complete flag and disable DMA
 DMA1->IFCR = DMA_IFCR_CTCIF2;
 DMA1_Channel2->CCR &= ~DMA_CCR_EN;
 USART1->CR3 &= ~USART_CR3_DMAT;

 // Enable USART TC interrupt to detect final byte transmission on the line
 USART1->CR1 |= USART_CR1_TCIE;

 //Switch_Status_LED_OFF;
 }

 if (isr & DMA_ISR_TEIF2)
 {
 DMA1->IFCR = DMA_IFCR_CTEIF2;
 // Optional: set error flag or retry logic
 }
}


void USART1_IRQHandler(void)
{
 uint32_t isr = USART1->ISR;
 volatile uint32_t dummy;

 // Handle Transmission Complete
 if (isr & USART_ISR_TC)
 {
 	//Switch_Status_LED_ON;

 	USART1->ICR = USART_ICR_TCCF; // Clear Transmission Complete
 USART1->CR1 &= ~USART_CR1_TCIE; // Disable TC interrupt

 CLEAR_DE; // Return RS485 to receive mode
 uart_tx_busy = 0;

 Toggle_Reset_LED;

 //Switch_Status_LED_OFF;
 }

 // Just clear **all** USART error flags if any error is present
 if (isr & (USART_ISR_PE | USART_ISR_FE | USART_ISR_NE | USART_ISR_ORE))
 {
 USART1->ICR = USART_ICR_PECF | USART_ICR_FECF | USART_ICR_NCF | USART_ICR_ORECF; // Clear all errors
 dummy = USART1->RDR; // Read RDR to clear error conditions
 (void)dummy;
 }
}


__attribute__((naked)) void HardFault_Handler(void)
{
 __asm volatile
 (
 "movs r0, #4 \n"
 "mov r1, lr \n"
 "tst r0, r1 \n"
 "beq _msp_used \n"
 "mrs r0, psp \n"
 "b HardFault_Handler_C \n"
 "_msp_used: \n"
 "mrs r0, msp \n"
 "b HardFault_Handler_C \n"
 );
}

void HardFault_Handler_C(uint32_t *stacked_regs)
{
 // Extract stacked CPU registers at fault time
 volatile uint32_t r0 = stacked_regs[0];
 volatile uint32_t r1 = stacked_regs[1];
 volatile uint32_t r2 = stacked_regs[2];
 volatile uint32_t r3 = stacked_regs[3];
 volatile uint32_t r12 = stacked_regs[4];
 volatile uint32_t lr = stacked_regs[5]; // Link Register
 volatile uint32_t pc = stacked_regs[6]; // Program Counter
 volatile uint32_t psr = stacked_regs[7]; // Program Status Register

 // Optional: Place breakpoint here to examine values in debugger
 // You can also send them over UART or log to flash

 (void)r0;
 (void)r1;
 (void)r2;
 (void)r3;
 (void)r12;
 (void)lr;
 (void)pc;
 (void)psr;

 // Stay here or call reset
 Error_Handler();
}

void Error_Handler(void)
{
 // Blink PB4 forever to indicate error
 while (1)
 {
 	Toggle_Reset_LED;
 for (volatile uint32_t i = 0; i < 100000; i++); // Delay loop
 }
}

void Init_Tim14_1ms(void)
{
 RCC->APB1ENR |= RCC_APB1ENR_TIM14EN;

 TIM14->PSC = 39;
 TIM14->ARR = 999;
 TIM14->CNT = 0;

 TIM14->SR &= ~TIM_SR_UIF; // Clear any pending flag
 TIM14->DIER |= TIM_DIER_UIE; // Update interrupt enable
 TIM14->CR1 |= TIM_CR1_CEN; // Counter enable

 NVIC_EnableIRQ(TIM14_IRQn); // Enable IRQ in NVIC
}

void TIM14_IRQHandler(void)
{
 if (TIM14->SR & TIM_SR_UIF)
 {
 TIM14->SR &= ~TIM_SR_UIF; // Clear update interrupt flag
 Ticks_1ms++; // Optional global ms counter

 Flag_1ms = 1; // Set 1ms flag

 // Increment 100ms counter and set flag when 100ms elapsed
 if (++Ticks_100ms >= 100)
 {
 Flag_100ms = 1;
 Ticks_100ms = 0; // Reset for next 100 ms period
 }
 }
}

void delay_ms(uint32_t ms)
{
 uint32_t start = Ticks_1ms;
 while ((Ticks_1ms - start) < ms);
}

void Config_System_Clock(void)
{
 // 1. Enable HSE (High Speed External oscillator)
 RCC->CR |= RCC_CR_HSEON;
 while ((RCC->CR & RCC_CR_HSERDY) == 0); // Wait for HSE ready

 // 2. Configure PLL
 // PLL source = HSE, PLL multiplier = 5
 RCC->CFGR &= ~(RCC_CFGR_PLLSRC | RCC_CFGR_PLLMUL); // Clear PLL config
 RCC->CFGR |= RCC_CFGR_PLLSRC_HSE_PREDIV | RCC_CFGR_PLLMUL5; // HSE / 1 * 5

 // 3. Enable PLL
 RCC->CR |= RCC_CR_PLLON;
 while ((RCC->CR & RCC_CR_PLLRDY) == 0); // Wait for PLL ready

 // 4. Select PLL as system clock
 RCC->CFGR &= ~RCC_CFGR_SW;
 RCC->CFGR |= RCC_CFGR_SW_PLL;
 while ((RCC->CFGR & RCC_CFGR_SWS) != RCC_CFGR_SWS_PLL); // Wait until PLL used as system clock
}

void block_delay_500ms_for_loop(void)
{
	for (uint32_t i = 0; i < 5000001; i++)
	{
	 __NOP();
	}
}

Now I started implementing RX via DMA. 

Thank you.

waclawek.jan
Super User
May 20, 2025

While Tx is fully under the mcu's control and so DMA is relatively straighforward; Rx through DMA is tricky, as it operates on data from outside, which may be corrupted. Some folks combine Rx DMA with timeouts (often in the form of the UART's IDLE feature), but I personally simply don't use DMA and handle it in interrupt. UART is relatively slow.

JW