Skip to content

riscv: add mseccfg/h CSR registers - #338

Merged
romancardenas merged 6 commits into
rust-embedded:masterfrom
rmsyn:riscv/register/mseccfg
Aug 13, 2025
Merged

romancardenas merged 6 commits into
rust-embedded:masterfrom
rmsyn:riscv/register/mseccfg

Conversation

@rmsyn

@rmsyn rmsyn commented Aug 12, 2025

Copy link
Copy Markdown
Contributor

Adds the mseccfg and mseccfgh CSRs for machine-mode security configuration.

Adds basic unit tests for the new CSRs.

Minor fixups for broken mip/sip unit tests.

@rmsyn
rmsyn requested a review from a team as a code owner August 12, 2025 22:42
rmsyn added 5 commits August 12, 2025 22:48
Adds the `mseccfg` machine security configuration CSR register.
Fixes previous read-write tests for the MIP and SIP CSRs, which are now
read-only.
Adds basic unit tests for the `mseccfg` CSR register.
Adds the `mseccfgh` machine security configuration (high-bits) CSR.
Adds basic unit tests for the `mseccfgh` CSR register.
@rmsyn
rmsyn force-pushed the riscv/register/mseccfg branch from 94382ed to dd08a52 Compare August 12, 2025 22:49
@rmsyn

rmsyn commented Aug 12, 2025

Copy link
Copy Markdown
Contributor Author

Warning: "useed" should be "used".
Warning: "useed" should be "used".

No spellcheck, it shouldn't. @romancardenas how do I tell spellcheck to ignore this? Edit: I think I figured it out with the added typo.toml entry. Please let me know if you want me to adjust the regexp.

@rmsyn
rmsyn force-pushed the riscv/register/mseccfg branch from de9e7c0 to 1537fdd Compare August 12, 2025 23:05

@romancardenas romancardenas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@romancardenas
romancardenas added this pull request to the merge queue Aug 13, 2025
Merged via the queue into rust-embedded:master with commit 00a818c Aug 13, 2025
139 checks passed
@rmsyn
rmsyn deleted the riscv/register/mseccfg branch August 14, 2025 03:07
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