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

[NUCLEO_F302R8] Add cmsis and hal files + change F401RE clock to 84MHz #215

Merged
merged 13 commits into from
Mar 19, 2014
Merged

Conversation

bcostm
Copy link
Contributor

@bcostm bcostm commented Mar 18, 2014

No description provided.

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 18, 2014

Hello bcostm,

have you had time to read the message about ST platforms on mbed-devel?
I can see this commit fixes the clock issue , just for F401RE. Is really target F401RE supported by default with just 16MHz clock enabled? My assumption is based on the comments present in the code (added for mbed).
I wonder what does comments in the code /* [ADDED FOR MBED] */ mean?

How does those 2 immediate merge commits got there?

Regards,
0xc0170

@bcostm
Copy link
Contributor Author

bcostm commented Mar 19, 2014

Hello,
I have read some messages on mbed-devel but not all of them. I am actually going to change all the clocks to the maximum possible using the HSI+PLL on all the Nucleo platforms. It will be on GitHub soon.
I have just added this comment to see what has been changed in the original system_stxxx file coming from the CubeF4 or StdPeriph Libraries. That's all.
Regards

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 19, 2014

@bcostm, can you look at this thread: https://groups.google.com/forum/#!topic/mbed-devel/Z6xw7MTR7Aw

emilmont added a commit that referenced this pull request Mar 19, 2014
[NUCLEO_F302R8] Add cmsis and hal files + change F401RE clock to 84MHz
@emilmont emilmont merged commit ace35df into ARMmbed:master Mar 19, 2014
bridadan pushed a commit that referenced this pull request Jun 21, 2016
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.

3 participants