-
Notifications
You must be signed in to change notification settings - Fork 47
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 all ASAP7 standard cell libraries. #206
Conversation
@QuantamHD - Does this direction make sense to you? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to make sense so far. You'll need to include the file groups in some kind of OpenROAD related rule, but it's definietly a good way to organize the files.
visibility = [":data_visibility"], | ||
) | ||
|
||
# FIXME: What about NLDM liberty? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Prefer using NLDM over CCS for OpenROAD. We don't have support for CCS yet so it's not adding a lot of value, but does consume a good bit of disk.
Looks like one of the target names is off. |
bd48ec1
to
0a303b1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@QuantamHD - Can you please take another look.
Now getting close to having rvt, lvt, slvt going for the rev 26, 27 & 28 cells.
The naming scheme I ended up with is |
The rev28 version of verilog_counter seems to work for the |
04e9455
to
886ca78
Compare
Let me know when this is ready. |
@QuantamHD - I'm working through making everything compile but would like you to review before that happens. |
Signed-off-by: Tim 'mithro' Ansell <[email protected]>
Signed-off-by: Tim 'mithro' Ansell <[email protected]>
Signed-off-by: Tim 'mithro' Ansell <[email protected]>
7ca046d
to
f2c1c44
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have done a rebase based on the ASAP7 repo reorg now.
It actually makes a bunch of things a bit cleaner.
@QuantamHD - PTAL.
Looking now |
LGTM! Looks really beautiful @mithro |
Currently debugging;
|
|
Currently debugging a few small things;
|
|
|
|
Signed-off-by: Tim 'mithro' Ansell <[email protected]>
Signed-off-by: Tim 'mithro' Ansell <[email protected]>
Signed-off-by: Tim 'mithro' Ansell <[email protected]>
Signed-off-by: Tim 'mithro' Ansell <[email protected]>
Signed-off-by: Tim 'mithro' Ansell <[email protected]>
Signed-off-by: Tim 'mithro' Ansell <[email protected]>
Signed-off-by: Tim 'mithro' Ansell <[email protected]>
No description provided.