Skip to main content
ONadr.1
ONadr.1
Senior III
May 22, 2023
Question

Bug in HAL STM32C011J6M6 when MCO Output is set.

  • May 22, 2023
  • 3 replies
  • 2508 views

Hello,

I have a very simple project to generate 50/60Hz PWM signal with MCO 3MHz output for other HW. So far my code is just turning on the PWM, everything else is generated by CubeMX. But the MCU falls into HardFault when MCO (Pin 4 PF2-NRST) is set as MCO. I found that HAL_GPIO_Init() causes this HatdFault on line 198 where the GPIO base address is 0x5000 0C00 (Port D). If I correct the address to 0x5000 0000 (Port A) by debugger, everything works fine. Apparently there is an error in the calculation of this base address in the HAL library.

Option byte NRST_MODE = 2

Exported project is attached.

This topic has been closed for replies.

3 replies

Tesla DeLorean
Tesla DeLorean
Guru
May 22, 2023

The listing file suggests it is passing 0x50000000 (0xA0 << 23)

GPIOF passes as 0x50001400

You're enabling PA12

PA0/PA1/PA2/PF2-NRST are bonded to the same, sure there's no conflict there?

I'd probably watch the order of initialization very carefully, especially what's in SystemClock_Config() and then where you bind the pin. Is SYSCFG clock enabled?

Tips, Buy me a coffee, or three.. PayPal Venmo (See Profile) Up vote any posts that you find helpful, it shows what's working..
Tesla DeLorean
Tesla DeLorean
Guru
May 22, 2023

Problems is how HAL_RCC_MCOConfig() stm32c0xx_hal_rcc.c determines which GPIO port it's supposed to be enabling.

You call

 SystemClock_Config(); 

Before

 MX_GPIO_Init();

and HAL_SYSCFG_SetPinBinding(HAL_BIND_SO8_PIN4_PF2); 

@Piranha​ what a festering mess this all is..

Tips, Buy me a coffee, or three.. PayPal Venmo (See Profile) Up vote any posts that you find helpful, it shows what's working..
ONadr.1
ONadr.1Author
Senior III
May 23, 2023

Thanks for your response. I have no time to do some deeper investigation now. You're probably right about the port address, it also works with the base address of port F and there shouldn't be a conflict. The problem is probably in the function to calculate the base address, but it is a very unintuitive (for me) algorithm and it would take more time to find the source of the problem.

ONadr.1
ONadr.1Author
Senior III
May 23, 2023

So I found the bug in the HAL firmware. It is in stm32c0xx_hal_gpio_ex.h file. Line 302 has an ill-defined value because this MCU does not have port D and port E implemented.

// original code 
#define GPIO_GET_INDEX(__GPIOx__) (((__GPIOx__) == (GPIOA))? 0uL :\
 ((__GPIOx__) == (GPIOB))? 1uL :\
 ((__GPIOx__) == (GPIOC))? 2uL :\
 ((__GPIOx__) == (GPIOF))? 3uL : 4uL)
 
//new code
#define GPIO_GET_INDEX(__GPIOx__) (((__GPIOx__) == (GPIOA))? 0uL :\
 ((__GPIOx__) == (GPIOB))? 1uL :\
 ((__GPIOx__) == (GPIOC))? 2uL :\
 ((__GPIOx__) == (GPIOF))? 5uL : 6uL)

GPIOx port mapping is on the picture from reference manual.


_legacyfs_online_stmicro_images_0693W00000bkh4QQAQ.png

Tesla DeLorean
Tesla DeLorean
Guru
May 23, 2023

@Sara BEN HADJ YAHYA​ @Imen DAHMEN​ @Amel NASRI​ 

Can we get some development eyes on this.

The attached project calls these prior to pin binding.

https://github.com/STMicroelectronics/stm32c0xx_hal_driver/blob/main/Src/stm32c0xx_hal_rcc.c#L81

https://github.com/STMicroelectronics/stm32c0xx_hal_driver/blob/main/Src/stm32c0xx_hal_rcc.c#L785

It uses HAL_GPIO_Init() in THREE places, TWO in MX_GPIO_Init() and ONE in HAL_RCC_MCOConfig()

If as the OP notes GPIO_GET_INDEX() is used to determine an address, this will fail too.

Tips, Buy me a coffee, or three.. PayPal Venmo (See Profile) Up vote any posts that you find helpful, it shows what's working..
I
Imen.D
ST Technical Moderator
May 25, 2023

Thank you @Community member​ for noticing us. I will check this reported issue and take the necessary action for fix.

Imen

In order to give better visibility on the answered topics, please click on 'Best answer' on the reply which solved your issue or answered your question. Thanks
Terms & ConditionsAccessibility statement