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

dma_core_wrap: Remove param DmaAddrWidth of idma_reg64_frontend.sv #16

Merged
merged 1 commit into from
Oct 28, 2022
Merged

dma_core_wrap: Remove param DmaAddrWidth of idma_reg64_frontend.sv #16

merged 1 commit into from
Oct 28, 2022

Conversation

Vyeow
Copy link

@Vyeow Vyeow commented Oct 28, 2022

No description provided.

@Vyeow Vyeow requested a review from thommythomaso October 28, 2022 10:48
@Vyeow Vyeow requested a review from micprog as a code owner October 28, 2022 10:48
@Vyeow Vyeow changed the title dma_core_wrap: Remove param DmaAddrWidth dma_core_wrap: Remove param DmaAddrWidth of idma_reg64_frontend.sv Oct 28, 2022
thommythomaso
thommythomaso approved these changes Oct 28, 2022
View reviewed changes
Copy link
Collaborator

@thommythomaso thommythomaso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@thommythomaso thommythomaso merged commit 6ad5695 into master Oct 28, 2022
@thommythomaso thommythomaso deleted the fix/dma_core_wrap branch October 28, 2022 12:54
@thommythomaso thommythomaso temporarily deployed to github-pages October 28, 2022 12:57 Inactive
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

@micprog micprog Awaiting requested review from micprog micprog is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Vyeow @thommythomaso @Vyae