-
Notifications
You must be signed in to change notification settings - Fork 32
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
Deleted incorrect statement about ASR needed for all M and S mode CSRs #462
Conversation
Thinking about it, it might be better to amend the statements as follows:
|
Also, this statement is not true any longer for unprivileged CSRs anymore: riscv-cheri/src/riscv-integration.adoc Line 1054 in 12c5565
I propose: |
I think the definiton of ASR-perm is correct - so maybe the PR is ok as it is? |
…n.adoc Co-authored-by: Tariq Kurd <tariqandlaura@gmail.com> Signed-off-by: Franz Fuchs <francislaus@users.noreply.github.com>
Co-authored-by: Tariq Kurd <tariqandlaura@gmail.com> Signed-off-by: Franz Fuchs <francislaus@users.noreply.github.com>
The
mtidc
andstidc
registers do not requireASR
permission when read. This makes the deleted statements wrong.