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

Net types not explicitly declared, but default_nettyp set to none #55

Open
heavySea opened this issue May 5, 2021 · 1 comment
Open

Comments

@heavySea
Copy link

heavySea commented May 5, 2021

With 08cd6eb and 581068f the compiler directive ``default_nettype none` was introduced to some (not all) files, including all testbench files.
The directive requires that the type of nets of all ports in the modules must be defined.

The design of the management soc uses implicit port type declarations most of the time. Moreover the produced netlists from yosys do not include port type declarations!

Using any other simulator than Icarus Verilog, above described problem will result in "missing net type declarations" errors.

This problem is closely related to the google/skywater-pdk#198, where the use of implicit net type declarations was already discussed but without a resolution.

Due to the use of implicit declarations everywhere, I would suggest to remove the compiler directive ``default_nettype none`.

heavySea referenced this issue in heavySea/caravel May 5, 2021
Remove the compiler directives related to default port type
heavySea referenced this issue in heavySea/caravel May 5, 2021
Remove the compiler directives related to default port type
This was referenced May 5, 2021
@WallieEverest
Copy link

If we could get the PDK libraries to explicitly declare wires, then this discussion would be resolved.
For example:
change output Y;
to output wire Y;

Better yet, just migrate to the Verilog 2001 convention of fully qualified port names.

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