-
Notifications
You must be signed in to change notification settings - Fork 10
Added RISC-V example #50
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
Conversation
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.
Looks good to me.
## Compiling the C++ Application Software | ||
|
||
Before compiling and updating the software application make sure to click the | ||
`Stop` button to allow the binaries (.elf files) to be updated. |
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.
Can you clarify here that the stop button is in the web interface?
Before compiling and updating the software application make sure to click the | ||
`Stop` button to allow the binaries (.elf files) to be updated. | ||
|
||
There are two main executables binaries that need to be compiled. One is |
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.
Can you provide the names of the two executable and refer to them by name in the next sentence?
[Example Setup](#example_setup) section | ||
|
||
If all goes well, then you should see the text `"ALL_DONE"` at the end. At this | ||
point you can click on the `Start` button and the updated .elf files will be used. |
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.
Again clarify that start button is in the web interface.
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.
Minor: typo overal -> overall
(bayer interpolation, gamma correction, enhancement and scaling), which are part | ||
of the original AppNote. Then the video frames go though some additional | ||
processing (HLS modules) before they get encoded by the H264 module. The RISC-V | ||
processor then encapsulates the encoded frames into RTP packets and stream 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.
stream -> streams
## get_pixel() | ||
|
||
This function can be thought of as a demux module. It receives a video stream | ||
and either forwards it downstream or writes it to DDR memory depending on an option |
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.
Avoid "it" here to improve clarity. This function receives .... either forwards the input downstream or writes the input to DDR memory
connectivity has been setup correctly before making any changes. | ||
|
||
**NOTE: The precompiled bitstream will only work in boards with Production Silicon, | ||
not with Engineering Sample silicon. If your board has the later, you will need to |
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.
later -> latter
The precompiled embedded Linux image contains the necessary video drivers to use the camera. | ||
|
||
**NOTE: As your board may have a different Linux image, it is strongly | ||
suggested to flash it with the precompiled Linux image for this example.** |
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.
Suggest to avoid it. you are strongly suggested to flash the board
./script_support/additional_configurations/smarthls/run_libero.sh | ||
``` | ||
Notice the relative path from which the script has to be executed. The compilation | ||
process may take about 40 min. depending on how fast the machine is. |
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.
Can this Tcl script be run from inside Libero for Windows users?
No description provided.