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

Openlane implementation #12

Draft
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

kareefardi
Copy link

This a draft pull request of an implementation of caravel_openframe_wrapper_project

@RTimothyEdwards
Copy link
Owner

@kareefardi : Please do not clog up this repository with megabytes of data that will always be reproduced by a build. Only check in those files needed for setup of openlane.

@RTimothyEdwards
Copy link
Owner

@kareefardi : Any update on this? Can you regenerate a pull request with only the minimum set of files needed to run a successful synthesis flow?

@kareefardi
Copy link
Author

Yes. I will create a PR with necessary files for running OL. One thing that I am aware is missing is power connections to caravel openframe itself

@RTimothyEdwards
Copy link
Owner

@kareefardi : Missing where?

@kareefardi
Copy link
Author

kareefardi commented Jul 6, 2023

The PR doesn't establish the power connections between the example wrapper and caravel_openframe itself. What I had in mind was to create a macro similar to https://github.com/efabless/caravel/blob/main/lef/manual_power_connections.lef but I didn't have time to do it.

@RTimothyEdwards
Copy link
Owner

@kareefardi : Okay, yes, that looks like the right approach. The open frame isn't going to have an internal set of power rings because it's not specifically for digital, but for digital projects I think it makes sense that the user will need to define the power rings and power connections, but that if we make useful assumptions, we can make those for the example project that anybody can copy over to their own projects and use, and it will work for most digital openframe projects.

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