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

2 - Add hls bridge for FINN kernel in Coyote #946

Open
wants to merge 14 commits into
base: dev
Choose a base branch
from

Conversation

ADGLY
Copy link

@ADGLY ADGLY commented Jan 3, 2024

This PR should be merged after #930.

This PR is now the first one that should be merged.

This PR allows circumventing the limited address space offered by Coyote for the AXILite. An HLS bridge is added between the AXILite control interface from Coyote and the FINN AXILite signals (tlast marker + writeable weights). By configuring the two registers of the HLS bridge (address offset + value), it is possible to write to a full 32 bits address space.

This PR also makes it so that axi stream width converters do not need to be instantiated manually. Upon AXI4Stream width mismatch, a converter is instantiated.

The new commits start at: "Generate width converters".

@ADGLY ADGLY changed the title Add hls bridge Add hls bridge for FINN kernel in Coyote Jan 3, 2024
@ADGLY ADGLY changed the title Add hls bridge for FINN kernel in Coyote 2 - Add hls bridge for FINN kernel in Coyote Jan 3, 2024
@ADGLY ADGLY force-pushed the add_hls_bridge branch 3 times, most recently from dac8919 to e3b6cf5 Compare January 5, 2024 16:24
@ADGLY ADGLY force-pushed the add_hls_bridge branch 2 times, most recently from 2f67a9c to b013ac0 Compare January 6, 2024 16:34
@ADGLY ADGLY mentioned this pull request Jan 13, 2024
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.

1 participant