abs-tudelft / abs-tudelft/tydi

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

Ouverte Adaptée aux débutants
#225 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
🐬 specification
Langage dominant
Rust
Étoiles
15
Forks
5
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

## 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).

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Review the specification pages linked in the issue: 'signals', 'signal-omission', and 'complexity-c'. Compare the stated conditions for strb encoding (C≥8 vs C≥7). The fix is to update the documentation to consistently state C≥7 for individual lane validity and C≥8 for lane-based last signals. Verify no other references are inconsistent.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Domaine
documentation
Type d'issue
Documentation
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
70/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.