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

AXI4+ATOP on chip memory slave #115

Closed
wants to merge 28 commits into from
Closed

AXI4+ATOP on chip memory slave #115

wants to merge 28 commits into from

Conversation

WRoenninger
Copy link
Collaborator

@WRoenninger WRoenninger commented Jun 25, 2020

This adds two modules:

  • axi_to_mem: Salve module, max throughout simultaneous read/writes 50%, read or write 100%.
  • axi_to_mem_banked: With enough banks 100% throughput with simultaneous reads/writes.

Notes to Bender:
For the dependencies pulp-platform/tech_cells_generic has been directly added. In there the tc_sram module is used for the testbench to serve a memory model.
-> For all conflicts in the dependency tree of Bender, the versions specified by the Bender.yml of this repo has to be used.

Edit: Now bender.lock is also checked in so CI should pass now

This depends on changes form #116.

@WRoenninger WRoenninger requested a review from andreaskurth June 25, 2020 13:47
@WRoenninger
Copy link
Collaborator Author

The conflict in the Bender dependencies, leads to the failure of the ci.

@WRoenninger
Copy link
Collaborator Author

Rebased onto current master.

@WRoenninger
Copy link
Collaborator Author

Timeout for the complete CI seems no longer to be sufficient enough long when adding axi_to_mem_banked to ci.

@WRoenninger
Copy link
Collaborator Author

Rebased onto current master for ci fix by #129.

Ci will time-out as testing for axi_to_mem with the full parameterizations runs for multiple hours.

@stevobailey
Copy link
Contributor

Bump

@micprog micprog mentioned this pull request Feb 25, 2022
3 tasks
@micprog
Copy link
Member

micprog commented Aug 30, 2022

Merged with #244 and #248

@micprog micprog closed this Aug 30, 2022
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