From 9509d9e361dc85ac709130788e022240ce7d2182 Mon Sep 17 00:00:00 2001
From: Florent Kermarrec <florent@enjoy-digital.fr>
Date: Mon, 24 Jul 2017 13:38:12 +0200
Subject: [PATCH] gen/genlib/cdc/gearbox: fix possible pointers overlap by
 removing AsyncResetSynchronizers.

read/write clocks don't have the same frequencies, using AsyncResetSynchronizers cause differents delay when releasing reset and can cause pointers overlap.
---
 litex/gen/genlib/cdc.py | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/litex/gen/genlib/cdc.py b/litex/gen/genlib/cdc.py
index 1fa66543f..3ba1a2627 100644
--- a/litex/gen/genlib/cdc.py
+++ b/litex/gen/genlib/cdc.py
@@ -211,17 +211,15 @@ class Gearbox(Module):
 
         # # #
 
-        reset = Signal()
+        rst = Signal()
         cd_write = ClockDomain()
         cd_read = ClockDomain()
         self.comb += [
+            rst.eq(ResetSignal(idomain) | ResetSignal(odomain)),
             cd_write.clk.eq(ClockSignal(idomain)),
             cd_read.clk.eq(ClockSignal(odomain)),
-            reset.eq(ResetSignal(idomain) | ResetSignal(odomain))
-        ]
-        self.specials += [
-            AsyncResetSynchronizer(cd_write, reset),
-            AsyncResetSynchronizer(cd_read, reset)
+            cd_write.rst.eq(rst),
+            cd_read.rst.eq(rst)
         ]
         self.clock_domains += cd_write, cd_read