-
Notifications
You must be signed in to change notification settings - Fork 85
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
GF180MCU lef and tlef metal layers name mismatch and missing SITE definition #274
Comments
This is an error in the Google repository and should be logged as an issue at either https://github.com/google/globalfoundries-pdk-libs-gf180mcu_fd_sc_mcu7t5v0 or at https://github.com/google/globalfoundries-pdk-libs-gf180mcu. I expect Tim and Proppy would prefer if you just fix the technology LEF files and generate a pull request. Note that this error also applies to the mcu9t5v0 standard cell library. |
Please start with logging the issue. The fastest solution will be for you to send a fix pull request but maybe someone from MABrains can help fix it too? @atorkmabrains |
I logged both issues and created PRs for the metal layer names miss-match issue. I am working through the CLA for the PRs. |
I have signed an individual CLA. I think someone needs to re-trigger the CI. |
@kareefardi How can we help you? |
Fixed in google's repos |
Not quite sure that this repo is the right place for this issue.
gf180mcuC/libs.ref/gf180mcu_fd_sc_mcu7t5v0/lef/gf180mcu_fd_sc_mcu7t5v0.lef
has uppercase metal layer names whilegf180mcuC/libs.ref/gf180mcu_fd_sc_mcu7t5v0/techlef/gf180mcu_fd_sc_mcu7t5v0.tlef
has Pascal case metal layer names (METAL1
vsMetal1
). Furthermore, there are noSITE
definitions in the techlef - nor the cells lef. Attached are build logs.build_gf180mcu-2022-08-31-04-33-01.tar.gz
The text was updated successfully, but these errors were encountered: