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.

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

[rtl] cleanups and optimizations #569

Merged
merged 11 commits into from
Apr 5, 2023
Merged

[rtl] cleanups and optimizations #569

merged 11 commits into from
Apr 5, 2023

Conversation

stnolting
Copy link
Owner

@stnolting stnolting commented Apr 5, 2023

  • "The NEORV32 does not implement the user-mode time[h] registers. Any access to these registers will trap. It is recommended that the trap handler software provides a means of accessing the platform-defined MTIME machine timer." - same concept is used in Ibex/openTitan 😉
  • VHDL code cleanups
  • minor logic optimizations
  • fix minor bug in NEORV32 runtime environment: do not try to load the trap-causing instruction word when a instruction access fault exception has been raised

@stnolting stnolting added the HW hardware-related label Apr 5, 2023
@stnolting stnolting self-assigned this Apr 5, 2023
do not try to load the trap-causing instruction word when a instruction access fault exception has been raised
* add PMP execute test
* cleanups and optimizations
@stnolting stnolting marked this pull request as ready for review April 5, 2023 17:49
@stnolting stnolting merged commit 1132030 into main Apr 5, 2023
@stnolting stnolting deleted the dev040423 branch April 5, 2023 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HW hardware-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant