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

Update dependencies (fixes #13), fix bugs, add USB support #14

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

valpackett
Copy link

(Continues #11)

TODO:

  • implement flash latency management directly in RCC (?) instead of the dp.FLASH.acr fiddling in the USB example /* RM0038 3.3.1 */
  • figure out the "correct" USB startup_delay72 copy-pasted from F1 does just work but may be excessive?
  • hide the apb1 fiddling in USB enable into functions like other HALs do: USB::enable_unchecked() etc. (how do they do that?)

This was referenced Jan 10, 2022
@hdhoang
Copy link
Contributor

hdhoang commented Jan 13, 2022

Thank you! I will try to find some time to run examples on my keyboard. USB support is essential

@hdhoang
Copy link
Contributor

hdhoang commented Jun 25, 2022

thanks for porting it to rtic@1 as well! I'm working through their examples again.

The usage of .write() was disabling the PLL's source (!) exactly
when the PLL itself was enabled, causing the PLL to never come up.
24 is not a real restriction: anything above 16 requires
setting LATENCY (1 WS) for flash already.
32 is when the flash maxes out even in LATENCY mode.
Mostly based on the other STM32 HALs
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