Mem seqRead specification out of date

[sbeamer]

The description for the seqRead in the Mem section (9) is out of date. The current text describes using the output register for seqRead.

Chisel currently leaves the output alone and will attempt to incorporate the input address register. This difference should be changed in the manual. Perhaps a better name for the seqRead flag might be retimeAddress?

[schoeberl]

Very good. That fits for memories available in current FPGAs: you need to use a register in the input (address, data, wren). The output register is optional.

Cheers, Martin

On 3 Feb, 2015, at 22:49, Scott Beamer [email protected] wrote:

The description for the seqRead in the Mem section (9) is out of date. The current text describes using the output register for seqRead.

Chisel currently leaves the output alone and will attempt to incorporate the input address register. This difference should be changed in the manual. Perhaps a better name for the seqRead flag might be retimeAddress?

— Reply to this email directly or view it on GitHub.