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.

Sign up for GitHub

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

Jump to bottom

edalize: add fpga_interchange variant to nextpnr flow #94

Merged
merged 1 commit into from Apr 20, 2021
Merged

edalize: add fpga_interchange variant to nextpnr flow #94

merged 1 commit into from Apr 20, 2021

Conversation

kowalewskijan
Copy link

@kowalewskijan kowalewskijan commented Apr 8, 2021 •
edited

The PR introduces changes necessary to emit nextpnr-fpga_interchange variant configuration.
Dependencies:

TODOs:

  • FASM generation
  • bitstream generation

Signed-off-by: Jan Kowalewski jkowalewski@antmicro.com

@kowalewskijan kowalewskijan mentioned this pull request Apr 8, 2021
Closed
8 tasks
acomodi
acomodi reviewed Apr 9, 2021
View reviewed changes
Copy link

@acomodi acomodi left a comment

Choose a reason for hiding this comment

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

I believe that this should be upstreamed already instead of getting it into the SymbiFlow fork.

Once that is done we should probably get the symbiflow/symbiflow branch rebased on top of mainline, as it is quite a while since it was updated.

edalize/nextpnr.py Outdated Show resolved Hide resolved
@tmichalak tmichalak added this to To Do in FPGA interchange bootstrapping via automation Apr 13, 2021
@tmichalak tmichalak moved this from To Do to Blocked in FPGA interchange bootstrapping Apr 13, 2021
@kowalewskijan kowalewskijan changed the title [WIP] edalize: add fpga_interchange variant to nextpnr flow edalize: add fpga_interchange variant to nextpnr flow Apr 16, 2021
@kowalewskijan
Copy link
Author

With current changes I was able to generate the bitstream using fpga-tool-perf counter example. In the meantime I'll prepare an upstreamed version of the SymbiFlow:symbiflow branch so we can merge fork changes to mainline.

edalize: add fpga_interchange variant to nextpnr flow
d24d830 
Signed-off-by: Jan Kowalewski <jkowalewski@antmicro.com>
@kowalewskijan kowalewskijan moved this from Blocked to In progress in FPGA interchange bootstrapping Apr 16, 2021
acomodi
acomodi approved these changes Apr 20, 2021
View reviewed changes
Copy link

@acomodi acomodi left a comment

Choose a reason for hiding this comment

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

LGTM. I think that this can be merged in this fork, but we'll definitely need to move forward integrating everything into upstream.

@acomodi acomodi merged commit d9d753c into SymbiFlow:symbiflow Apr 20, 2021
FPGA interchange bootstrapping automation moved this from In progress to Done Apr 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@acomodi acomodi acomodi approved these changes

@tmichalak tmichalak Awaiting requested review from tmichalak

Assignees
No one assigned
Labels
None yet
Projects
FPGA interchange bootstrapping
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@kowalewskijan @acomodi