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

pwr: Add power configuration module #7

Merged
merged 1 commit into from
Jul 16, 2024
Merged

pwr: Add power configuration module #7

merged 1 commit into from
Jul 16, 2024

Conversation

astapleton
Copy link
Contributor

The Power configuration module manages voltage scaling modes, which need to be set correctly based on power usage considerations, as well as desired clock speed. The blinky example incorporates the power module and demonstrates choosing a voltage scaling mode.

src/pwr.rs Show resolved Hide resolved
src/pwr.rs Outdated
// ************************************
// TODO: Warning! STM32H503 Errata 2.2.13 states that for Rev A, Z this transition should only
// be executed from RAM
// Move this function into RAM!
Copy link
Member

Choose a reason for hiding this comment

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

This is a pretty impressive errata! I wonder if any Rev A or Z hardware ever made it out into the wild? Assuming that current dev boards have revision Y parts on them, I wouldn't worry about it too much.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, it's a fun one. I don't know whether Rev A or Z was ever released widely. It seems as though the dev boards that ST sells and the processors that I've used since Q3 2023 are Rev Y. I can probably just leave a note here, instead of a TODO, given how unlikely it seems that someone would actually run into this.

Base automatically changed from as/fix-prelude to master June 10, 2024 22:03
@astapleton astapleton merged commit 35d3a65 into master Jul 16, 2024
14 checks passed
@astapleton astapleton deleted the as/pwr branch July 16, 2024 16:41
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.

2 participants