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

Implicit wires and default_nettype #2

Closed
olofk opened this issue May 5, 2021 · 2 comments
Closed

Implicit wires and default_nettype #2

olofk opened this issue May 5, 2021 · 2 comments

Comments

@olofk
Copy link

olofk commented May 5, 2021

Working on packaging caravel-lite with FuseSoC with first objective to run the caravel tests with FuseSoC using different simulators. Currently, only icarus works since other simulators require module ports to explciitly specify wire in addition to direction when used with defualt_netttype none. Modelsim is also not too happy about mixing port styles (although there is a -mixedansiports that helps to some extent.

I have started to manually change to explicit wires in all the verilog files and got some simulations running with both xsim and modelsim but before doing too much work I would just like to ask

  1. Is this repo intended to replace caravel or is my efforts better spent on the caravel repo?
  2. Are you open to accept such a change?
@Manarabdelaty
Copy link
Contributor

  1. This repo is automatically updated when we push to caravel master, so no development is done here.
  2. I think this PR seems relevant to your issue update gpio control block caravel#57

@olofk
Copy link
Author

olofk commented May 5, 2021

Great. Thanks for the info. Yes, this is very much related. I'll close this and look into the linked issue

@olofk olofk closed this as completed May 5, 2021
Manarabdelaty pushed a commit that referenced this issue May 8, 2021
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