-
Notifications
You must be signed in to change notification settings - Fork 667
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
Update NoC example config to match new PRCI organization #1509
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MultiNoCConfig is fixed, but SharedNoCConfig results in the same error.
I'm not able to reproduce an error for SharedNoC... can you post the error? |
Error looks like this. Maybe there is some scala-level recursion problem in constellation when computing routing tables? |
Does it always fail like this? This looks like a bug in the chisel QMC minimizer. |
I'll merge this for now |
Yeah, it fails like this... Okay sounds good |
@abejgonzalez Can you try building the |
I can successfully run both configs to Verilog generation with this PR merged in. |
Hmm.. thanks! |
NoC node mapping needs to be updated to match the new PRCI bus organization.
Related PRs / Issues:
Type of change:
Impact:
Contributor Checklist:
main
as the base branch?changelog:<topic>
label?changelog:
label?.conda-lock.yml
file if you updated the conda requirements file?Please Backport
?