Skip to content

LEF obstruction for nwell seems incomplete #300

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

Open
smunaut opened this issue Apr 7, 2024 · 5 comments
Open

LEF obstruction for nwell seems incomplete #300

smunaut opened this issue Apr 7, 2024 · 5 comments

Comments

@smunaut
Copy link
Contributor

smunaut commented Apr 7, 2024

So this is originally from a .mag

If you compare the generated GDS:
gds-view

And then the generated LEF:
lef-view

One of the large N-Well obstruction is missing from the LEF.

(Those all comes from PCell subcells in the original MAG. On the top it's two PCell overlapping their guardring).

@smunaut
Copy link
Contributor Author

smunaut commented Apr 7, 2024

Or maybe it's omitted on purpose because it's far away from the borders ?

@RTimothyEdwards
Copy link
Owner

I'm not sure why it's missing the nwell shape. I don't see a prBoundary layer; if it doesn't exist, then magic will take the extents of geometry as the bounding box, and should include everything inside that bounding box in the LEF file.

@smunaut
Copy link
Contributor Author

smunaut commented Apr 7, 2024

There is a prBoundary, it definitely englobes all the shapes shown.

This is the design btw : https://github.com/TinyTapeout/tt_asw_1v8/blob/main/mag/tt_asw_1v8.mag

@RTimothyEdwards
Copy link
Owner

Thanks for the link to the layout. That's clearly wrong, and I'll need to investigate.

@RTimothyEdwards
Copy link
Owner

@smunaut : I found out that "lef write" has an inconsistent policy about MASTERSLICE layers. The nwell that got found was part of the routine checking for stuff outside the prBoundary, which includes MASTERSLICE layers. But the routine that fills in the center area when using the "-hide" option does not include MASTERSLICE layers. After thinking it over, I decided to allow the center area to be filled with nwell. That might be a bad solution, though, because it implies that there is nwell in areas of the cell where there isn't any. So I have provisionally implemented that solution, but I am still thinking about it.

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