hw: Add multicast support #32
Draft
Conversation
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
Lore0599
force-pushed
the
lleone/multicast
branch
from
cdb7a01 to
5310b87
Compare
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Lore0599
force-pushed
the
lleone/multicast
branch
from
4dcb88d to
74d21ed
Compare
hw/picobello_pkg.sv
Outdated
Comment on lines
196
to
197
| $write(" mask_y: {offset: %0d, len: %0d} },", | ||
| sam_multicast[i].idx.mask_y.offset, |
There was a problem hiding this comment.
[verible-verilog-format] reported by reviewdog 🐶
Suggested change
| $write(" mask_y: {offset: %0d, len: %0d} },", | |
| sam_multicast[i].idx.mask_y.offset, | |
| $write(" mask_y: {offset: %0d, len: %0d} },", sam_multicast[i].idx.mask_y.offset, |
Lore0599
force-pushed
the
lleone/multicast
branch
from
74d21ed to
8ec022a
Compare
Comment on lines
+196
to
+197
| $write(" mask_y: {offset: %0d, len: %0d} },", | ||
| SamMcast[i].idx.mask_y.offset, |
There was a problem hiding this comment.
[verible-verilog-format] reported by reviewdog 🐶
Suggested change
| $write(" mask_y: {offset: %0d, len: %0d} },", | |
| SamMcast[i].idx.mask_y.offset, | |
| $write(" mask_y: {offset: %0d, len: %0d} },", SamMcast[i].idx.mask_y.offset, |
| typedef logic [aw_bt'(AxiCfgN.AddrWidth)-1:0] user_mask_t; | ||
|
|
||
| typedef struct packed { | ||
| user_mask_t mcast_mask; |
There was a problem hiding this comment.
[verible-verilog-format] reported by reviewdog 🐶
Suggested change
| user_mask_t mcast_mask; | |
| user_mask_t mcast_mask; |
Lore0599
force-pushed
the
lleone/multicast
branch
from
233f542 to
41167ad
Compare
Lore0599
force-pushed
the
lleone/multicast
branch
from
41167ad to
7220afe
Compare
Lore0599
force-pushed
the
lleone/multicast
branch
from
53ff190 to
63545f0
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Collective Communication: Multicast
This PR introduces multicast capabilities to the Picobello SoC. The feature is not yet tested, but backward compatibility with the unicast version is ensured for the
mini_picobelloconfiguration and for this reason could be already merged for preliminary backend trials.🆕 ADDED:
floo_noc: now points to the multicast branch.snitch_cluster: points to a local multicast integration branch.🔜 Both branches will be merged into their respective main branches soon.
mcast_snitch_cluster.hjson: A temporary multicast-specific configuration file for Snitch Cluster.🔜 This file will be removed and merged with the generic configuration once multicast is extended to the full system (7x4 mesh).
picobello_pkg: Added helper functions to update the SAM with multicast-specific information and introduced debug-friendly printing functions to visualize the SAM.picobello_top: Cluster tiles now support multicast while non-cluster tiles (cheshire and memory tile) do not support the feature.📋 TODO: