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

Reorganise setups #61

Merged
merged 5 commits into from
Jun 10, 2021
Merged

Reorganise setups #61

merged 5 commits into from
Jun 10, 2021

Conversation

umarcor
Copy link
Collaborator

@umarcor umarcor commented Jun 10, 2021

In this PR, Arty and Nexys example setups/scripts/READMEs are updated. Now, when create_project.tcl is sourced, the bitstream is generated straightaway. By the way, the duplicated scripts for Nexys boards are merged into a single one.

Then, subdirs setups/vivado, setups/quartus and setups/radiant are created. The "board" subdirs are reorganised/moved accordingly. The structure we have corresponds to the tools, not necessarily to the boards. For instance, UPduino is supported both in setups/radiant/UPduino_v3 and setups/osflow/UPduino_v3.

A setup/vivado/README.md is added for reducing the duplication in Arty and Nexys READMEs. Both batch and GUI modes are explained.

@stnolting
Copy link
Owner

Very nice! Thank you for cleaning up 😉

@stnolting stnolting merged commit 4a49850 into stnolting:master Jun 10, 2021
@umarcor
Copy link
Collaborator Author

umarcor commented Jun 10, 2021

Nice! Note that the Fomu PR is "broken" now. Do not worry. I'll fix (rebase) that 😉

@umarcor umarcor deleted the vivado-run branch June 10, 2021 13:55
@stnolting
Copy link
Owner

Nice! Note that the Fomu PR is "broken" now.

Right. But I thought it does not matter since FOMU is still under construction.
I'm sorry if this has caused you extra work. 😟

@umarcor
Copy link
Collaborator Author

umarcor commented Jun 10, 2021

Right. But I thought it does not matter since FOMU is still under construction.
I'm sorry if this has caused you extra work. 😟

No worries! That was expected 😉

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.

None yet

2 participants