abs-tudelft / abs-tudelft/tydi

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

Abierto
#225 0 comentarios 0 reacciones 0 asignados Ver en GitHub
🐬 specification
Lenguaje dominante
Rust
Estrellas
15
Forks
5
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.