Skip to content

Add more registers, and MPU support #7

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

Merged
merged 14 commits into from
Feb 24, 2025

Conversation

jonathanpallant
Copy link
Contributor

Updates #4 to add a bunch more registers.

I didn't do this with a macro to allow us to special-case each register in future, as demand requires.

  • I added a PMSAv8-32 EL1 MPU driver. In doing so I realised a bunch of registers were Armv8-R only, so I moved them.
  • Added support for Armv7-R MPUs. Similar, but different, to Armv8-R MPUs.
  • I added a Generic Timer driver, and tested in on QEMU's Cortex-R52 machine. I'm not aware of a Armv7-R processor that has the Generic Timer, but there are some Armv7-A processors that do.

jonathanpallant and others added 14 commits February 18, 2025 18:34
Comes with big scary warning sign though.
The values are taken from the Cortex-R52 Reference Manual. We
only have basic reads/writes for the new ones - I haven't drilled
down into the bitfields.
Allows me to drop the decode function.
It just stubs out all the timer code.
The unit test code needs to see all the modules.
@jonathanpallant jonathanpallant requested a review from a team February 24, 2025 11:11
@jonathanpallant
Copy link
Contributor Author

Replaces #6, which was merged into master after the main branch was changed to main :/

@jonathanpallant jonathanpallant merged commit 016cdf9 into rust-embedded:main Feb 24, 2025
88 checks passed
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