google / google/CFU-Playground

Disconnected reset in proj/mnv2_first/ CFU

Open
#110 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Verilog
Stars
565
Forks
160
PR merge metrics
No merged PRs in 30d

Description

With a recent PR, we renamed the top reset port on the CFU to be "reset" instead of "rst". However, now, inside of the generated cfu.v, the top port 'reset' is not connected to anything, while a signal 'rst' appears to be undriven.

I'm not sure where 'rst' is declared; I suspect it is implicit with the nMigen generation.

This was discovered by Yosys trimming away most of the CFU.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.