Skip to content

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

Merged
merged 1 commit into from
Nov 8, 2023
Merged

Added RISC-V example #50

merged 1 commit into from
Nov 8, 2023

Conversation

ManuelSaldana
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@Alireza-Mellat Alireza-Mellat left a 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.
Copy link
Collaborator

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
Copy link
Collaborator

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.
Copy link
Collaborator

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.

Copy link
Collaborator

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
Copy link
Collaborator

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
Copy link
Collaborator

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
Copy link
Collaborator

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.**
Copy link
Collaborator

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.
Copy link
Collaborator

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?

@Alireza-Mellat Alireza-Mellat merged commit 7f3bc85 into main Nov 8, 2023
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.

3 participants