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

cli: add new show backend which shows a diagram of the generated RTL #627

Closed
wants to merge 1 commit into from

Conversation

hansfbaier
Copy link

Hi,
I added a new backend to show diagrams of the RTL represented by the nmigen code.
Please let me know, what you think and what you would like to have changed.

Kind regards,
Hans

@whitequark
Copy link
Member

whitequark commented Aug 17, 2021

Honestly, I think the entire cli module should be deprecated and reworked. It was quickly put together to make the examples work, and it doesn't have much of a design. Extending it is something I'd like to avoid until it is reworked.

@hansfbaier
Copy link
Author

Yes, it's a bit tedious to use.
It would be really cool, if one could just hand it the class name of the Elaboratable,
it would scan it via reflection and instantiate it on its own by default, so that no
extra code would be needed to do the basic tasks like generate/simulate/show.
Anyway, thanks for taking the time.

@hansfbaier
Copy link
Author

I think I found a more elegant solution.
A commandline tool which already can handle show, and also will handle generation and other common tasks.
https://github.com/hansfbaier/nmigen-tool

@hansfbaier hansfbaier closed this Aug 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants