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

Adapt for new parameterization #2090

Closed
wants to merge 2 commits into from

Conversation

zarubaf
Copy link

@zarubaf zarubaf commented Jul 28, 2023

Companion commit to openhwgroup/cva6#1321.

It deprecates the top-level ArianeCfg config struct in favor of the CVA6 one. Furthermore, the testbench now defaults to the Default config struct in the ariane_pkg. While that isn't super useful for reproducibility, this should ideally limit the changes to coer-v-verif at the moment. Once the config struct is stable a "known-good" configuration can be fixed here.

@github-actions
Copy link

❌ failed run, report available here.

1 similar comment
@github-actions
Copy link

❌ failed run, report available here.

@ASintzoff
Copy link
Contributor

@zarubaf Could you rebase your PR on latest cva6/dev commit? It is needed as the path of config_pkg_generator.py was missing in cva6.py.

Signed-off-by: Florian Zaruba <[email protected]>
@zarubaf zarubaf force-pushed the zarubaf/fix/merge-parameterization branch from bd61fa7 to ce32d96 Compare July 28, 2023 17:22
@zarubaf
Copy link
Author

zarubaf commented Jul 28, 2023

@zarubaf Could you rebase your PR on latest cva6/dev commit? It is needed as the path of config_pkg_generator.py was missing in cva6.py.

Ooops, I guess that was me when cleaning up the root dir. Re-based on cva6/dev!

@github-actions
Copy link

❌ failed run, report available here.

@zarubaf
Copy link
Author

zarubaf commented Sep 19, 2023

No longer applicable.

@zarubaf zarubaf closed this Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants