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] VHDL cleanups #476

Merged
merged 5 commits into from
Jan 14, 2023
Merged

[rtl] VHDL cleanups #476

merged 5 commits into from
Jan 14, 2023

Conversation

stnolting
Copy link
Owner

@stnolting stnolting commented Jan 14, 2023 •
edited
Loading

  • rework VHDL coding style to prevent some of Quartus' and Vivado's Warning messages
  • rework PMP CSR read access logic
  • fix minor bug in instruction fetch unit

@stnolting
[rtl] CPU CTRL: VHDL style edits
a2e2f04 
- use if-generate for issue engine configuration (depending on C extension)
- do not use variables is sync. processes
@stnolting
[rtl] FIFO: minor optimiziation
27b7e71
@stnolting
[rtl] code cleanup
c457279
@stnolting
[rtl] rework PMP CSR read access
ce0663c 
+ fix minor bug in instruction fetch
@stnolting stnolting added HW Hardware-related coding-style Related to the HW/SW coding style labels Jan 14, 2023
@stnolting stnolting self-assigned this Jan 14, 2023
@stnolting stnolting marked this pull request as ready for review January 14, 2023 20:57
@stnolting stnolting merged commit 85fb3be into main Jan 14, 2023
@stnolting stnolting deleted the vhdl_cleanups branch January 14, 2023 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@stnolting stnolting

Labels
coding-style Related to the HW/SW coding style HW Hardware-related
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@stnolting