hdl.mem: use 1 as reset value for ReadPort.en.
authorwhitequark <whitequark@whitequark.org>
Fri, 20 Sep 2019 19:36:19 +0000 (19:36 +0000)
committerwhitequark <whitequark@whitequark.org>
Fri, 20 Sep 2019 19:51:13 +0000 (19:51 +0000)
commit4b3a068b15e5669dee3b8defc9e4b54ab85f94f3
treecaff70b6d0c502b1d02d96e06abc81fdf4270c20
parent91ef2f58e3ba8cf1c5455af2b9ac44ac170d281e
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