From 29b20a0e39b2f74fea53d00da7484b9c693fe11b Mon Sep 17 00:00:00 2001 From: Cesar Strauss Date: Thu, 31 Dec 2020 12:20:13 +0000 Subject: [PATCH] Clarify reason for holding rdmaskn valid during the entire cycle --- src/soc/experiment/test/test_compalu_multi.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/soc/experiment/test/test_compalu_multi.py b/src/soc/experiment/test/test_compalu_multi.py index 98ffcc3e..7eb98f57 100644 --- a/src/soc/experiment/test/test_compalu_multi.py +++ b/src/soc/experiment/test/test_compalu_multi.py @@ -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) -- 2.30.2