Lynn, Owen

Scratch DDR SDRAM Controller

There appears to be a bug in the control logic that allows the core to lock up.  In either the read or write command, the first state sets an op and waits for an  ack.  Then the next state waits for busy_n to be released.  Since the idle state is set up to clear the busy_n and to check for read or write commands (which set the busy_n) the core can get caught in STATE2 (for write) or STATE4 (for read)  There are lots of ways this could be fixed.  One way would be to clear the op code after it has been acknowledged.  This would need to be done in both STATE1 and STATE3, with the STATE1 code shown below.

when STATE1 =>
	op <= "10";
	addr <= "0000000000"& x"6001";
	data_i <= "11110001";
	if (op_ack = '1') then
-- begin new code
		op <= "00";
-- end new code          
		dram_driver_state <= STATE2;
	end if;

Thanks,

James

op not cleared before second part of read or write command

Here is an patch to clean up a little bit with librarys:

Index: sdram_support.vhd
===================================================================
--- sdram_support.vhd	(revision 24)
+++ sdram_support.vhd	(working copy)
@@ -22,13 +22,7 @@
 ----------------------------------------------------------------------------------
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 
----- Uncomment the following library declaration if instantiating
----- any Xilinx primitives in this code.
---library UNISIM;
---use UNISIM.VComponents.all;
 
 entity cmd_bank_addr_switch is
 	port(
@@ -57,13 +51,8 @@
 
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
+use IEEE.numeric_std.ALL;
 
----- Uncomment the following library declaration if instantiating
----- any Xilinx primitives in this code.
-library UNISIM;
-use UNISIM.VComponents.all;
 
 entity wait_counter is
 	generic(
@@ -79,7 +68,7 @@
 
 architecture impl of wait_counter is
 	
-	signal reg : std_logic_vector(BITS-1 downto 0);
+	signal reg : unsigned(BITS-1 downto 0) := (others => '0');
 	
 begin
 
@@ -87,7 +76,7 @@
 	begin
 		if (rst = '1') then
 			done <= '0';
-			reg <= CONV_STD_LOGIC_VECTOR(CLKS, BITS);
+			reg <= to_unsigned(CLKS, BITS);
 		elsif (rising_edge(clk)) then
 			if (reg > x"00") then
 				done <= '0';
@@ -103,13 +92,11 @@
 
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 
----- Uncomment the following library declaration if instantiating
----- any Xilinx primitives in this code.
 library UNISIM;
-use UNISIM.VComponents.all;
+use UNISIM.VComponents.dcm_sp;
+use UNISIM.VComponents.bufg;
+use UNISIM.VComponents.oddr2;
 
 entity sdram_dcm is
 	port(
@@ -235,13 +222,9 @@
 
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 
----- Uncomment the following library declaration if instantiating
----- any Xilinx primitives in this code.
 library UNISIM;
-use UNISIM.VComponents.all;
+use UNISIM.VComponents.oddr2;
 
 -- just a 2 bit wide ODDR2
 entity oddr2_2 is
@@ -295,13 +278,9 @@
 
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 
----- Uncomment the following library declaration if instantiating
----- any Xilinx primitives in this code.
 library UNISIM;
-use UNISIM.VComponents.all;
+use UNISIM.VComponents.oddr2;
 
 -- just a 3 bit wide ODDR2
 entity oddr2_3 is
@@ -371,13 +350,7 @@
 
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 
----- Uncomment the following library declaration if instantiating
----- any Xilinx primitives in this code.
-library UNISIM;
-use UNISIM.VComponents.all;
 
 -- 2 oddr2_2's
 entity oddr2_4 is
@@ -434,13 +407,9 @@
 	
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 
----- Uncomment the following library declaration if instantiating
----- any Xilinx primitives in this code.
 library UNISIM;
-use UNISIM.VComponents.all;
+use UNISIM.VComponents.oddr2;
 
 -- one ODDR2 and 3 4-bit oddr2_4's
 entity oddr2_13 is
@@ -527,13 +496,7 @@
 
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 
----- Uncomment the following library declaration if instantiating
----- any Xilinx primitives in this code.
-library UNISIM;
-use UNISIM.VComponents.all;
 
 -- 4 4-bit oddr2_4's
 entity oddr2_16 is
@@ -614,13 +577,7 @@
 
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 
----- Uncomment the following library declaration if instantiating
----- any Xilinx primitives in this code.
-library UNISIM;
-use UNISIM.VComponents.all;
 
 entity inout_switch_2 is
 	port (
@@ -638,13 +595,7 @@
 
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 
----- Uncomment the following library declaration if instantiating
----- any Xilinx primitives in this code.
-library UNISIM;
-use UNISIM.VComponents.all;
 
 entity inout_switch_16 is
 	port (
Index: sdram_writer.vhd
===================================================================
--- sdram_writer.vhd	(revision 24)
+++ sdram_writer.vhd	(working copy)
@@ -22,14 +22,7 @@
 ----------------------------------------------------------------------------------
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 
----- Uncomment the following library declaration if instantiating
----- any Xilinx primitives in this code.
---library UNISIM;
---use UNISIM.VComponents.all;
-
 -- Uses ODDR2 registers to generate the required DDR signals. Don't have to be as
 --  careful with the timings as with sdram_reader, but you need to be able to feed
 --  the ODDR2's within their setup and hold windows. Or very very hilarious things
Index: sdram.vhd
===================================================================
--- sdram.vhd	(revision 24)
+++ sdram.vhd	(working copy)
@@ -22,8 +22,6 @@
 ----------------------------------------------------------------------------------
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 
 ---- Uncomment the following library declaration if instantiating
 ---- any Xilinx primitives in this code.
Index: scratch.vhd
===================================================================
--- scratch.vhd	(revision 24)
+++ scratch.vhd	(working copy)
@@ -22,10 +22,9 @@
 ----------------------------------------------------------------------------------
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 library UNISIM;
-use UNISIM.VComponents.all;
+use UNISIM.VComponents.bufg;
+use UNISIM.VComponents.dcm_sp;
 
 
 entity scratch is
Index: sdram_reader.vhd
===================================================================
--- sdram_reader.vhd	(revision 24)
+++ sdram_reader.vhd	(working copy)
@@ -21,14 +21,7 @@
 ----------------------------------------------------------------------------------
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 
----- Uncomment the following library declaration if instantiating
----- any Xilinx primitives in this code.
---library UNISIM;
---use UNISIM.VComponents.all;
-
 -- I strongly suggest you run this in the post-PAR simulator first and then start
 --  making changes to it after looking at what goes on at the post-PAR level. Don't
 --  say I didn't warn you. 
@@ -80,4 +73,4 @@
 		end if;
 	end process;
 
-end impl;
\ No newline at end of file
+end impl;
Index: sdram_init.vhd
===================================================================
--- sdram_init.vhd	(revision 24)
+++ sdram_init.vhd	(working copy)
@@ -22,14 +22,7 @@
 ----------------------------------------------------------------------------------
 library IEEE;
 use IEEE.STD_LOGIC_1164.ALL;
-use IEEE.STD_LOGIC_ARITH.ALL;
-use IEEE.STD_LOGIC_UNSIGNED.ALL;
 
----- Uncomment the following library declaration if instantiating
----- any Xilinx primitives in this code.
---library UNISIM;
---use UNISIM.VComponents.all;
-
 entity sdram_init is
 	port(
 		clk_000 : in std_logic;
Index: scratch_isim_tb.vhd
===================================================================
--- scratch_isim_tb.vhd	(revision 24)
+++ scratch_isim_tb.vhd	(working copy)
@@ -29,11 +29,8 @@
 --------------------------------------------------------------------------------
 LIBRARY ieee;
 USE ieee.std_logic_1164.ALL;
-USE ieee.std_logic_unsigned.all;
 USE ieee.numeric_std.ALL;
  
-library UNISIM;
-use UNISIM.VComponents.all;
 
 ENTITY scratch_isim_tb IS
 	port(
@@ -145,27 +143,7 @@
 	);
 		  
 
-   -- Clock process definitions
-   clk_process :process
-   begin
-		clk <= '0';
-		wait for clk_period/2;
-		clk <= '1';
-		wait for clk_period/2;
-   end process;
- 
+   clk <= not clk after clk_period/2;
+   rst <= '1', '0' after 5 * clk_period;
 
-   -- Stimulus process
-   stim_proc: process
-   begin		
-      -- hold reset state for 100ms.
-      wait for 100ms;	
-
-      wait for clk_period*10;
-
-      -- insert stimulus here 
-
-      wait;
-   end process;
-
 END;

switch to std librarys and library cleanup

The DCM didn't lok in simulation beacuse the reset is fixed to '0'.

Add the following line in scratch_isim_tb.vhd will help:

rst <= '1', '0' after 5 * clk_period;

regards

DCM dont lock

Hi everyone,

I downloaded that project, used ISE11.5 first and after downloading the bit-file to the spartan e3 starterboard no led were flashing.
I downgraded to ISE11.1 and tried it again and then it worked.
But downloading the same bit-file at a different time nothing is working.
I thought about a problem with my board and tried a microblaze design, there the ddr is working reliable.

The chance the leds flashes are about 5%.
So is that a bug during initialization, does anyone else have the same problem?
Any suggestions?

Thank you.

In regards
Steffen Mauch

not working reliable

I saw another post mentioning that to use this controller, it requires an external 100mhz clock.  Is this true?

It synthesized, etc etc and programmed on my Spartan3E 500 board like a charm! One of the few projects that does that direct-from-download w/ no tweaking.

I'm getting L->R on the LEDS ON ON ON ON OFF ON OFF ON.

In order to use this code, where does a hobbyist get a 100mhz clock?  I mean for use with the xilinx starter kit?  I suppose I need some type of function generator?

Appreciate any help!

Keith

Browse

Scratch DDR SDRAM Controller