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

axi_pkg: Add iomsb to avoid underflow in array lengths #355

Merged
merged 1 commit into from
Dec 2, 2024
Merged

axi_pkg: Add iomsb to avoid underflow in array lengths #355

merged 1 commit into from
Dec 2, 2024

Conversation

CyrilKoe
Copy link
Contributor

axi_id_serialize_intf such as used here calls axi_id_serialize without setting IdMapNumEntries. This causes VCS to complain about an array bound greater than 2^31-1.
Thus, iomsb function may be useful in axi.

@paulsc96 paulsc96 mentioned this pull request Nov 29, 2024
Open
2 tasks
@thommythomaso thommythomaso self-assigned this Dec 2, 2024
@thommythomaso thommythomaso merged commit c5ad810 into master Dec 2, 2024
3 checks passed
@thommythomaso thommythomaso deleted the ck/vcs branch December 2, 2024 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thommythomaso thommythomaso thommythomaso approved these changes

Assignees

@thommythomaso thommythomaso

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@CyrilKoe @thommythomaso