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

use correct csr privlevel path #2196

Merged
merged 2 commits into from
Sep 13, 2023

Conversation

silabs-krdosvik
Copy link

@silabs-krdosvik silabs-krdosvik commented Sep 13, 2023

RTL has added an if statement that needs to be included in the path.

Formal compiles
Ci check is all red
But after this bug is fixed: openhwgroup/cv32e40s#499
It is all green except clic config tests (clic and debug_test2) (which I think is a clic problem and not something caused by this change, as that would be weird)

Signed-off-by: Kristine Dosvik <[email protected]>
@silabs-mateilga
Copy link
Contributor

Looks good, but we should probably delay merging until the mentioned bug is fixed.

@silabs-krdosvik
Copy link
Author

Agreed! Now it is fixed! So would be nice to merge this asap

@silabs-hfegran
Copy link
Contributor

This requires hash update of the S as well right? that's missing from this PR

Copy link
Contributor

@silabs-hfegran silabs-hfegran left a comment

Choose a reason for hiding this comment

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

Which hash of the 40S broke this? and which hash fixes it? core-v-verif needs to be updated to match the OK hash before this PR can be merged

Signed-off-by: Kristine Dosvik <[email protected]>
@silabs-hfegran silabs-hfegran merged commit 3d660f2 into openhwgroup:cv32e40s/dev Sep 13, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants