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

Adjustable overclock for F405 and F411 #5210

Merged
merged 1 commit into from Feb 18, 2018

Conversation

DieHertz
Copy link
Member

@DieHertz DieHertz commented Feb 16, 2018

Minimal-impact adjustable overclock from #5193 for F405 and F411.

  • DISABLE_OVERCLOCK replaced by USE_OVERCLOCK defined for supported targets
  • F405 overclock is now one of OFF, 192MHZ, 216MHZ, and 240MHZ
  • Fixed invalid (and potentially dangerous) CCM register access on F411xE and F446 which have no CCM memory
  • Offset _estack 16 bytes from RAM end to avoid stomping on hot reboot flags while marking stack with 16 on devices with no CCM. Also I've noticed that it may be prone to errors even on F405 with CCM as flash ISR vector table contains _estack rather than _heap_stack_end, allowing to overrun reboot flags before they are used. This shouldn't be an issue when using linker properly like in Refactor startup assembly and reboot flags #5193 though
  • Fixed cpu_overclock setting doing nothing on F411xE, now we have either OFF, 108MHZ, or 120MHZ to choose from

More generic refactor and rewrite of Assembly & C startup/system files delayed for 3.4.

Tested on OMNIBUSF4SD and NOX v2.15

@DieHertz
Copy link
Member Author

Doesn't work as of now, will debug tomorrow :-)

@fujin
Copy link
Collaborator

fujin commented Feb 17, 2018

Confirmed behaviour on F405: 192MHZ, 216MHZ, 240MHZ all function as anticipated.

@mikeller
Copy link
Member

Confirmed working with 96MHz, 108MHz, 120MHz on MATEKF411RX.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants