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] Try to fix Quartus latch warnings #434

Merged
merged 3 commits into from
Oct 25, 2022
Merged

Conversation

stnolting
Copy link
Owner

Quartus 21.1 shows the following message:

Warning (10631): VHDL Process Statement warning at neorv32_cpu_control.vhd(1635): inferring latch(es) for signal or variable "csr", which holds its previous value in one or more paths through the process

But there are no latches in the code! The CSR read and write processes are entirely synchronous. Furthermore, the synthesis report does not show any latches (Compilation Report -> Analysis & Synthesis -> Optimization Results -> Register Statistics).

This PR tries to fix this issue by reformatting certain parts of the VHDL code.

See discussion #421 for more information.

@stnolting stnolting added HW Hardware-related experimental Experimental feature labels Oct 21, 2022
@stnolting stnolting self-assigned this Oct 21, 2022
Tie all unimplemented CSRs / CSR bits to zero (if certain ISA extensions are disabled)
@stnolting stnolting marked this pull request as ready for review October 24, 2022 19:59
@stnolting stnolting merged commit b3ba2a6 into main Oct 25, 2022
@stnolting stnolting deleted the fix_latches_warning branch October 25, 2022 04:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
experimental Experimental feature HW Hardware-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant