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

Add FuseSoC support for spm, s44 #1460

Closed
wants to merge 3 commits into from

Conversation

olofk
Copy link
Contributor

@olofk olofk commented Oct 30, 2022

This adds a core description file for the spm core that exposes targets for linting and for building a GDSII using OpenLANE.

This PR is part of a larger effort to upstream OpenLANE support through FuseSoC+Edalize for ALL example designs that OpenLANE uses. The ambition is to avoid stale copies of files and instead making sure that any fixes comes to benefit to all users. The effort can be tracked klasnordmark/openlane-examples#2 This core was one of the only cores where we couldn't find a proper upstream, which is why we file this PR towards OpenLANE itself.

Quick FuseSoC instructions:

#install FuseSoC
pip3 install fusesoc
#Create and enter a new workspace
mkdir workspace && cd workspace
#Register spm as a library in the workspace
fusesoc library add spm /path/to/spm
#...if repo is available locally or...
fusesoc library add spm https://github.com/The-OpenROAD-Project/OpenLane
#...to get the upstream repo

#To run lint
fusesoc run --target=lint efabless::spm
#To build with OpenLANE running in a docker container
EDALIZE_LAUNCHER=el_docker fusesoc run --target=sky130 efabless::spm
#List all targets
fusesoc core show efabless::spm

This adds a core description file for the spm core that exposes targets
for linting and for building a GDSII using OpenLANE.

Quick FuseSoC instructions:

 #install FuseSoC
pip3 install fusesoc
 #Create and enter a new workspace
mkdir workspace && cd workspace
 #Register spm as a library in the workspace
fusesoc library add spm /path/to/spm
 #...if repo is available locally or...
fusesoc library add spm https://github.com/The-OpenROAD-Project/OpenLane
 #...to get the upstream repo

 #To run lint
fusesoc run --target=lint efabless::spm
 #To build with OpenLANE running in a docker container
EDALIZE_LAUNCHER=el_docker fusesoc run --target=sky130 efabless::spm
 #List all targets
fusesoc core show efabless::spm
@donn donn changed the title Add FuseSoC support for spm design Add FuseSoC support for spm, s44 design Nov 9, 2022
@donn donn changed the title Add FuseSoC support for spm, s44 design Add FuseSoC support for spm, s44 Nov 9, 2022
@@ -0,0 +1,9 @@
set ::env(DESIGN_NAME) "spm"
Copy link
Member

Choose a reason for hiding this comment

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

It seems like this is duplicating config.json. Is that right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. Good catch. I pushed a fix to remove that line

@donn
Copy link
Collaborator

donn commented Apr 13, 2023

Sorry, @olofk- This took a long time to respond to, @shalan was supposed to follow up but it appears to have fallen through.

We haven't merged this because we're not sure entirely what "FuseSoC support" entails..? Does this allow the test designs to be used as an input in FuseSoC, because this isn't meant to be used as a design repository, these designs serve as tests and examples.

@donn donn added the waiting on op Information has been requested from the Issue Author label Apr 13, 2023
@donn
Copy link
Collaborator

donn commented Jun 13, 2023

Stale.

@donn donn closed this Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting on op Information has been requested from the Issue Author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants