gotten over the logic-dyslexia of what in/out mean in VHDL.

BRAM can now be read/written using the contents of a file to initialise
from, at the command-line

diff --git a/Makefile b/Makefile
index 6bc4294731821dfb3fa869d89be9c7edb6349d07..7d0d320a7744c61bb16ddf1bd03bfe13f1402474 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -147,7 +147,7 @@ RAM_INIT_FILE=hello_world/hello_world.hex
 # Linux
 #MEMORY_SIZE=16777216 # 268435456
 #RAM_INIT_FILE=dtbImage.microwatt.hex
-SIM_MAIN_BRAM=true
+SIM_MAIN_BRAM=false
 
 FPGA_TARGET ?= ORANGE-CRAB
 

diff --git a/fpga/top-generic.vhdl b/fpga/top-generic.vhdl
index 6cec9fef243381a797fd2fc45b0f382a1ac2bacf..a9fb34ea012e330f27baea72be14c34c5566fb08 100644 (file)
--- a/fpga/top-generic.vhdl
+++ b/fpga/top-generic.vhdl
@@ -31,8 +31,8 @@ entity toplevel is
        bram_we : out std_ulogic;
        bram_re : out std_ulogic;
     bram_addr : out std_logic_vector(log2ceil(MEMORY_SIZE) - 3- 1 downto 0);
-    bram_di   : inout std_logic_vector(63 downto 0);
-    bram_do   : out std_logic_vector(63 downto 0);
+    bram_di   : out std_logic_vector(63 downto 0);
+    bram_do   : in std_logic_vector(63 downto 0);
     bram_sel  : out std_logic_vector(7 downto 0)
        );
 end entity toplevel;

diff --git a/soc.vhdl b/soc.vhdl
index 73d6292817986a1a46992b9fb9bd6d09e3e32945..352cfca7b1552dc7e7a375ec08268650d18c3d78 100644 (file)
--- a/soc.vhdl
+++ b/soc.vhdl
@@ -92,8 +92,8 @@ entity soc is
     bram_we : out std_ulogic;
     bram_re : out std_ulogic;
     bram_addr : out std_logic_vector(log2ceil(MEMORY_SIZE) - 3- 1 downto 0);
-    bram_di   : inout std_logic_vector(63 downto 0);
-    bram_do   : out std_logic_vector(63 downto 0);
+    bram_di   : out std_logic_vector(63 downto 0);
+    bram_do   : in std_logic_vector(63 downto 0);
     bram_sel  : out std_logic_vector(7 downto 0);
 
        -- UART0 signals:

diff --git a/verilator/microwatt-verilator.cpp b/verilator/microwatt-verilator.cpp
index e62962f1cb33880afe285ef6c70d87ca5d15b40d..a4a8f25b222922e0fec0d8f7d23f6f1bb70e6047 100644 (file)
--- a/verilator/microwatt-verilator.cpp
+++ b/verilator/microwatt-verilator.cpp
@@ -77,6 +77,8 @@ static void mem_write(unsigned char *mem,
     }
 }
 
+#define BRAM_DEBUG
+
 int main(int argc, char **argv)
 {
        Verilated::commandArgs(argc, argv);
@@ -143,14 +145,27 @@ int main(int argc, char **argv)
                tick(top);
        top->ext_rst = 1;
 
+    unsigned long long bram_do = 0;
+
        while(!Verilated::gotFinish()) {
                tick(top);
 
+        // read/write the memory to/from the mmap'd file (if given)
+        if (mem != NULL) {
+            top->bram_do = bram_do;
+            if (top->bram_re ) {
+                bram_do = ((unsigned long long*)mem)[top->bram_addr];
+            }
+            if (top->bram_we) {
+                mem_write(mem, top->bram_addr, top->bram_di, top->bram_sel);
+            }
+        }
+
                uart_tx(top->uart0_txd);
                top->uart0_rxd = uart_rx();
 
+#ifdef BRAM_DEBUG
         if (top->bram_we) {
-            mem_write(mem, top->bram_addr, top->bram_di, top->bram_sel);
             fprintf(dump, "bram wr addr %08x dout %16lx sel %x ",
                     top->bram_addr, top->bram_di, top->bram_sel);
             ascii_dump((unsigned char*)&top->bram_di, 8, dump);
@@ -160,7 +175,7 @@ int main(int argc, char **argv)
         if (next_read) {
             fprintf(dump, "bram rd addr %08x din %16lx sel %x ",
                     bram_addr, top->bram_do, top->bram_sel);
-            if (bram_data1 != top->bram_do) { // check contents
+            if ((mem != NULL) && bram_data1 != top->bram_do) { // check contents
                 fprintf(dump, "bram != %16lx ", bram_data1 );
             }
             ascii_dump((unsigned char*)&top->bram_do, 8, dump);
@@ -173,6 +188,7 @@ int main(int argc, char **argv)
         if ((mem != NULL) && next_read) {
             bram_data = ((unsigned long long*)mem)[bram_addr];
         }
+#endif // BRAM_DEBUG
        }
 
     fclose(dump);

diff --git a/wishbone_bram_wrapper.vhdl b/wishbone_bram_wrapper.vhdl
index fe26d588961e1c2aa22ceb5cf15591820d2646ee..4e3fb506dfecb66bb6bf165251957b5d2547f26d 100644 (file)
--- a/wishbone_bram_wrapper.vhdl
+++ b/wishbone_bram_wrapper.vhdl
@@ -26,8 +26,8 @@ entity wishbone_bram_wrapper is
     bram_we : out std_ulogic;
     bram_re : out std_ulogic;
     bram_addr : out std_logic_vector(log2ceil(MEMORY_SIZE) - 3- 1 downto 0);
-    bram_di   : inout std_logic_vector(63 downto 0);
-    bram_do   : out std_logic_vector(63 downto 0);
+    bram_di   : out std_logic_vector(63 downto 0);
+    bram_do   : in std_logic_vector(63 downto 0);
     bram_sel  : out std_logic_vector(7 downto 0)
 
        );
@@ -67,7 +67,7 @@ begin
 
     -- Verilator access to bram signals
     bram_sel <= wishbone_in.sel;
-    bram_do <= wishbone_out.dat;
+    wishbone_out.dat <= bram_do;
     bram_di <= wishbone_in.dat;
     bram_addr <= ram_addr;
     bram_we <= ram_we;