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

Set Verilator version v5.006 #211

Merged
merged 1 commit into from
Aug 6, 2024
Merged

Set Verilator version v5.006 #211

merged 1 commit into from
Aug 6, 2024

Conversation

rgantonio
Copy link

@rgantonio rgantonio commented Aug 6, 2024

This PR sets the Verilator version to v5.006. This is more stable as it is in accordance with what the Snitch cluster is using.

Why need this?

  • The CONVOLVE project must use the reduced AXI bandwidth to minimize routing congestion problems. The current AXI version they use is only working in v5.006 that's why they use it.

Nothing changes functionally and this is actually more "stable"

This will help PR #173 to finish building.

Please approve 😭

@rgantonio rgantonio added the clean up To keep repos clean label Aug 6, 2024
@rgantonio rgantonio self-assigned this Aug 6, 2024
Copy link

@jorendumoulin jorendumoulin left a comment

Choose a reason for hiding this comment

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

hmm okay

is there someone maintaining the AXI that fails with verilator?

@jorendumoulin
Copy link

is this https://github.com/pulp-platform/axi?

@rgantonio
Copy link
Author

is this https://github.com/pulp-platform/axi?

Yes! According to Morritz, we have to use version 0.39.1.

We have this in our bender already. Then when PR #173 was made a month ago, it was meant for the CONVOLVE project. Then now it seems we need to add the switch who will use SNAX.

We found it was a versioning problem because @xiaoling-yi pointed out it builds on hers. Then when I checked it did build.

We won't use this lower bandwidth AXI for our internal tapeout. But, for other partners, they need to.

@rgantonio
Copy link
Author

rgantonio commented Aug 6, 2024

Also thank you!!! @jorendumoulin

@rgantonio rgantonio merged commit a03725b into main Aug 6, 2024
28 checks passed
@rgantonio rgantonio deleted the ry/veri-version branch August 6, 2024 08:28
rgantonio added a commit that referenced this pull request Aug 7, 2024
rgantonio added a commit that referenced this pull request Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clean up To keep repos clean
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants