Skip to content

Conversation

@VMois
Copy link
Owner

@VMois VMois commented Oct 18, 2023

Related #32

VMois added 8 commits October 25, 2023 16:17
…lt mask on reset

Usage of mask by both KvTransfer and Merger shifts responsibility of knowing which buffers are excluded to Controller.
The Controller should know and handle empty buffers, whatever all data was received, etc.
KvTransfer and Merger should have limited knowledge of what is going outside.

- Adds NextIndexSelector module that selects next index based on mask value;
- Modifies KvTransfer to use NextIndexSelector;
- Allows default mask to be modified for Merger when io.reset is set.
- add additional signal metadataValid to indicate that current deq.bits value is valid metadata (it is more of a hack than long-term solution).
@VMois VMois added wontfix This will not be worked on invalid This doesn't seem right labels Nov 4, 2023
@VMois
Copy link
Owner Author

VMois commented Nov 4, 2023

Ignore this PR. I am keeping it here just in case until the thesis is ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid This doesn't seem right wontfix This will not be worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants