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

Document FIFO properties #1538

Open
rroohhh opened this issue Oct 28, 2024 · 1 comment
Open

Document FIFO properties #1538

rroohhh opened this issue Oct 28, 2024 · 1 comment

Comments

@rroohhh
Copy link
Contributor

rroohhh commented Oct 28, 2024

It would be nice if various properties of the FIFOs would be documented. What comes to mind is

  • maximum throughput (SyncFIFO of depth == 1 only does a throughput of 50%, same for SyncFIFOBuffered with depth <=2)
  • dependencies between the interface signals (does w_stream.ready depend combinatorially on r_stream.ready?, does r_stream.valid depend combinatorially on w_stream.valid?)
@whitequark
Copy link
Member

We could do that, but I think I'm more interested in deprecating lib.fifo in favor of a stream-based lib.queue with nicer properties (e.g. higher throughput in edge cases).

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

No branches or pull requests

2 participants