-
Notifications
You must be signed in to change notification settings - Fork 86
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
Added target for openlane flow #245
Conversation
hardware/asic/README.md
Outdated
|
||
In order to run OpenLane from Makefile provided in the hardware/asic/skywater folder, you need to export following environment variable. | ||
```bash | ||
export OPENLANE_DIRECTORY=<path to OpenLane root directory> |
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 suggest OPENLANE_HOME
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.
Changed accordingly
hardware/asic/README.md
Outdated
@@ -62,9 +61,16 @@ sudo usermod -aG docker $USER | |||
```bash | |||
newgrp docker |
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 don't get this: the group has been created already. Isn't it created when you install docker?
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.
No sir the make in openlane will not do this .......... I was stuck at this point ......... it is important
hardware/asic/README.md
Outdated
```bash | ||
make mount | ||
``` | ||
The "make mount" command opens up a bash terminal in which you run the following tcl script to generate a design configuration of your design: | ||
```bash | ||
flow.tcl -design design_name -init_design_config |
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.
show the bash prompt here
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.
Added
hardware/asic/skywater/Makefile
Outdated
#soc flow for OpenLane | ||
soc: | ||
cd $(OPENLANE_DIRECTORY) && \ | ||
$(DOK_CMD) sh -c "./flow.tcl -design APU --init_design_config -overwrite" |
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.
is this an alternative to moving into the directory ? Why have both?
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.
Right now it is ok as u said in hangout conversation
hardware/asic/README.md
Outdated
```bash | ||
make mount | ||
``` | ||
The "make mount" command opens up a bash terminal in which you run the following tcl script to generate a design configuration of your design: |
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.
In which line in the README the user is instructed to create the design files? Where should the user put them?
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 provided the required explanation i will update the PR soon
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.
Kindly check changes
@@ -1,19 +1,26 @@ | |||
#Makefile for OpenLane and OpenRAM started on 2/11/21 | |||
|
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.
ROOT_DIR=../../..
need this so your makefile can find root
also add debug target to display variables |
Hi Sir,
This PR has changes for target for OpenLane flow. This target first starts the Docker container and then runs the flow for asic soc.
Things to be addressed next include,
regards