Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

STM32G4xx: Wrong SystemCoreClock variable #22245

Closed
martinjaeger opened this issue Jan 27, 2020 · 2 comments · Fixed by #22311
Closed

STM32G4xx: Wrong SystemCoreClock variable #22245

martinjaeger opened this issue Jan 27, 2020 · 2 comments · Fixed by #22311
Assignees
@FRASTM
Labels
bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32 priority: low Low impact/importance bug

Comments

@martinjaeger
Copy link
Member

Description
Because of a bug in STM32Cube LL, the SystemCoreClock variable is set wrong if the actual system frequency is >80 MHz.

More details are described in a related (but already closed) PR:
zephyrproject-rtos/hal_stm32#34 (comment)

I raised this issue to track the bug, as I can't raise an issue in the original HAL repository.

Impact
Boards with STM32G431 are hardly useable with high clock frequencies, as e.g. UART baud rates are get configured wrong.
Preliminary workaround could be to reduce the system frequency to <80 MHz.
@martinjaeger martinjaeger added the bug The issue is a bug, or the PR is fixing a bug label Jan 27, 2020
@martinjaeger martinjaeger changed the title STM32G431: Wrong SystemCoreClock variable STM32G4xx: Wrong SystemCoreClock variable Jan 27, 2020
@martinjaeger martinjaeger mentioned this issue Jan 27, 2020
Closed
@ulfalizer
Copy link
Collaborator

ulfalizer commented Jan 28, 2020 •
edited
Loading

Semi-unrelated, but could we enable issues on module repos? I ran into it too. @nashif

@erwango erwango added the platform: STM32 ST Micro STM32 label Jan 28, 2020
@jhedberg jhedberg added the priority: low Low impact/importance bug label Jan 28, 2020
@FRASTM FRASTM mentioned this issue Jan 29, 2020
Merged
@rosterloh
Copy link
Contributor

zephyrproject-rtos/hal_stm32#36

@erwango erwango added the has-pr label Feb 12, 2020
@FRASTM FRASTM mentioned this issue Mar 11, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@FRASTM FRASTM

Labels
bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32 priority: low Low impact/importance bug
Projects
None yet
Milestone
No milestone
6 participants
@ulfalizer @jhedberg @rosterloh @martinjaeger @erwango @FRASTM