Skip to content

Comments

Fixes and improvements from the PULP iDMA integration effort#49

Closed
da-gazzi wants to merge 13 commits intomasterfrom
georgr/pr
Closed

Fixes and improvements from the PULP iDMA integration effort#49
da-gazzi wants to merge 13 commits intomasterfrom
georgr/pr

Conversation

@da-gazzi
Copy link
Contributor

@da-gazzi da-gazzi commented Jun 28, 2024

  • register frontend now supports setting source and destination protocols
  • legalizer is instantiated correctly (before it would never use bursts when any used protocol did not support it)
  • register frontend correctly sets number of N-D repetitions when not using the maximum number of dimensions
  • some compilation/optimization fixes (specific to vopt)

thommythomaso and others added 13 commits June 20, 2024 17:14
@da-gazzi da-gazzi requested a review from thommythomaso as a code owner June 28, 2024 14:01
thommythomaso
thommythomaso reviewed Jul 4, 2024
View reviewed changes
target/rtl/.gitignore
Copy link
Collaborator

@thommythomaso thommythomaso Jul 4, 2024

Choose a reason for hiding this comment

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

There should be no generated files checked in.

This was referenced Aug 6, 2024
Closed
Closed
@sermazz sermazz mentioned this pull request Feb 19, 2025
Merged
@thommythomaso
Copy link
Collaborator

thommythomaso commented Feb 27, 2025

This PR will be merged as part of #66. Closed.

@thommythomaso thommythomaso deleted the georgr/pr branch February 27, 2025 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@thommythomaso thommythomaso thommythomaso left review comments

Assignees

@thommythomaso thommythomaso

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@da-gazzi @thommythomaso