From: Luke Kenneth Casson Leighton <lkcl@lkcl.net>
Date: Mon, 29 Nov 2021 21:47:24 +0000 (+0000)
Subject: testing comments
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=e773400775f479a98f9dc527e54884e096113691;p=pinmux.git

testing comments
---

diff --git a/src/spec/testing_stage1.py b/src/spec/testing_stage1.py
index ec9eff1..aa88807 100644
--- a/src/spec/testing_stage1.py
+++ b/src/spec/testing_stage1.py
@@ -408,10 +408,20 @@ def test_case0():
     yield Delay(delayVal)
     yield Settle()
     for _ in range(20):
-        yield top.gpio.gpio2.o.eq(~top.gpio.gpio2.o)
+        # get a value first (as an integer).  you were trying to set
+        # it to the actual Signal
+        gpio_o2 = yield top.gpio.gpio2.o
+        # then set it
+        yield top.gpio.gpio2.o.eq(~gpio_o2)
+
+        # ditto: here you are trying to set to an AST expression
+        # which is inadviseable (likely to fail)
         yield top.gpio.gpio3.o.eq(~top.gpio.gpio3.o)
         yield Delay(delayVal)
         yield Settle()
+        # again you are trying to set something equal to the Signal
+        # rather than to a value.  this is attempting to change the
+        # actual HDL which is completely inappropriate
         yield top.uart.rx.eq(~top.intermediary)
         yield Delay(delayVal)
         yield Settle()