Skip to content
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

incorrect path to sky130 klayout lyp file #332

Open
proppy opened this issue Dec 21, 2022 · 2 comments
Open

incorrect path to sky130 klayout lyp file #332

proppy opened this issue Dec 21, 2022 · 2 comments

Comments

@proppy
Copy link
Contributor

proppy commented Dec 21, 2022

Currently open_pdks installs technology file in the current directory layout:

/home/proppy/.volare/sky130A/libs.tech/klayout
/home/proppy/.volare/sky130A/libs.tech/klayout/drc
/home/proppy/.volare/sky130A/libs.tech/klayout/scripts
/home/proppy/.volare/sky130A/libs.tech/klayout/tech
/home/proppy/.volare/sky130A/libs.tech/klayout/tech/sky130A.lyp
/home/proppy/.volare/sky130A/libs.tech/klayout/tech/sky130A.lyt
/home/proppy/.volare/sky130A/libs.tech/klayout/lvs
/home/proppy/.volare/sky130A/libs.tech/klayout/pymacros
/home/proppy/.volare/sky130A/libs.tech/klayout/macros
/home/proppy/.volare/sky130A/libs.tech/klayout/ruby
/home/proppy/.volare/sky130A/libs.tech/klayout/python
/home/proppy/.volare/sky130A/libs.tech/klayout/klayoutrc

which is slightly different from the one in https://github.com/mabrains/sky130_klayout_pdk

This causes reference to the lyp file not be resolved my initializing klayout with technology:

$ KLAYOUT_HOME=$PDK_ROOT/sky130A/libs.tech/klayout klayout -e
...
Warning: Initialization of layers failed: Unable to open file: /home/proppy/.volare/sky130A/libs.tech/klayout/tech/sky130/sky130A.lyp (errno=2)

One can fix the reference to the lyt file to drop the sky130 directory:
https://github.com/mabrains/sky130_klayout_pdk/blob/main/sky130_tech/tech/sky130/sky130.lyt#L7-L9
in order to workaround this issue:

 <base-path>$(appdata_path)/tech</base-path>
 <original-base-path>$(appdata_path)/tech</original-base-path>
 <layer-properties_file>sky130A.lyp</layer-properties_file>
@xobs
Copy link

xobs commented Jan 10, 2023

Which one is correct? sky130 and gf180 are definitely different, as evidenced by the fact that klayout doesn't work with gf180: The-OpenROAD-Project/OpenLane#1593

@donn donn mentioned this issue Jan 10, 2023
@proppy proppy changed the title incorrect path to klayout lyp file incorrect path to sky130 klayout lyp file Feb 2, 2023
@proppy
Copy link
Contributor Author

proppy commented Jul 12, 2023

another error related to this is klayout failing to load the lyp file:

Warning: Initialization of layers failed: Unable to open file: /home/proppy/mixed-signal-sky130a-env/share/pdk/sky130A/libs.tech/klayout/tech//sky130.lyp (errno=2)

I think this is due to the installation renaming the file from being named after the TECH (sky130) to being named after the pdk variant sky130A:

cp ${KLAYOUT_PATH}/sky130_tech/tech/sky130/${TECH}.lyt ${KLAYOUT_STAGING_$*}/tech/${SKY130$*}.lyt ; \

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 a pull request may close this issue.

2 participants