From: Cesar Strauss <cestrauss@gmail.com>
Date: Mon, 25 May 2020 22:50:32 +0000 (-0300)
Subject: Check that busy_o doesn't rise on its own
X-Git-Tag: div_pipeline~820
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=a3fa9e226b53ae792650b7d50a29cb2b86bcf792;p=soc.git

Check that busy_o doesn't rise on its own
---

diff --git a/src/soc/experiment/compalu_multi.py b/src/soc/experiment/compalu_multi.py
index 081472ee..beb2179a 100644
--- a/src/soc/experiment/compalu_multi.py
+++ b/src/soc/experiment/compalu_multi.py
@@ -388,18 +388,22 @@ class CompUnitParallelTest:
         # Operation cycle should not take longer than this:
         self.MAX_BUSY_WAIT = 50
 
+        # Minimum duration in which issue_i will be kept inactive,
+        # during which busy_o must remain low.
+        self.MIN_BUSY_LOW = 5
+
     def driver(self):
 
         print("Begin parallel test.")
 
         # issue_i starts inactive
         yield self.dut.issue_i.eq(0)
-        yield
 
-        # busy_o must start inactive
-        # TODO: Check a few times that busy_o doesn't rise on its own
-        busy_o = yield self.dut.busy_o
-        assert not busy_o
+        for n in range(self.MIN_BUSY_LOW):
+            yield
+            # busy_o must remain inactive. It cannot rise on its own.
+            busy_o = yield self.dut.busy_o
+            assert not busy_o
 
         # activate issue_i to begin the operation cycle
         yield self.dut.issue_i.eq(1)