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

'M' Property Error on Single PNP #86

Open
d-m-bailey opened this issue Jan 4, 2024 · 2 comments
Open

'M' Property Error on Single PNP #86

d-m-bailey opened this issue Jan 4, 2024 · 2 comments

Comments

@d-m-bailey
Copy link
Contributor

netgen 1.5.264

A single pnp transistor is extracted without an M parameter. Matching to a single pnp transistor in the source spice with an m parameter causes a mismatch.
Here’s the reduced layout:

X0 GND GND eg sky130_fd_pr__pnp_05v5_W0p68L0p68 M=8                               
X1 GND GND eu sky130_fd_pr__pnp_05v5_W0p68L0p68                                   

and here’s the schematic

XQ2 vss vss net2 sky130_fd_pr__pnp_05v5_W0p68L0p68 NE=1 m=8
XQ1 vss vss vn sky130_fd_pr__pnp_05v5_W0p68L0p68 NE=1 m=1

and here’s the error message

Netlists match uniquely with property errors.
Circuit 2 bandgap instance sky130_fd_pr__pnp_05v5_W0p68L0p68:Q1 property "m" has no match in circuit 1.

Currently, the NE parameter in the source is ignored in the test setup file.
If the NE parameter is removed from the source, no warning message is generated.

To duplicate, export PDK_ROOT and PDK for sky130, then

1. tar xzf test-netgen.tgz
2. cd test-netgen
3. netgen -batch source lvs.script

test-netgen.tgz

You should be able to open the schematic user_analog_project_wrapper.sch in the xschem directory.
Note that this is not the original schematic data but has been modified to (almost) match the layout.

@RTimothyEdwards
Copy link
Owner

@d-m-bailey : Thanks for the bug report and reproducible result. I discovered the reason and why I could not duplicate the issue in a simple test. Occasionally the "M" parameter gets recorded as type double (I forget the reason why), which was happening in this example, and that specific case was not being tested. It is now fixed in netgen version 1.5.265.

@d-m-bailey
Copy link
Contributor Author

Excellent!

There was another property error in the test data, but that is due to a mismatch in the dummy mosfet counts and is not an issue.

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