Skip to content

updated asap7 platform config.mk to support multi vt#3223

Closed
jeffng-or wants to merge 1 commit into
The-OpenROAD-Project:masterfrom
jeffng-or:asap7-multi-vt-infra
Closed

updated asap7 platform config.mk to support multi vt#3223
jeffng-or wants to merge 1 commit into
The-OpenROAD-Project:masterfrom
jeffng-or:asap7-multi-vt-infra

Conversation

@jeffng-or

Copy link
Copy Markdown
Contributor
  • Updated jpeg_lvt to leverage ASAP7_USE_VT instead of manually adding the LVT asap7 lib files - more in line with aes_lvt and ethmac_lvt
  • Extended ASAP7_USE_VT to support one or more Vt's.
  • Default remains a single Vt (RVT)
  • The first Vt specified is treated as the primary Vt and is used to set the tie hi/tie lo and other cells
  • The other Vts are treated as secondary and their Lib, LEF, GDS, and fill cells are added to the appropriate lists
  • Note that designs that use CCS will not be able to use multi-Vt unless we add the LVT and SLVT CCS files to the repo

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we using 6T cells?

@povik I've forgotten why we need this workaround but it would be nice if we didn't. Do you know/recall the history?

@povik povik Jun 11, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can see it was added in #1447 for use in the EQY check. If we switch to use Liberty models as discussed in #3179 we won't need this anymore.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the 6T question, it doesn't look like it. We only have only using 7.5T LEF/Lib/GDS in our repo.

I ended up duplicating what was already there for RVT. but can remove all the 6T files as part of this PR.

@jeffng-or jeffng-or force-pushed the asap7-multi-vt-infra branch from 1fe6a02 to 4dc8353 Compare June 11, 2025 16:53
removed 6T files for eqy

Signed-off-by: Jeff Ng <jeffng@precisioninno.com>
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 this pull request may close these issues.

3 participants