Reset processing on assert edge only

cocotbext/eth/gmii.py

@@ -216,16 +216,16 @@ class GmiiSource(Reset):
             if self._run_cr is not None:
                 self._run_cr.kill()
                 self._run_cr = None
-        else:
-            self.log.info("Reset de-asserted")
-            if self._run_cr is None:
-                self._run_cr = cocotb.scheduler.start_soon(self._run())
 
             self.active = False
             self.data <= 0
             if self.er is not None:
                 self.er <= 0
             self.dv <= 0
+        else:
+            self.log.info("Reset de-asserted")
+            if self._run_cr is None:
+                self._run_cr = cocotb.scheduler.start_soon(self._run())
 
     async def _run(self):
         frame = None
@@ -379,13 +379,13 @@ class GmiiSink(Reset):
             if self._run_cr is not None:
                 self._run_cr.kill()
                 self._run_cr = None
+
+            self.active = False
         else:
             self.log.info("Reset de-asserted")
             if self._run_cr is None:
                 self._run_cr = cocotb.scheduler.start_soon(self._run())
 
-        self.active = False
-
     async def _run(self):
         frame = None
         self.active = False

cocotbext/eth/mii.py

@@ -117,16 +117,16 @@ class MiiSource(Reset):
             if self._run_cr is not None:
                 self._run_cr.kill()
                 self._run_cr = None
-        else:
-            self.log.info("Reset de-asserted")
-            if self._run_cr is None:
-                self._run_cr = cocotb.scheduler.start_soon(self._run())
 
             self.active = False
             self.data <= 0
             if self.er is not None:
                 self.er <= 0
             self.dv <= 0
+        else:
+            self.log.info("Reset de-asserted")
+            if self._run_cr is None:
+                self._run_cr = cocotb.scheduler.start_soon(self._run())
 
     async def _run(self):
         frame = None
@@ -273,13 +273,13 @@ class MiiSink(Reset):
             if self._run_cr is not None:
                 self._run_cr.kill()
                 self._run_cr = None
+
+            self.active = False
         else:
             self.log.info("Reset de-asserted")
             if self._run_cr is None:
                 self._run_cr = cocotb.scheduler.start_soon(self._run())
 
-        self.active = False
-
     async def _run(self):
         frame = None
         self.active = False

cocotbext/eth/ptp.py

@@ -186,10 +186,6 @@ class PtpClock(Reset):
             if self._run_cr is not None:
                 self._run_cr.kill()
                 self._run_cr = None
-        else:
-            self.log.info("Reset de-asserted")
-            if self._run_cr is None:
-                self._run_cr = cocotb.scheduler.start_soon(self._run())
 
             self.ts_96_s = 0
             self.ts_96_ns = 0
@@ -205,6 +201,10 @@ class PtpClock(Reset):
                 self.ts_step <= 0
             if self.pps is not None:
                 self.pps <= 0
+        else:
+            self.log.info("Reset de-asserted")
+            if self._run_cr is None:
+                self._run_cr = cocotb.scheduler.start_soon(self._run())
 
     async def _run(self):
         while True:

cocotbext/eth/rgmii.py

@@ -117,14 +117,14 @@ class RgmiiSource(Reset):
             if self._run_cr is not None:
                 self._run_cr.kill()
                 self._run_cr = None
-        else:
-            self.log.info("Reset de-asserted")
-            if self._run_cr is None:
-                self._run_cr = cocotb.scheduler.start_soon(self._run())
 
             self.active = False
             self.data <= 0
             self.ctrl <= 0
+        else:
+            self.log.info("Reset de-asserted")
+            if self._run_cr is None:
+                self._run_cr = cocotb.scheduler.start_soon(self._run())
 
     async def _run(self):
         frame = None
@@ -287,13 +287,13 @@ class RgmiiSink(Reset):
             if self._run_cr is not None:
                 self._run_cr.kill()
                 self._run_cr = None
+
+            self.active = False
         else:
             self.log.info("Reset de-asserted")
             if self._run_cr is None:
                 self._run_cr = cocotb.scheduler.start_soon(self._run())
 
-        self.active = False
-
     async def _run(self):
         frame = None
         self.active = False

cocotbext/eth/xgmii.py

@@ -222,14 +222,14 @@ class XgmiiSource(Reset):
             if self._run_cr is not None:
                 self._run_cr.kill()
                 self._run_cr = None
-        else:
-            self.log.info("Reset de-asserted")
-            if self._run_cr is None:
-                self._run_cr = cocotb.scheduler.start_soon(self._run())
 
             self.active = False
             self.data <= 0
             self.ctrl <= 0
+        else:
+            self.log.info("Reset de-asserted")
+            if self._run_cr is None:
+                self._run_cr = cocotb.scheduler.start_soon(self._run())
 
     async def _run(self):
         frame = None
@@ -401,13 +401,13 @@ class XgmiiSink(Reset):
             if self._run_cr is not None:
                 self._run_cr.kill()
                 self._run_cr = None
+
+            self.active = False
         else:
             self.log.info("Reset de-asserted")
             if self._run_cr is None:
                 self._run_cr = cocotb.scheduler.start_soon(self._run())
 
-        self.active = False
-
     async def _run(self):
         frame = None
         self.active = False