lpddr4/utils: rework `once` helper function to be more generic

litedram/phy/lpddr4/sim.py

@@ -9,7 +9,7 @@ from litex.soc.interconnect.stream import ClockDomainCrossing
 from litex.soc.interconnect.csr import AutoCSR
 
 from litedram.common import TappedDelayLine, tXXDController
-from litedram.phy.lpddr4.utils import delayed, once, SimLogger
+from litedram.phy.lpddr4.utils import delayed, edge, SimLogger
 from litedram.phy.lpddr4.commands import MPC
 
 
@@ -75,14 +75,11 @@ class PulseTiming(Module):
         tctrl = tXXDController(t)
         self.submodules += tctrl
 
-        self.sync += [
+        self.sync += If(self.trigger, triggered.eq(1)),
-            If(self.trigger, triggered.eq(1)),
-            ready_d.eq(tctrl.ready),
-        ]
         self.comb += [
             self.ready.eq(triggered & tctrl.ready),
-            self.ready_p.eq(self.ready & ~ready_d),
+            self.ready_p.eq(edge(self, self.ready)),
-            once(self, self.trigger, tctrl.valid.eq(1)),
+            tctrl.valid.eq(edge(self, self.trigger)),
         ]
 
 
@@ -223,7 +220,7 @@ class CommandsSim(Module, AutoCSR):  # clock domain: clk_p
         fsm.act("NORMAL",
             cmds_enabled.eq(1),
             self.tzqlat.trigger.eq(1),
-            once(self, init_delays & self.handle_cmd & ~self.tzqlat.ready,
+            If(init_delays & self.handle_cmd & ~self.tzqlat.ready,
                 self.log.warn("tZQLAT violated")
             ),
         )

litedram/phy/lpddr4/utils.py

@@ -24,10 +24,11 @@ def delayed(mod, sig, cycles=1, **kwargs):
     mod.submodules += delay
     return delay.output
 
-def once(mod, cond, *ops):
+def edge(mod, cond):
-    sig = Signal()
+    """Get a signal that is high on a rising edge of `cond`"""
-    mod.sync += If(cond, sig.eq(1))
+    cond_d = Signal()
-    return If(~sig & cond, *ops)
+    mod.sync += cond_d.eq(cond)
+    return  ~cond_d & cond
 
 class ConstBitSlip(Module):
     def __init__(self, dw, i=None, o=None, slp=None, cycles=1):
@@ -203,9 +204,7 @@ class SimLogger(Module, AutoCSR):
     def log(self, fmt, *args, level=DEBUG, once=True):
         cond = Signal()
         if once:  # make the condition be triggered only on rising edge
-            cond_d = Signal()
+            condition = edge(self, cond)
-            self.sync += cond_d.eq(cond)
-            condition = ~cond_d & cond
         else:
             condition = cond