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

Added Installation and CRAM instructions. #62

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

PythonLinks
Copy link

No description provided.

Copy link
Collaborator

@josuah josuah left a comment

Choose a reason for hiding this comment

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

Thank you this looks almost ready!
A very small note on changing the cmake command to something more canonical, but I can do the change afterward too.


## TODO

Pasa an argument to the cmake file with the name of the bitstream to be used.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Small typo here:

Suggested change
Pasa an argument to the cmake file with the name of the bitstream to be used.
Pass an argument to the cmake file with the name of the bitstream to be used.


If you want to install a different bitstream on the FPGA, just overwrite `rgb_blink.bin` with your bitstream and rerun

`cmake .`
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it is good to advise people to use mkdir build && cd build && cmake .. as otherwise it creates all sorts of temporary files everywhere.

Here are commands that seems to be the way to go:

cmake -S . -B build
cmake --build build

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.

2 participants