From d82beee5408082b5f25187d5f1c48fe63068e111 Mon Sep 17 00:00:00 2001
From: Sebastien Bourdeauducq <sb@m-labs.hk>
Date: Tue, 10 Jul 2018 17:04:07 +0800
Subject: [PATCH] grabber: make parser EOP a pulse

---
 artiq/gateware/grabber/core.py | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/artiq/gateware/grabber/core.py b/artiq/gateware/grabber/core.py
index f9559071c..e300f6541 100644
--- a/artiq/gateware/grabber/core.py
+++ b/artiq/gateware/grabber/core.py
@@ -34,8 +34,8 @@ class Parser(Module, AutoCSR):
             ("a", 8),
             ("b", 8),
             ("c", 8),
-            ("stb", 1),  # dval
-            ("eop", 1),  # ~fval (i.e. not together with stb)
+            ("stb", 1),
+            ("eop", 1),
         ])
 
         # # #
@@ -46,14 +46,14 @@ class Parser(Module, AutoCSR):
         lval = Signal()
         fval = Signal()
         dval = Signal()
+        last_lval = Signal()
+        last_fval = Signal()
         self.comb += [
             Cat(dval, fval, lval).eq(cl[14:17]),
             pix.stb.eq(dval),
-            pix.eop.eq(~fval),
+            pix.eop.eq(~fval & last_fval),
             Cat(pix.a, pix.b, pix.c).eq(Cat(cl[i] for i in bitseq))
         ]
-        last_lval = Signal()
-        last_fval = Signal()
         self.sync.cl += [
             last_lval.eq(lval),
             last_fval.eq(fval),