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

adding cace support plus removing unnecessary pins #2

Open
wants to merge 2 commits into
base: tapeout_ci2406
Choose a base branch
from

Conversation

areda0
Copy link

@areda0 areda0 commented Sep 10, 2024

  • a .yaml file supporting latest CACE format was created
  • extra CACE conditions for ena, vref_ext, vref_sel were added
  • extra DVDD and DVSS pins in both sch, layout, and symbol views were removed
  • the template tb was updated with CACE substitutions and measure statement was edited
  • files locations were adapted to match the standard CACE files hierarchy
  • LVS script for netgen_lvs was added

Summary:
With these changes the block passes: LVS , magic_drc , klayout_drc_full.
Area is measured but there isn't a spec to be compared against
The block tb doesn't include a load cap
Block still needs some work on the relevant specs verification (creating missing tbs and adding missing measurements)

Copy link
Collaborator

@RTimothyEdwards RTimothyEdwards left a comment

Choose a reason for hiding this comment

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

Removing DVDD and DVSS is not correct. They are definitely part of the schematic and layout.

Copy link
Collaborator

@RTimothyEdwards RTimothyEdwards left a comment

Choose a reason for hiding this comment

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

This cannot be merged. Pins DVDD and DVSS were very much part of the circuit and cannot be removed without breaking the whole circuit. Please fix.

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.

2 participants