Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[rtl] Cleanup/update assertions and "auto-configuration" #738

Merged
merged 5 commits into from
Nov 26, 2023

Conversation

stnolting
Copy link
Owner

Invalid generic configurations (for example a cache block size that is not a power of two) will no longer result in an error assertion. Instead, the invalid configuration is auto-adjusted and the user is informed with a warning. For example:

  assert not ((nblocks_valid_c = false) or (block_size_valid_c = false)) report
    "[NEORV32] Auto-adjusting invalid d-cache size configuration(s)." severity warning;

@stnolting stnolting added enhancement New feature or request HW Hardware-related labels Nov 25, 2023
@stnolting stnolting self-assigned this Nov 25, 2023
@stnolting stnolting marked this pull request as ready for review November 25, 2023 21:30
@stnolting stnolting merged commit 21fea6d into main Nov 26, 2023
@stnolting stnolting deleted the assert_cleanup branch November 26, 2023 17:35
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request HW Hardware-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant