search

chipsalliance / firrtl-spec

The specification for the FIRRTL language
44 stars 28 forks source link

[Proposal] Stronger and more complete reset types #161

Open darthscsi opened 9 months ago

darthscsi commented 9 months ago

Make all resets use a uniform type. Expand that type to cover asynchronous set, synchronous release resets. Expand that type to handle both active-high and active-low resets.

Provide a couple of ops that allow writing generic code on uninferred reset signals.

Tighten type conversion to make UInt<1> not equivalent to synchronous reset.

mmaloney-sf commented 9 months ago

I love this.

I added a few comments, mostly in terms of the presentation.