-
Notifications
You must be signed in to change notification settings - Fork 3k
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
STM32: Fix system clock setup for XTAL and/or internal source on STM32H743 #13640
Conversation
@isaev-d, thank you for your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will approve, just remove the unexpected space
and correct the PR header into H743 :-)
Thx!
@@ -72,7 +72,7 @@ void SetSysClock(void) | |||
#if ((CLOCK_SOURCE) & USE_PLL_HSE_XTAL) | |||
/* 2- If fail try to start with HSE and external xtal */ | |||
if (SetSysClock_PLL_HSE(0) == 0) | |||
#endif | |||
#endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor comment, remove space
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, space removed = )
Also I changed error message to compile time check. Check it please, @jeromecoutant
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One styling request above to resolve
Pull request has been modified.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a comment on the PR header:
internal source on stm32f743
=> internal source on STM32H743 :-)
CI started |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
Summary of changes
Fix #13591
PLL configuration for system clock was wrong for cases of internal clock source or external resonator.
Impact of changes
Applies to nucleo H743 board. Another members of series also could be effected, but that wasn't tested.
Migration actions required
Documentation
To test XTAL create file mbed_app.json with following detectives:
{ "target_overrides": { "NUCLEO_H743ZI2": { "target.clock_source": "USE_PLL_HSE_XTAL", "target.hse_value": "25000000" } } }
To use internal clock source:
{ "target_overrides": { "NUCLEO_H743ZI2": { "target.clock_source": "USE_PLL_HSI" } } }
Pull request type
Test results
Reviewers
@jeromecoutant