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

Wrong commands in LEF file #106

Open
thesourcerer8 opened this issue Dec 4, 2022 · 9 comments
Open

Wrong commands in LEF file #106

thesourcerer8 opened this issue Dec 4, 2022 · 9 comments

Comments

@thesourcerer8
Copy link

Expected Behavior

The LEF files should be conforming to the fileformat specification

Actual Behavior

There are wrong commands in the LEF files:
grafik

@thesourcerer8
Copy link
Author

gf180mcu_fd_sc_mcu9t5v0__antennaDIFFAREA should be ANTENNADIFFAREA instead.

@thesourcerer8
Copy link
Author

Somehow ANTENNA was replaced by with the antenna cell name, perhaps a TCL scope variable override issue?

@QuantamHD
Copy link
Collaborator

Nice catch, we'll get a fix in.

@RTimothyEdwards
Copy link
Collaborator

This is essentially a duplicate of #104, since the same string replacement caused both.

@mithro
Copy link
Contributor

mithro commented Dec 5, 2022

I believe this is fixed by google/globalfoundries-pdk-libs-gf180mcu_fd_sc_mcu9t5v0#23

@umarcor
Copy link
Collaborator

umarcor commented Dec 5, 2022

google/globalfoundries-pdk-libs-gf180mcu_fd_sc_mcu9t5v0#23 was merged. Can this be closed?

@thesourcerer8
Copy link
Author

Can I get these changes into the gfmpw-0d Caravel branch, or will this go into a different branch (which one?), or is there another way to get this fix into a repo with gfmpw-0d ?

@QuantamHD
Copy link
Collaborator

@jeffdi @RTimothyEdwards

@RTimothyEdwards
Copy link
Collaborator

This fix has been in open_pdks since three weeks ago, and by extension should be in volare. It does not specifically "go into" the caravel repository, although the various commit numbers recorded for the library repositories should all be up-to-date.

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

No branches or pull requests

5 participants