riscv: fix mstatus/sstatus helpers and field mask#396
riscv: fix mstatus/sstatus helpers and field mask#396romancardenas merged 1 commit intorust-embedded:masterfrom
Conversation
|
I will need some time to review this PR. Can you share with me the link to the ISA specification you used for these modifications? I usually use this: https://docs.riscv.org/reference/isa/_attachments/riscv-privileged.pdf There is a new release (from February) and looks like there is a few more fields |
I referred to this ISA Spec: https://docs.riscv.org/reference/isa/priv/supervisor.html#sstatus If there is anything I have missed, I am willing to keep improving it :) |
romancardenas
left a comment
There was a problem hiding this comment.
Looks good to me!! Please, address the changes in CHANGELOG.md and the minor docs formatting comment.
Ok, I've made the changes. |
riscv: complete mstatus and sstatus status helpers
Align the mstatus and sstatus helpers with the privileged spec.