-
Notifications
You must be signed in to change notification settings - Fork 162
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
riscv: add mcountinhibit
module
#213
riscv: add mcountinhibit
module
#213
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR!
About returning a result/option: I think it is better if we assert
for now and discuss in #212 about transitioning to a panic-less strategy
10f4b25
to
b1acbfd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Clippy is complaining in merge queue: https://github.com/rust-embedded/riscv/actions/runs/9201547297 @rmsyn please can you update your PR? |
b1acbfd
to
6e29920
Compare
Did something change from the PR CI runs? I fixed the comment clippy was unhappy with, but it was unrelated to my code changes... |
6e29920
to
96b548d
Compare
Adds the `riscv::register::mcountinhibit` module for the `mcountinhibit` CSR.
96b548d
to
93c6d40
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
seems that nightly has new lints that previously didn't trigger
Adds the
riscv::register::mcountinhibit
module for themcountinhibit
CSR.