From 35f670dae971bc315ed85246f0e01a49c3a05df8 Mon Sep 17 00:00:00 2001
From: matt <matt@221aa14e-8319-0410-a670-987f0aec2ac5>
Date: Thu, 26 Feb 2009 04:44:02 +0000
Subject: timing fix.  The line address in the buffers still updates now even
 if there is an error.  Doesn't matter, since the error means the buffer is
 useless anyway.  This makes meeting timing much easier since the address
 update does not depend on the error signal which comes late.

git-svn-id: http://gnuradio.org/svn/gnuradio/trunk@10524 221aa14e-8319-0410-a670-987f0aec2ac5
---
 control_lib/buffer_int.v | 62 +++++++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 61 insertions(+), 1 deletion(-)

(limited to 'control_lib')

diff --git a/control_lib/buffer_int.v b/control_lib/buffer_int.v
index e362d93f2..c33f2779d 100644
--- a/control_lib/buffer_int.v
+++ b/control_lib/buffer_int.v
@@ -131,6 +131,8 @@ module buffer_int
 	   
 	   WRITING :
 	     begin
+		if(wr_write_i)
+		  addr_o <= addr_o + 1;  // This was the timing problem, so now it doesn't depend on wr_error_i
 		if(wr_error_i)
 		  begin
 		     state <= ERROR;
@@ -141,7 +143,6 @@ module buffer_int
 		     if(wr_write_i)
 		       begin
 			  wr_ready_o <= 0;
-			  addr_o <= addr_o + 1;
 			  if(addr_o == (lastline-1))
 			    wr_full_o <= 1;
 			  if(addr_o == lastline)
@@ -176,6 +177,65 @@ module buffer_int
    assign     idle = (state == IDLE);
 endmodule // buffer_int
 
+
+
+// These are 2 other ways for doing the WRITING state, both work.  First one is faster, but confusing
+/*
+	     begin
+		// Gen 4 values -- state, wr_ready_o, addr_o, wr_full_o
+		if(~wr_error_i & wr_write_i & (addr_o == (lastline-1)))
+		  wr_full_o <= 1;
+		if(wr_error_i | wr_write_i | wr_done_i)
+		  wr_ready_o <= 0;
+		if(wr_error_i)
+		  state <= ERROR;
+		else if(wr_done_i | (wr_write_i & (addr_o == lastline)))
+		  state <= DONE;
+		// This one was the timing problem...  now we increment addr_o even if there is an error
+		if(wr_write_i)
+		  addr_o <= addr_o + 1;
+	     end // case: WRITING
+*/	   
+		
+/*      begin
+		if(wr_error_i)
+		  begin
+		     state <= ERROR;
+		     wr_ready_o <= 0;
+		  end
+		else
+		  begin
+		     if(wr_write_i)
+		       begin
+			  wr_ready_o <= 0;
+			  addr_o <= addr_o + 1;
+			  if(addr_o == (lastline-1))
+			    wr_full_o <= 1;
+			  if(addr_o == lastline)
+			    state <= DONE;
+		       end
+		     if(wr_done_i)
+		       begin
+			  state <= DONE;
+			  wr_ready_o <= 0;
+		       end
+		  end // else: !if(wr_error_i)
+	     end // case: WRITING
+*/
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 // Unused old code
    //assign     rd_empty_o = (state != READING); // && (state != PRE_READ);
    //assign     rd_empty_o = rd_empty_reg;         // timing fix?
-- 
cgit v1.2.3