Merged
Conversation
Lore0599
force-pushed
the
bump_snitch
branch
from
2e43f2d to
e63ae9e
Compare
Lore0599
force-pushed
the
bump_snitch
branch
from
e63ae9e to
0bd275a
Compare
colluca
reviewed
Feb 18, 2026
colluca
left a comment
colluca
left a comment
There was a problem hiding this comment.
Just a small comment, otherwise I think the Snitch integration is solid
Xeratec
reviewed
Feb 20, 2026
Comment on lines
+20
to
+23
| revision: 8e04779f341eb2c89412aae92223a292beef487e | ||
| version: null | ||
| source: | ||
| Git: https://github.com/pulp-platform/axi.git | ||
| Git: https://github.com/colluca/axi |
Member
There was a problem hiding this comment.
@colluca Is there a plan to upstream the axi multicast branch? I think we should try to avoid using forks in our upstream repos if possible.
There was a problem hiding this comment.
Yes, this is the famous multicast PR (pulp-platform/axi#398). Ball is in Michael's court again since a few weeks, will ping him again.
| dependencies: | ||
| register_interface: { git: "https://github.com/pulp-platform/register_interface.git", version: 0.4.3 } | ||
| axi: { git: "https://github.com/pulp-platform/axi.git", version: 0.39.2 } | ||
| axi: { git: https://github.com/colluca/axi, rev: multicast } |
Member
There was a problem hiding this comment.
colluca
approved these changes
Feb 25, 2026
colluca
left a comment
colluca
left a comment
There was a problem hiding this comment.
I think the new Snitch integration is good now :)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bump Snitch Cluster
This PR bumps the Snitch cluster, following the update performed in #81 (@Paolo309).
It also updates the integration of the Snitch cluster within the top-level make flow.
Main Changes
Dependencies
Bender.ymland corresponding.lockfiles to align with upstream dependencies.axito https://github.com/colluca/axi (branchmulticast).chimera_cluster.svMake flow
snitch-hw-inittosn-hw-all, consistent with the upstream Snitch cluster documentation.chs-sim-alltarget intochim-sim.Python environment
pyproject.tomlfile to manage Python dependencies.iis-envto0.29.1.