BSP USART1 pins should be PA9 and PA10 for STM32H747I-DISCO

STMicroelectronics / STM32CubeH7

STM32Cube MCU Full Package for the STM32H7 series - (HAL + LL Drivers, CMSIS Core, CMSIS Device, MW libraries plus a set of Projects running on all boards provided by ST (Nucleo, Evaluation and Discovery Kits))

https://www.st.com/en/embedded-software/stm32cubeh7.html

Other

490 stars 302 forks source link

BSP USART1 pins should be PA9 and PA10 for STM32H747I-DISCO #208

Closed VictorTagayun closed 1 year ago

VictorTagayun commented 2 years ago

As per readme, USART1 pins are PB14 and PB15. https://github.com/STMicroelectronics/STM32CubeH7/blob/b340b13929e36a3427b8d94e8b1006022f82273f/Projects/STM32H747I-DISCO/Examples/UART/UART_WakeUpFromStopUsingFIFO/readme.txt#L47

Pls chk if it should be. PA9 = USART1_TX
PA10 = USART1_RX

as per main.h https://github.com/STMicroelectronics/STM32CubeH7/blob/b340b13929e36a3427b8d94e8b1006022f82273f/Projects/STM32H747I-DISCO/Examples/UART/UART_WakeUpFromStopUsingFIFO/CM7/Inc/main.h#L45

ASELSTM commented 2 years ago

Hi @VictorTagayun,

You are right the UART Tx and Rx pins mentioned in the readme.txt file are uncoherent with the pins actually configured. The reademe.txt file should then be updated as follow:

Board: STM32H747I-DISCO
-Tx Pin: PB.14
+Tx Pin: PA.9
-Rx Pin: PB.15
+Rx Pin: PA.10

This fix will be implemented and made available in the frame of a next release.

With regards,

ASELSTM commented 2 years ago

ST Internal Reference: 125777

ASELSTM commented 1 year ago

Hi @VictorTagayun,

Thank you for your contribution. This issue has been fixed in the frame of version v1.11.0 of the STM32CubeH7. Please allow me then to close this thread.

With regards,