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

projects/scripts/adi_board: Updated ad_cpu interconnect #1337

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

IstvanZsSzekely
Copy link
Contributor

@IstvanZsSzekely IstvanZsSzekely commented May 24, 2024

PR Description

On testbenches, if more than 16 interconnects are used, then the validation fail, since one of the Master interfaces is always AXI4Full (DDR). In order to fix this issue, cascading interconnects are automatically created when connecting IPs to the CPU.
Added option to cascade interconnects in testbenches if 16 or more modules are connected to the CPU.
Added an additional variable to choose between a cascaded or non-cascaded interconnect option for HDL. For Testbenches it's automatically set to be cascaded.

PR Type

  • Bug fix (change that fixes an issue)
  • New feature (change that adds new functionality)
  • Breaking change (has dependencies in other repos or will cause CI to fail)

PR Checklist

  • I have followed the code style guidelines
  • I have performed a self-review of changes
  • I have compiled all hdl projects and libraries affected by this PR
  • I have tested in hardware affected projects, at least on relevant boards
  • I have commented my code, at least hard-to-understand parts
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe files, Copyright etc)
  • I have not introduced new Warnings/Critical Warnings on compilation
  • I have added new hdl testbenches or updated existing ones

acostina
acostina previously approved these changes Jun 14, 2024
@IstvanZsSzekely IstvanZsSzekely marked this pull request as draft September 6, 2024 06:35
- Added option to cascade interconnects in if 16 or more modules are connected to the CPU
- Added option to enable cascading, by default it is disabled

Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants