open-source-silicon.dev

<@U016EM8L91B> is there a reason `all.spice` doesn't include `W3p40L3p40`

No, I don't.  We inherited that from SkyWater.  I wrote `sky130.lib.spice` to take care of most of the issues of files that don't get included from `all.spice`.

<@U016EM8L91B> Sorry that was a terrible attempt at a question - the PNP device in the open_pdks build of aky130A only seems to include the `W0p68L0p68` PNP device but not the `W3p40L3p40`.  Is there any particular reason for that?

i see.  Do you know of any reason that I shouldn't use it? or should I modify my design to use the one which is included?

The 0.68x0.68 and 3.4x3.4 devices got somehow very screwed up by the scripts that generated the open PDK from the SkyWater sources.  Only the 0.68x0.68 device is currently in the PDK, and it is variously referenced as either "0p68" or "3p40".  I need to sort all that out and write a patch file to fix it.

First, make sure that you're simulating with the right device.  If so, and if you need the 3.4x3.4 device, then I'll go dredge it up from the SkyWater sources and write the patch file.

I can probably switch to the 0.68 device without too much trouble. Do you think that would be the safest option?

I honestly have no idea.  I can't let the repository sit in the state of having mixed-up references to the two devices, so I need to do something about it.

Ok thanks. Is that something you're planning on doing in time for this shuttle?

<@U01H2JMLFLL>:  Yes, and I just updated open_pdks with a correction a few minutes ago (to the <http://opencircuitdesign.com|opencircuitdesign.com> repository, as usual, with the github mirror updating in about 6 hours).  With the update, you should be able to use either the 0.68x0.68 or the 3.4x3.4 devices, and the views should all be consistent.

Thanks <@U016EM8L91B>!! I'm pulling it in right now to give it a try

<@U016EM8L91B>, just to follow up on this - even with the latest version of open_pdks, in order to simulate with the `pnp_05v5_W3p40L3p40` I've had to add the line
```.include "../../libs.ref/sky130_fd_pr/spice/sky130_fd_pr__pnp_05v5_W3p40L3p40.model.spice"```
Under the line:
```.include "../../libs.ref/sky130_fd_pr/spice/sky130_fd_pr__pnp_05v5_W0p68L0p68.model.spice"```
to `all.spice`.  Did you leave it out while making the changes or is there a different way you intend for me to be including the model?

Don't include `all.spice`.  Instead, you should be using `.lib &lt;path_to&gt;/sky130A/libs.tech/ngspice/sky130.lib.spice &lt;corner&gt;`.  The `sky130.lib.spice` file is what pulls in all of the models (`all` is not all. . .).

<@U016EM8L91B> so I am actually using `sky130A/libs.tech/ngspice/sky130.lib.spice` as you say and that includes `all.spice` but the 3.4 PNP device model is not included

Oh, okay.  I'll make a patch for the lib.spice file.