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.

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

stm32cube: stm32g4xx: Fix SystemCoreClock value > 80MHz #36

Merged
merged 1 commit into from
Feb 12, 2020
Merged

stm32cube: stm32g4xx: Fix SystemCoreClock value > 80MHz #36

merged 1 commit into from
Feb 12, 2020

Conversation

rosterloh
Copy link
Contributor

SystemCoreClock value should be updated after AHB prescale
is set back to 1.

Signed-off-by: Richard Osterloh richard.osterloh@gmail.com

@martinjaeger
Copy link
Member

@FRASTM don't you agree this should be fixed for both HSI and HSE?

@rosterloh
Copy link
Contributor Author

@martinjaeger I totally agree. Updated.

@martinjaeger
Copy link
Member

@martinjaeger I totally agree. Updated.

Thanks :)

Copy link
Member

@erwango erwango left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor comment.
Also miss README update (patch list)

stm32cube/stm32g4xx/drivers/src/stm32g4xx_ll_utils.c Outdated Show resolved Hide resolved
stm32cube/stm32g4xx/drivers/src/stm32g4xx_ll_utils.c Outdated Show resolved Hide resolved
stm32cube/stm32g4xx/drivers/src/stm32g4xx_ll_utils.c Outdated Show resolved Hide resolved
SystemCoreClock value should be updated after AHB prescale
is set back to 1.

Signed-off-by: Richard Osterloh <richard.osterloh@gmail.com>
@rosterloh
Copy link
Contributor Author

@erwango Hopefully that sorts it

@FRASTM
Copy link
Collaborator

FRASTM commented Feb 6, 2020

READme for stm32cube updated in the #37
commit: ae44693

@galak galak merged commit 3c98339 into zephyrproject-rtos:master Feb 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants