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

Make antenna checker understand global route format #3892

Closed
eder-matheus opened this issue Aug 22, 2023 · 4 comments · Fixed by #5098
Closed

Make antenna checker understand global route format #3892

eder-matheus opened this issue Aug 22, 2023 · 4 comments · Fixed by #5098
Assignees
Labels
ant Antenna Checker

Comments

@eder-matheus
Copy link
Collaborator

eder-matheus commented Aug 22, 2023

Description

When evaluating the global routing results for antenna violations, we translate the global route format (i.e., global segments) to actual wires, having to create local connections with the pins.

This translation sometimes has issues, leading to crashes, specifically on the odb::orderWires function.

Suggested Solution

The antenna checker tool should be able to understand the global route format, removing the necessity of translating the global route to wires. This would remove a large piece of code that performs the translation, remove the calls for odb::orderWires, and basically would simplify the process of checking antenna violations on the global route results.

A suggested approach is to use boost::polygon to create the routing topology from the wires. With that, orderWires would not be needed at all, and the antenna checker would traverse only through the boost::polygon structure.

Additional Context

I'm not 100% sure if supporting the global route format in the antenna checker is possible. Any input on potential problems this could lead will be appreciated.

@maliberty
Copy link
Member

I think this is possible. Antenna checking just needs a topology and metal area attached.

@maliberty
Copy link
Member

@eder-matheus are you planning to look at the antenna code change?

@openroadie
Copy link
Contributor

@maliberty : We discussed it in the weekly meeting and I was going to take a look since @eder-matheus has some other higher priority items on his list.

@maliberty maliberty added the ant Antenna Checker label Oct 17, 2023
@maliberty maliberty removed their assignment Mar 2, 2024
@maliberty
Copy link
Member

In #4673 I've accepted some degradation in sky130hs/aes that originates from a new detour leading to more antenna fixing. Once the antenna fixing is improved please see if this design is improved. Currently there are some odd antennas on the driver side.

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

Successfully merging a pull request may close this issue.

4 participants