From acef5f523b6f92048bbaab98b2873b0382d5d1f2 Mon Sep 17 00:00:00 2001
From:  <john@csquare.ca>
Date: Thu, 11 Oct 2018 16:56:36 -0400
Subject: [PATCH] Fix issue where we ignore important timing parameters.

---
 litedram/core/bankmachine.py | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/litedram/core/bankmachine.py b/litedram/core/bankmachine.py
index 5b6c52a..42b81bb 100644
--- a/litedram/core/bankmachine.py
+++ b/litedram/core/bankmachine.py
@@ -216,18 +216,20 @@ class BankMachine(Module):
             track_close.eq(1)
         )
         fsm.act("ACTIVATE",
-            sel_row_addr.eq(1),
-            track_open.eq(1),
-            cmd.valid.eq(1),
-            cmd.is_cmd.eq(1),
-            cmd.is_activate.eq(1),
-            If(cmd.ready,
-                NextState("TRCD")
-            ),
-            cmd.ras.eq(1)
+            If(activate_allowed,
+                sel_row_addr.eq(1),
+                track_open.eq(1),
+                cmd.valid.eq(1),
+                cmd.is_cmd.eq(1),
+                cmd.is_activate.eq(1),
+                If(cmd.ready,
+                    NextState("TRCD")
+                ),
+                cmd.ras.eq(1)
+            )
         )
         fsm.act("REFRESH",
-            If(twtpcon.ready,
+            If(twtpcon.ready & precharge_allowed,
                 self.refresh_gnt.eq(1),
             ),
             track_close.eq(1),