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

allow different module names for verilog export #176

Closed
mattvenn opened this issue Jun 11, 2020 · 4 comments
Closed

allow different module names for verilog export #176

mattvenn opened this issue Jun 11, 2020 · 4 comments

Comments

@mattvenn
Copy link

currently the module is always exported with name top, which tends to interfere with the traditional name of the top level module in any existing project.

would be good to have a box to type the name of the desired module.

@chipmuenk
Copy link
Owner

You're right, "filter" or a user supplied name would be better. Nice that someone is using that feature :-)
Currently, Verilog export is more a proof of concept (not sure if it makes sense to generate a netlist for a FIR filter when most FPGA tools have optimized FIR Cores that just need the coefficients). At the moment, we are working on a (n)Migen implementation for IIR filters, I'll add a text entry for the module name then.

@mattvenn
Copy link
Author

Hi,
yes I've been experimenting with it recently. I'm using it with the open source toolchain, and we don't have FIR implementations (as far as I know), so this tool is actually really interesting for that.
Thanks,
Matt

@chipmuenk
Copy link
Owner

In the master branch, the top level module name now is derived from the Verilog file name (lower cased and sanitized i.e. removing all non-alphanumeric characters. Please check whether this works for you, it will also be part of v0.4.0 that will be released in the next 1 ... 2 weeks.

@mattvenn
Copy link
Author

mattvenn commented Sep 28, 2020 via email

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

No branches or pull requests

2 participants