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

Inconsistency: Strobe only encodes individual lane validity at C≥8 #225

Open
matthijsr opened this issue Mar 26, 2022 · 0 comments
Open
Labels
🐬 specification Item related to the specification

Comments

@matthijsr
Copy link

Issue

https://abs-tudelft.github.io/tydi/specification/physical.html#signals

Mentions that strb encoding individual lane validity is contingent on C≥8. This is repeated in a few other places.

However, https://abs-tudelft.github.io/tydi/specification/physical.html#signal-omission makes strb contingent on C≥7.
And https://abs-tudelft.github.io/tydi/specification/physical.html#complexity-c likewise suggests that "The indices of the active data lanes can be described with a simple range." only applies to C<7.

Assumed/Suggested Fix

strb encoding individual lane validity/activity should be contingent on C≥7. As otherwise, C=7 is identical in functionality to C=6.
Instead, the intent was most likely to have C≥7 add support for individual lane validity (a strb bit per element lane), while C≥8 adds support for individual lane-based sequence terminations (a last signal per element lane).

@mbrobbel mbrobbel added the 🐬 specification Item related to the specification label Mar 28, 2022
@mbrobbel mbrobbel changed the title [Spec] Inconsistency: Strobe only encodes individual lane validity at C≥8 Inconsistency: Strobe only encodes individual lane validity at C≥8 Mar 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐬 specification Item related to the specification
Projects
None yet
Development

No branches or pull requests

2 participants