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

IO cells schematics implemented in Xschem #171

Open
wants to merge 13 commits into
base: dev
Choose a base branch
from

Conversation

prabhatdubey92
Copy link

Fixes #<issue_number_goes_here>

  • Tests pass
  • Appropriate changes to README are included in PR

Signed-off-by: Prabhat Kumar Dubey <[email protected]>
Signed-off-by: Prabhat Kumar Dubey <[email protected]>
Signed-off-by: Prabhat Kumar Dubey <[email protected]>
Signed-off-by: Prabhat Kumar Dubey <[email protected]>
Signed-off-by: Prabhat Kumar Dubey <[email protected]>
Signed-off-by: prabhatdubey92 <[email protected]>
@KrzysztofHerman KrzysztofHerman changed the title Dev IO cells schematics Xschem Jul 30, 2024
@KrzysztofHerman KrzysztofHerman changed the title IO cells schematics Xschem IO cells schematics implemented in Xschem Jul 30, 2024
@sergeiandreyev
Copy link
Contributor

absolute path part should be removed

Signed-off-by: prabhatdubey92 <[email protected]>
Signed-off-by: prabhatdubey92 <[email protected]>
Copy link
Contributor

Choose a reason for hiding this comment

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

why did you choose to change to the worst case?

Signed-off-by: prabhatdubey92 <[email protected]>
@sergeiandreyev
Copy link
Contributor

what has to be updated in sg13g2_io_Gallery_tb.sch:

  • resistors -> .lib cornerRES... instead of .include separate files
  • temperature 127->27
  • .tran in .control section 2u->1u

we should write somewhere explicitly that xschem tool should be started from libs.ref/sg13g2_io/xschem folder for the IO cells symbols to be found correctly

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