platforms: ihp-sg13g2: Fix track pitch#4004
Open
dnltz wants to merge 2 commits intoThe-OpenROAD-Project:masterfrom
Open
platforms: ihp-sg13g2: Fix track pitch#4004dnltz wants to merge 2 commits intoThe-OpenROAD-Project:masterfrom
dnltz wants to merge 2 commits intoThe-OpenROAD-Project:masterfrom
Conversation
Contributor
Author
|
This is related to following issue: IHP-GmbH/IHP-Open-PDK#734 |
maliberty
reviewed
Mar 17, 2026
Comment on lines
+1
to
+2
| make_tracks Metal1 -x_offset 0.0 -x_pitch 0.48 -y_offset 0.0 -y_pitch 0.42 | ||
| make_tracks Metal2 -x_offset 0.0 -x_pitch 0.48 -y_offset 0.0 -y_pitch 0.42 |
Member
There was a problem hiding this comment.
The tech lef has a single pitch per layer. Why is this changed to differ by direction?
Contributor
Author
There was a problem hiding this comment.
My understand is that LEF only supports one pitch per layer. The LEF file matches the minimum pitch per direction.
IHP is using 0.48 x 3.76 for standardcell placement. So, with this change all tracks align with the standard cell placement.
Contributor
Author
There was a problem hiding this comment.
Thanks! I will update IHP's PDK first and sync ORFS afterwards.
Contributor
Author
There was a problem hiding this comment.
dnltz
force-pushed
the
WIP/dnltz/ihp-sg13g2-fix-make-tracks
branch
from
8168483 to
f295162
Compare
The routing directions changed some time ago in the IHP PDK. Additionally, this file uses out-dated pitch values for M1-4 and wrong values for M5+TM1. Signed-off-by: Daniel Schultz <dnltz@aesc-silicon.de>
Update detailedroute__antenna_diodes_count to match latest test. Signed-off-by: Daniel Schultz <dnltz@aesc-silicon.de>
dnltz
force-pushed
the
WIP/dnltz/ihp-sg13g2-fix-make-tracks
branch
from
f295162 to
5a5f586
Compare
Contributor
Author
|
I changed the TopMetal1 pitch back to 3.28 since that's the correct value. |
Member
|
I think this needs to wait for the upstream discussion to resolve. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The routing directions changed some time ago in the IHP PDK. Additionally, this file uses out-dated pitch values for M1-4 and wrong values for M5+TM1.