IHP-Open-PDK icon indicating copy to clipboard operation
IHP-Open-PDK copied to clipboard

IO library fixes & improvements

Open FatsieFS opened this issue 9 months ago • 5 comments

This PR contains critical fix for the IOPadVdd cell. Previous gds should not be used anymore for new tape-outs.
For the rest some small improvements to the generated files and docs.

FatsieFS avatar May 13 '24 17:05 FatsieFS

@sergeiandreyev @KrzysztofHerman
Current CDL seems out of date. The IOPadVdd cell in the CDL still contains the ESD diodes and not the RC clamp.

stafverhaegen-chipflow avatar May 13 '24 17:05 stafverhaegen-chipflow

BTW, this thus contain fix for #101 There is still a difference between gds I uploaded in that issue and the current gds. The former still contained a DRC with a connection with too big width on metal2.

stafverhaegen-chipflow avatar May 13 '24 17:05 stafverhaegen-chipflow

I only now see that Boris did still find that the bulk is connected to iovdd; so wait with merging until this is clarified.

FatsieFS avatar May 13 '24 17:05 FatsieFS

Hello Staf, let's have a sync-up on IO cells with Alexey Balashov, to be all on the same page I guess we're going out of sync now..

sergeiandreyev avatar May 14 '24 07:05 sergeiandreyev

@FatsieFS , we just had a meeting with Mabrains team who are developing KLayout LVS for our OpenPDK.. could you please take a look at the slides on IO cells, if all these items are known/fixed/could be waived https://docs.google.com/presentation/d/19rNEkZFATAg9Ojm-7nIBbsLfwG7sHYNntAm3xROcy3c/edit?usp=sharing Thank you in advance!

sergeiandreyev avatar May 14 '24 13:05 sergeiandreyev

@FatsieFS IOPadVdd has missing slits on M2. Maximum width without slits is 30u.

dnltz avatar May 18 '24 09:05 dnltz

@FatsieFS IOPadVdd has missing slits on M2. Maximum width without slits is 30u.

I updated patch with fixed gds. I did fix it but seems I did not properly update the gds file in the patch. Also rebased the patch on top of latest dev branch.

stafverhaegen-chipflow avatar May 21 '24 07:05 stafverhaegen-chipflow

@stafverhaegen-chipflow just tried your new GDS and looks good to me!

image

dnltz avatar May 21 '24 08:05 dnltz

@dnltz does it solve the problem for you? Shall we merge ?

KrzysztofHerman avatar May 21 '24 09:05 KrzysztofHerman

@KrzysztofHerman Slt.c is missing in the Open PDK rules but there is no area wider then 30u anymore. Should be fine.

dnltz avatar May 21 '24 09:05 dnltz

@KrzysztofHerman
I think you can close #101 also then.
There may still be open questions about LVS as discussed in https://github.com/IHP-GmbH/IHP-Open-PDK/pull/109#issuecomment-2110303660 but from our f2f discussion I understood Alexey would take the next steps there.

stafverhaegen-chipflow avatar May 21 '24 09:05 stafverhaegen-chipflow