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

Expose configuration of the SEVONPEND bit within the System Control Register (SCR) #539

Merged
merged 1 commit into from
Jun 30, 2024

Conversation

calebstewart
Copy link
Contributor

This PR exposes the SEVONPEND bit of the System Control Register (SCR) within the SCB type. This seems inline with the current implementation of the other two defined SCR fields: SLEEPDEEP and SLEEPONEXIT.

As a side-note, I'm relatively new to embedded development in rust, so if this has been omitted for some specific reason I'm unaware of, I'm happy to be corrected. :) That being said, I simply implemented these two new methods in the same way that set/clear_sleepdeep and set/clear_sleeponexit were implemented with the help of section B3.2.7 of the ARMv7-M Architecture Reference Manual.

@calebstewart calebstewart requested a review from a team as a code owner June 23, 2024 06:03
Copy link
Member

@newAM newAM left a comment

Choose a reason for hiding this comment

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

Thanks!

@newAM newAM added this pull request to the merge queue Jun 30, 2024
@newAM newAM removed this pull request from the merge queue due to a manual request Jun 30, 2024
@newAM newAM added this pull request to the merge queue Jun 30, 2024
@newAM
Copy link
Member

newAM commented Jun 30, 2024

Re-queued this to allow #542 to go through first which speeds up CI (and the merge queues) considerably.

Merged via the queue into rust-embedded:master with commit ab31964 Jun 30, 2024
13 of 14 checks passed
newAM added a commit that referenced this pull request Jun 30, 2024
newAM added a commit that referenced this pull request Jun 30, 2024
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