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

[enhancement] Specify I/O options per-channel #1384

Open
hongted opened this issue Apr 18, 2024 · 0 comments
Open

[enhancement] Specify I/O options per-channel #1384

hongted opened this issue Apr 18, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@hongted
Copy link
Collaborator

hongted commented Apr 18, 2024

What's hard to do? (limit 100 words)

Currently I/O options for channels are global and specified as side-band options. https://google.github.io/xls/codegen_options

This is too generic and often we would like to specify the options differently (ex. loop-back channels should not be flopped) while X output channel should have a fifo depth of 5, and Z output channel should have a fifo depth of 4.

Current best alternative workaround (limit 100 words)

Right now, the option is to have no I/O options and insert the additional options manually in Verilog.

Your view of the "best case XLS enhancement" (limit 100 words)

Two options are considered

  1. Enhance DSLX to support tags/configurations on each channel. (ex. this channel should have fifo depth of 5, or this channel should have no flops added).
  2. Enhance codegen options to specify options per-channel.
@hongted hongted added the enhancement New feature or request label Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant