Skip to content

Introduce combined modules - #5

Draft
wirthjohannes wants to merge 6 commits into
esa-tu-darmstadt:masterfrom
wirthjohannes:combined
Draft

Introduce combined modules#5
wirthjohannes wants to merge 6 commits into
esa-tu-darmstadt:masterfrom
wirthjohannes:combined

Conversation

@wirthjohannes

Copy link
Copy Markdown

This PR introduces combined modules (and interfaces) for AXI4, AXI4-Lite and AXI3.
With this you no longer need to instantiate two separate modules for RD and WR.

This change is entirely optional and does not break existing usages.

Besides this, the PR also gets rid of the compiler warnings produced by the dummy modules.

@wirthjohannes
wirthjohannes marked this pull request as draft October 11, 2024 20:10
@wirthjohannes

Copy link
Copy Markdown
Author

Replaced the GetS interfaces in the combined modules with the more common Get.
This makes the "normal" usage more in-line with the existing, split modules. But it does not allow snooping of values (for the combined modules).

If snooping is something which we want, we need to to figure out a good way to do this without impacting the "normal" usage. (Also until now snooping was implemented only in some of the split modules)

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

Successfully merging this pull request may close these issues.

1 participant