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

ariane_axi: Decrease AXI width to 48 #8

Closed
wants to merge 1 commit into from
Closed

ariane_axi: Decrease AXI width to 48 #8

wants to merge 1 commit into from

Conversation

niwis
Copy link
Collaborator

@niwis niwis commented Feb 13, 2023

Decrease the AXI width to 48. From https://github.com/pulp-platform/cva6/tree/cheshire

@Vyae @niwis
ariane_axi: Decrease AXI width to 48
450f832 
Signed-off-by: Nils Wistoff <nwistoff@iis.ee.ethz.ch>
@niwis
Copy link
Collaborator Author

niwis commented Feb 13, 2023

@Vyae can this also be achieved without modifying the cva6 code base by setting the AxiAddrWidth parameter of Ariane?

cva6/core/cva6.sv

Line 28 in 8df7147

parameter int unsigned AxiAddrWidth = ariane_axi::AddrWidth,

@niwis
Copy link
Collaborator Author

niwis commented Feb 13, 2023

Also, this change seems to break the CVA6 CI.

@joennlae joennlae mentioned this pull request Feb 17, 2023
Draft
@Vyeow
Copy link

Vyeow commented Feb 20, 2023

Yes it would be possible to use the parameter of CVA6 to set the address width. The parameter was not available in the version we initially used.

@Vyeow Vyeow closed this Feb 20, 2023
@Vyeow Vyeow deleted the addrwidth branch February 20, 2023 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Vyeow Vyeow

@creinwar creinwar

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@niwis @Vyeow @creinwar @Vyae