Skip to content
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

Fix for issue #116 #437

Merged

Conversation

silabs-oysteink
Copy link
Contributor

Added shadowcopy to mcause for both CLIC and CLINT.
Hooked up mcause_rd_error.

SEC clean.

Hooked up mcause_rd_error.

SEC clean.

Signed-off-by: Oystein Knauserud <Oystein.Knauserud@silabs.com>
@silabs-oysteink silabs-oysteink added the Component:RTL For issues in the RTL (e.g. for files in the rtl directory) label Apr 24, 2023
@silabs-oysteink
Copy link
Contributor Author

MASK for mcause will be introduced once PR #836 on E40X is merged over.

@@ -1945,7 +1946,7 @@ module cv32e40s_cs_registers import cv32e40s_pkg::*;
#(
.LIB (LIB),
.WIDTH (32),
.SHADOWCOPY (1'b0),
.SHADOWCOPY (1'b1),
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove comment on line where mcause_rd_error is declared (for me line 411)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated.

Signed-off-by: Oystein Knauserud <Oystein.Knauserud@silabs.com>
@Silabs-ArjanB Silabs-ArjanB merged commit 0ea0da9 into openhwgroup:master Apr 24, 2023
1 check passed
@Silabs-ArjanB Silabs-ArjanB mentioned this pull request Apr 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants