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

klayout import def warns about "Cannot determine routing layers for via" #1457

Closed
stefanottili opened this issue Aug 10, 2023 · 6 comments
Closed
Labels
Milestone

Comments

@stefanottili
Copy link

stefanottili commented Aug 10, 2023

Warning: Cannot determine routing layers for via: turnmet3 (line=5, cell=, file=turn_via.def)
Warning: Cannot determine routing layers for via: via23 (line=9, cell=, file=turn_via.def)
Warning: Cannot determine routing layers for via: via12 (line=13, cell=, file=turn_via.def)

The lef contains TYPE CUT/TYPE ROUTING for the via/metal layers.
The import looks ok because there are no path segments following the via references anyways.
So why the warnings ?

The order of the layers in a via definition is not guaranteed, so should be derived from the lef layer definitions.

test_turn_via.tgz

@stefanottili
Copy link
Author

The “turn” via is something that seems to be not used any more.
The lef ref states that a via has to have two routing and one cut layer.

@stefanottili
Copy link
Author

strm2oas --lefdef-lefs sky130_fd_sc_hd.tlef turn_via.def a.oas
works as expected, no warnings, turn-via shows up.
Using klayout to read lef/def the turn-via is missing.

@klayoutmatthias
Copy link
Collaborator

I missed that on 0.28.12. Will try to do it in the next minor release.

@klayoutmatthias klayoutmatthias added this to the 0.28.13 milestone Sep 13, 2023
@klayoutmatthias
Copy link
Collaborator

klayoutmatthias commented Oct 6, 2023

I only get this warning if the LEF is not seen.

In the "strm2oas --lefdef-lefs sky130_fd_sc_hd.tlef turn_via.def a.oas" the LEF is explicitly given. When I load with KLayout, by default all LEFs beside the DEF are read, but KLayout checks for *.lef, not *.tlef and this warning happens. It disappears when I add the *.tlef file to the reader options.

Is "tlef" a common extension for technology LEF? In that case I'd just add "tlef" as a recognized LEF extensions.

Matthias

@stefanottili
Copy link
Author

stefanottili commented Oct 6, 2023 via email

@stefanottili
Copy link
Author

#1504

fixes this issue too.
It was caused by the specified .tlef not being read at all.
It would still be nice to have .tlef in the list of lef file extensions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants