- Notifications
You must be signed in to change notification settings - Fork110
Journey through custom setup of SKY130#842
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
-
Hi there ! I installed the PDK throughRTimothyEdwards/open_pdks and modified a copy of skywater130.py to point on the new location. The first issue is that openroad seems to need a single lef file merged containing the standard cell library lef file and the tlef content. As of today, specifying the tlef file and the lef file from the PDK, as one (me) would expect, without edition through those lines : # APR tech filechip.set('pdk','aprtech',stackup,libtype,'lef',os.path.join(sclib_dir_ref,"techlef",f"{sclib_name}.tlef"))# ...# lefchip.add('library',libname,'lef',os.path.join(sclib_dir_ref,'lef',f'{sclib_name}.lef')) produce, at floorplan step, in openroad,
Followed by a lot of This might be an issue on OpenROAD side though. For reference, find attached my PDK file:csky130hd.py.txt . |
BetaWas this translation helpful?Give feedback.
All reactions
Replies: 3 comments 16 replies
-
I thought multi-lef issue was fixed a while back in openroad? What version of openroad are you using?
|
BetaWas this translation helpful?Give feedback.
All reactions
-
A git pull of siliconcompiler ? I've done it and it seems that it doesn't fix the issue. # APR tech filefortoolin ('openroad','klayout','magic'):chip.set('pdk','aprtech',tool,stackup,libtype,'lef',os.path.join(sclib_dir_ref,"techlef",f"{sclib_name}.tlef")) for SKY130 which should match the content of skywater130 which is : # Tech filefortoolin ('openroad','klayout','magic'):chip.set('pdk','aprtech',tool,stackup,libtype,'lef',pdkdir+'/apr/sky130_fd_sc_hd.tlef') I don't really know about techlef format thoug, it comes from SKY130 master too. |
BetaWas this translation helpful?Give feedback.
All reactions
-
Yes, "a git pull from siliconcompiler." Now that you are up to date, have you done a diff between the tlef that we have included in our repo? Another possibility, are you running this in an active openroad prompt. Hard to debug like this, so just offering possible some possibilities... |
BetaWas this translation helpful?Give feedback.
All reactions
-
I'm running it from siliconcompiler only, so no problem on this end. |
BetaWas this translation helpful?Give feedback.
All reactions
-
So, I did the diff and the only difference is that my version got this addition :
Which should probably not cause any major issue at this point. |
BetaWas this translation helpful?Give feedback.
All reactions
-
By the way, I get |
BetaWas this translation helpful?Give feedback.
All reactions
-
For some reason I get the |
BetaWas this translation helpful?Give feedback.
All reactions
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
-
On this subject, it seems that the Moreover, if the file path is relative, it must start with |
BetaWas this translation helpful?Give feedback.
All reactions
-
Thanks for sharing these additional details -- I think one thing that would be helpful is if you could zip/tar your build directory after a run and share that with us? I particularly want to take a look at the manifest files (*.pkg.json) dumped by SC, that would help me debug the problem. |
BetaWas this translation helpful?Give feedback.
All reactions
-
Thank you! I think it would also be helpful to see copies of the following files:
My best guess right now is that this is something subtle to do with differences in LEF content. I don't believe OpenROAD needs the tlef and library data merged, rather I think the "merged" in the filename in our case refers to a merged file containing all std cell definitions. We took our LEF files from theOpenROAD-flow-scripts repo, rather than directly from the open PDKs repo (so we were not responsible for the preprocessing). |
BetaWas this translation helpful?Give feedback.
All reactions
-
I'll send them to you, just for information, merged is the merging of the .tlef + the .lef using mergeLef tool from the openroad utils. The cleaned was an unsuccessful attempt and is not meaningful. |
BetaWas this translation helpful?Give feedback.
All reactions
-
Please find all mentioned files in this archive :sky130_fd_sc_hd_lef.tar.gz |
BetaWas this translation helpful?Give feedback.
All reactions
-
@suzizecat I am hoping this is a dead thread after 3 years, but just checking. A lot has happened with SC since 2022. |
BetaWas this translation helpful?Give feedback.
All reactions
-
Hi@aolofsson , It is indeed a dead thread (mainly because it was an experiment more than anything, so I went on other experiments) but IIRC I never quite fixed those issues in the end. I'll give another spin to SC when I'll have a suitable project to play around with 🙂 |
BetaWas this translation helpful?Give feedback.