Skip to content

Add ihp-sg13cmos5l#3988

Open
dnltz wants to merge 3 commits intoThe-OpenROAD-Project:masterfrom
dnltz:WIP/dnltz/ihp-sg13cmos5l
Open

Add ihp-sg13cmos5l#3988
dnltz wants to merge 3 commits intoThe-OpenROAD-Project:masterfrom
dnltz:WIP/dnltz/ihp-sg13cmos5l

Conversation

@dnltz
Copy link
Contributor

@dnltz dnltz commented Mar 12, 2026

This technology is almost identical to SG13G2, but is missing
the HBTs and has only 5 metal layers.
It's missing Metal5 and TopMetal2 for cheaper and faster
production runs.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is this removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry, I already fixed my branch but wanted to wait for the CI results.

I moved this change into a dedicated commit. This file is actually not used and sg13g2_io.cdl is used.

@dnltz dnltz force-pushed the WIP/dnltz/ihp-sg13cmos5l branch from 779ce87 to 67c44e4 Compare March 13, 2026 05:37
@dnltz
Copy link
Contributor Author

dnltz commented Mar 13, 2026

@maliberty one nandgate45 test failed. Can you restart this test?

Can you please also enable the ihp-sg13cmos5l runs after this PR got merged?

@dnltz dnltz force-pushed the WIP/dnltz/ihp-sg13cmos5l branch from 67c44e4 to 5b485cc Compare March 16, 2026 16:13
@dnltz
Copy link
Contributor Author

dnltz commented Mar 16, 2026

Also fixed make_track offset/pitch values for this PDK. Similar to #4004

@maliberty
Copy link
Member

Probably the failure just needs a merge of master to resolve as pr-merge is already passing.

@maliberty
Copy link
Member

Are the standard cells the same (ie could we just link the lef & lib & cdl)?

@dnltz dnltz force-pushed the WIP/dnltz/ihp-sg13cmos5l branch from 5b485cc to 3f78fad Compare March 17, 2026 05:53
dnltz added 3 commits March 17, 2026 07:10
Add IHP's SG13CMOS5L PDK.

This technology is almost identical to SG13G2, but is missing
the HBTs and has only 5 metal layers.
It's missing Metal5 and TopMetal2 for cheaper and faster
production runs.

Signed-off-by: Daniel Schultz <dnltz@aesc-silicon.de>
sg13g2_iocell.cdl is not used anymore and sg13g2_io.cdl is
used instead. Remove this old .cdl file.

Signed-off-by: Daniel Schultz <dnltz@aesc-silicon.de>
Add all ihp-sg13g2 examples for the CMOS5L PDK and modify
all required parts.

Signed-off-by: Daniel Schultz <dnltz@aesc-silicon.de>
@dnltz dnltz force-pushed the WIP/dnltz/ihp-sg13cmos5l branch from 3f78fad to 72cdf2b Compare March 17, 2026 06:11
@dnltz
Copy link
Contributor Author

dnltz commented Mar 17, 2026

Are the standard cells the same (ie could we just link the lef & lib & cdl)?

no, but the sram macros are the same and I replaced them with links to the same files in the ihp-sg13g2.

Ideally we can move all SRAM macros into a common ihp-common directory. I will open an issue here soon to discuss this topic.

@dnltz
Copy link
Contributor Author

dnltz commented Mar 17, 2026

Probably the failure just needs a merge of master to resolve as pr-merge is already passing.

That did work, thanks!

@dnltz dnltz requested a review from maliberty March 17, 2026 08:38
@@ -0,0 +1 @@
cut_rows
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does this process not need tapcells?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, they don't have one. Same for the SG13G2 process.

@@ -0,0 +1,3 @@
# IHP SG13CMOS5L official port for OpenROAD-flow-scripts

to be documented
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about a link to the upstream at least?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mind if I update the README for both IHP processes afterwards? SG13G2 is also TBD...

I think there are a couple more places where ORFS could mention support for IHP.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's fine so long as you'll follow up on it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's on my list together with the DRT-0349 warning. Currently working on tape-outs, but I have more time in April.

@@ -0,0 +1,5 @@
make_tracks Metal1 -x_offset 0.0 -x_pitch 0.48 -y_offset 0.0 -y_pitch 0.42
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same concern as your other PR - this doesn't match the tech LEF

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

Successfully merging this pull request may close these issues.

2 participants