Clarify reason for holding rdmaskn valid during the entire cycle
authorCesar Strauss <cestrauss@gmail.com>
Thu, 31 Dec 2020 12:20:13 +0000 (12:20 +0000)
committerCesar Strauss <cestrauss@gmail.com>
Thu, 31 Dec 2020 12:20:33 +0000 (12:20 +0000)
src/soc/experiment/test/test_compalu_multi.py

index 98ffcc3e729fd928e83ae20e63c60cc61e574bd4..7eb98f57ffb64a0df2b37a1bd184487497479006 100644 (file)
@@ -365,8 +365,7 @@ class OpSim:
         # were latched at the correct cycle
         # note: rdmaskn is not latched, and must be held as long as
         # busy_o is active
-        # todo: is the above restriction on rdmaskn intentional?
-        # todo: shouldn't it be latched by issue_i, like the others?
+        # See: https://bugs.libre-soc.org/show_bug.cgi?id=336#c44
         yield self.dut.oper_i.insn_type.eq(0)
         if hasattr(dut.oper_i, "invert_in"):
             yield self.dut.oper_i.invert_in.eq(0)