-
Notifications
You must be signed in to change notification settings - Fork 49
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
Move register layout definitions into wavedrom #414
Labels
v1.0
resolve for 1.0
Comments
Can you point me at a CSR diagram from the Privileged spec that you would expect the CSRs here to look like? |
Maybe mnstatus from the rnmi chapter 8.3? |
I think we are open to whatever is the preferred presentation by the documentation team. I was looking at the privileged spec itself, i.e. definition of machine mode. |
looks ok from my side |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The privileged spec defines the CSR layouts using wavedrom. The CLIC spec doesn't. This should be aligned.
The layout diagrams do not include descriptions, while the CLIC tables do.
Proposed Resolution:
Move CLIC CSR layouts to wavedrom. Merge the description text with from the layout table with the rest. There might be redundancies, in which the text that doesn't come from the table should take presedence.
The text was updated successfully, but these errors were encountered: