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

axi_address_resolver relies on non-existing module #1

Closed
bchetwynd opened this issue Sep 6, 2018 · 7 comments
Closed

axi_address_resolver relies on non-existing module #1

bchetwynd opened this issue Sep 6, 2018 · 7 comments
Assignees

Comments

@bchetwynd
Copy link

The axi_address_resolver component relies on the find_first_one component, which is not part of the repository. The axi_address_resolver component is used in axi_lite_xbar.sv.

Thus, when attempting to simulate using compile_vsim.sh / run_vsim.sh scripts, the simulation fails due to undefined module.

@FrancescoConti
Copy link
Member

I'm not entirely sure what the local testbench does, however you can find all the IP dependencies in the Bender.yml file. In this case, what you need is http://github.com/pulp-platform/common_cells .

I think the IP owners (@fabianschuiki @zarubaf ?) should explicitly call bender to fetch these in their script.

@bchetwynd
Copy link
Author

Clarification much appreciated.

@zarubaf
Copy link
Contributor

zarubaf commented Sep 6, 2018

Yes, @FrancescoConti is completely right. Currently we do not provide our bender flow as open-source. If you want to use the testbench you are welcome to do so but you need to resolve your dependencies manually. The find first zero can be found in the common cells repository as Francesco mentioned.

@bchetwynd
Copy link
Author

Thank you. For what it is worth, the find_first_one module is contained in a deprecated directory. However, including that file as suggested addresses the issue in the axi_lite_xbar.sv module.

@gurkaynak
Copy link

@bchetwynd is that by any chance Brendon from WPI end of 90s?

@bchetwynd
Copy link
Author

Yes, indeed :) Hello, Frank!

@fabianschuiki
Copy link
Collaborator

You can theoretically already use Bender, but you would have to compile it from source yourself (https://github.com/fabianschuiki/bender). We're still ironing out the flow of that tool and will provide precompiled binaries once things become more stable. But just in case you're feeling lucky 😉 .

maicolciani pushed a commit to AlSaqr-platform/axi that referenced this issue Apr 14, 2023
niwis pushed a commit that referenced this issue Jun 27, 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

No branches or pull requests

5 participants