hdl.mem: use 1 as reset value for ReadPort.en.
authorwhitequark <cz@m-labs.hk>
Fri, 20 Sep 2019 19:36:19 +0000 (19:36 +0000)
committerwhitequark <cz@m-labs.hk>
Fri, 20 Sep 2019 19:51:13 +0000 (19:51 +0000)
commit53b3468780c349d6a17572cfe617495fa3b0a281
treecaff70b6d0c502b1d02d96e06abc81fdf4270c20
parent9ede0bfc541192f42a5d4afe8829ee99b822dd8d
hdl.mem: use 1 as reset value for ReadPort.en.

This is necessary for consistency, since for transparent read ports,
we currently do not support .en at all (it is fixed at 1) due to
YosysHQ/yosys#760. Before this commit, changing port transparency
would require adding or removing an assignment to .en, which is
confusing and error-prone.

Also, most read ports are always enabled, so this behavior is also
convenient.
nmigen/hdl/mem.py
nmigen/test/test_hdl_mem.py
nmigen/test/test_sim.py