I've been getting the following message when hardening user_project_wrapper: `Result: The top level ...

Brandon Ong

12/27/2021, 3:11 PM

I've been getting the following message when hardening user_project_wrapper:

Result: The top level cell failed pin matching.

vccd1 and vssd1 pins are not being connected.

Brandon Ong

12/27/2021, 3:12 PM

Here's the link to my repo: https://github.com/BrandonOng22/caravel_user_project.git. I'm in the flash branch

Matt Venn

12/27/2021, 3:31 PM

can you post a picture of the user project wrapper gds that shows the PDN over your macro?

Matt Venn

12/27/2021, 3:31 PM

some people have been having problems with the PDN shorting which leads to LVS issues

Brandon Ong

12/27/2021, 3:37 PM

Screen Shot 2021-12-27 at 5.36.24 PM.png

Matt Venn

12/27/2021, 3:43 PM

normally I see more of a frame around the macros with the pins around the edge.

Matt Venn

12/27/2021, 3:43 PM

is the vertical met4 line getting interrupted by the macro or does it continue through it? I can't tell

Matt Venn

12/27/2021, 3:44 PM

here's an example

Matt Venn

12/27/2021, 3:44 PM

image.png

Matt Venn

12/27/2021, 3:44 PM

see the vertical met4 of the pdn, is interrupted for the macro

Brandon Ong

12/27/2021, 3:45 PM

I see what you mean, the m4 line continues through

Matt Venn

12/27/2021, 3:47 PM

I think this is a problem with the caravel repository

Matt Venn

12/27/2021, 3:47 PM

see this thread: https://skywater-pdk.slack.com/archives/C017HPHCMEY/p1640343090384300

Brandon Ong

12/27/2021, 4:22 PM

I tried changing both caravel and openlane to mpw-3 and it still didn't work. I think I'm getting a different error:

(no matching pin)                          |vccd1

Brandon Ong

12/27/2021, 4:23 PM

The other thread's problem is mismatched pins, mine looks like there isn't a pin connected at all

Matt Venn

12/27/2021, 4:23 PM

if you have shorts, then the output of the lvs check could be either

Matt Venn

12/27/2021, 4:24 PM

also, you definitely want to be on mpw-3a of openlane

Matt Venn

12/27/2021, 4:24 PM

that has the fixes for the setup/hold problems affecting mpw1 and 2

Brandon Ong

12/27/2021, 4:25 PM

I was on mpw3-a originally, so I'll switch back

Matt Venn

12/27/2021, 4:30 PM

try this version of the caravel submodule

Matt Venn

12/27/2021, 4:31 PM

5712871d27c08900d18edc72a7f534cc8be1b2dd

Brandon Ong

12/27/2021, 5:08 PM

Unfortunately that version didn't change anything

Mitch Bailey

12/27/2021, 5:24 PM

If you can find the extraction log, it might indicate that you have a short. Look for messages like

Copy code

Warning:  Ports "vccd1" and "vssd1" are electrically shorted.

Matt Venn

12/27/2021, 6:05 PM

@User I've just setup a github repo that uses github actions to install the version of the tools I'm using: https://github.com/mattvenn/project0_test

Brandon Ong

12/28/2021, 2:12 PM

Thanks @User . I tried using your tools but the problem still persists

Matt Venn

12/28/2021, 2:21 PM

If I get time I'll try here

Matt Venn

12/28/2021, 2:22 PM

Pls make sure your GitHub repo is up to date with your local files

Brandon Ong

12/28/2021, 2:48 PM

Thank you. I've updated the flash branch of my repo

Brandon Ong

12/28/2021, 2:48 PM

https://github.com/BrandonOng22/caravel_user_project.git

Matt Venn

12/28/2021, 5:29 PM

I don't know why there is no cutout on PDN for your design

Matt Venn

12/28/2021, 5:29 PM

maybe becaus it's so small?

Matt Venn

12/28/2021, 5:29 PM

anyway, I moved it to the left by 23 microns so it fits in the gap

Matt Venn

12/28/2021, 5:29 PM

but there are still lvs issue

Matt Venn

12/28/2021, 5:30 PM

image.png

Matt Venn

12/28/2021, 5:30 PM

but I wonder if that's actually something up with your design

Brandon Ong

12/28/2021, 5:35 PM

I'm not sure what could be causing that. I have simulated this on my own. I did hand design my macro using layers that aren't typically used in std cells, so I wonder if that is causing any problems

Brandon Ong

12/28/2021, 5:35 PM

Thanks for taking a look

Matt Venn

12/28/2021, 5:38 PM

maybe ask in the #analog-design channel for feedback on the gds of the flash macro?

Mitch Bailey

12/28/2021, 10:30 PM

What version of magic is everyone using? There were recently some changes to substrate extraction.

Brandon Ong

01/12/2022, 2:55 PM

@User I'm using magic version 8.3.209

Mitch Bailey

01/12/2022, 3:40 PM

I'm on 8.3.253. Do you know how to update?

Brandon Ong

01/12/2022, 3:53 PM

I'll try updating

Brandon Ong

01/12/2022, 4:00 PM

Unfortunately I get the same result

Mitch Bailey

01/13/2022, 1:28 AM

@User cloned your repo. Is the repo up to date? What branch are you using?

Brandon Ong

01/13/2022, 1:54 AM

I just updated the repo. I'm on the 'flash' branch. The bug I'm trying to figure out is why GND and PSUB are shorted together

Brandon Ong

01/13/2022, 1:54 AM

Thanks for taking a look

Mitch Bailey

01/13/2022, 8:04 AM

Is this the right file?

Copy code

bash-4.2$ cksum ../gds/flash_array_8x8.gds 
1430410556 39954 ../gds/flash_array_8x8.gds

From your screenshot, it looks like you're extracting

user_project_wrapper

, but the this file does not contain

flash_array_8x8

Copy code

bash-4.2$ cksum ../gds/user_project_wrapper.gds 
3200867709 9776126 ../gds/user_project_wrapper.gds

Brandon Ong

01/13/2022, 3:10 PM

Since it exits after an error, user_project_wrapper isn't edited in the gds folder. I've been looking at the following file:

caravel_user_project/openlane/user_project_wrapper_runs/user_project_wrapper/results/klayout/user_project_wrapper.gds

Mitch Bailey

01/13/2022, 5:43 PM

Ok, I'll try to run openlane to recreate it then.

Mitch Bailey

01/13/2022, 5:47 PM

I get the following

Copy code

Error: sta.tcl, 128 can't read "::env(CLOCK_PORT)": no such variable

which may be caused by a newer (or older) version of openroad. No clocks in your design, right?

Brandon Ong

01/13/2022, 5:49 PM

Right, no clocks

Brandon Ong

01/13/2022, 5:50 PM

I'm on the mpw-3a tag of open lane

Mitch Bailey

01/13/2022, 9:57 PM

Can you manually copy

caravel_user_project/openlane/user_project_wrapper_runs/user_project_wrapper/results/klayout/user_project_wrapper.gds

caravel_user_project/gds/user_project_wrapper.gds

, compress, and push to the repo?

Brandon Ong

01/13/2022, 10:06 PM

I just pushed. fyi I also changed the name of PSUB to VBPW

Mitch Bailey

01/13/2022, 10:49 PM

With magic 8.3.255 extraction, I get several of the following messages.

Copy code

box 230480 336398 230686 336424
feedback add "Illegal overlap between nwell and pwell (types do not connect)" medium

Open in Slack

Previous Next

What is Linen?

Linen is a search-engine friendly community platform. We offer integrations with existing Slack/Discord communities and make those conversations Google-searchable.