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.

Sign up for GitHub

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

Jump to bottom

Consolidate top-level CVA6 parameters into struct of parameters #744

Closed
MikeOpenHWGroup opened this issue Oct 1, 2021 · 3 comments
Closed

Consolidate top-level CVA6 parameters into struct of parameters #744

MikeOpenHWGroup opened this issue Oct 1, 2021 · 3 comments
Assignees
@zarubaf
Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory) Status:In Progress Work on this issue has started, but is not complete. Type:Enhancement For feature requests and enhancements

Comments

@MikeOpenHWGroup
Copy link
Member

This issue is related to #743.

The ariane_cfg_t struct does not "contain" all the parameters needed to fully define all CVA6 configurations. (There is even a TODO comment in the code about it.)

This issue can be closed when all parameters required to implement any of the "golden configurations" mentioned in #743 are added to ariane_cfg_t.
@MikeOpenHWGroup MikeOpenHWGroup mentioned this issue Oct 1, 2021
Open
@MikeOpenHWGroup MikeOpenHWGroup self-assigned this Feb 17, 2023
@MikeOpenHWGroup MikeOpenHWGroup added Component:RTL For issues in the RTL (e.g. for files in the rtl directory) Type:Enhancement For feature requests and enhancements Status:In Progress Work on this issue has started, but is not complete. labels Feb 17, 2023
@MikeOpenHWGroup
Copy link
Member Author

Hi @JeanRochCoulon. I believe this Issue can be closed now. Let me know if you agree and I will do it.

@JeanRochCoulon
Copy link
Contributor

This issue will be closed when #1321 will be merged. As this PR is in process, done by @zarubaf, I re-assign to him the issue

@zarubaf
Copy link
Contributor

zarubaf commented Oct 23, 2023

#1321 is merged

@zarubaf zarubaf closed this as completed Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zarubaf zarubaf

Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory) Status:In Progress Work on this issue has started, but is not complete. Type:Enhancement For feature requests and enhancements
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@zarubaf @MikeOpenHWGroup @JeanRochCoulon