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

[rtl] minor edits; update to VUnit v5 #605

Merged
merged 6 commits into from
Apr 28, 2023
Merged

[rtl] minor edits; update to VUnit v5 #605

merged 6 commits into from
Apr 28, 2023

Conversation

stnolting
Copy link
Owner

@stnolting stnolting commented Apr 27, 2023 •
edited
Loading

  • minor edits of CPU bus unit
  • reduced switching activity of CPU's memory data input logic (less dyn. power consumption)
  • small adjustments (for the perhaps upcoming A-extension)
  • 💡 update Vunit simulation script (for using VUnit v5)

@stnolting stnolting added the HW hardware-related label Apr 27, 2023
@stnolting stnolting self-assigned this Apr 27, 2023
@stnolting stnolting marked this pull request as ready for review April 27, 2023 15:31
@stnolting stnolting marked this pull request as draft April 28, 2023 06:15
@stnolting stnolting changed the title [rtl] minor edits [rtl] minor edits; update to VUnit v5 Apr 28, 2023
@stnolting stnolting marked this pull request as ready for review April 28, 2023 08:16
umarcor
umarcor approved these changes Apr 28, 2023
View reviewed changes
Copy link
Collaborator

@umarcor umarcor left a comment

Choose a reason for hiding this comment

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

LGTM!

@stnolting stnolting merged commit 0347add into main Apr 28, 2023
@stnolting stnolting deleted the dev270423 branch April 28, 2023 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@umarcor umarcor umarcor approved these changes

Assignees

@stnolting stnolting

Labels
HW hardware-related
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@stnolting @umarcor