A bug fix about Initvals in RAM

Author: shtaxxx

tests/extension/thread_/ram_initval/test_thread_ram_initval.py

@@ -1204,106 +1204,106 @@
     mem[921] = 32'h3a3;
     mem[922] = 32'h3a4;
     mem[923] = 32'h3a5;
-    mem[924] = 32'h3a6;
-    mem[925] = 32'h3a7;
-    mem[926] = 32'h3a8;
-    mem[927] = 32'h3a9;
-    mem[928] = 32'h3aa;
-    mem[929] = 32'h3ab;
-    mem[930] = 32'h3ac;
-    mem[931] = 32'h3ad;
-    mem[932] = 32'h3ae;
-    mem[933] = 32'h3af;
-    mem[934] = 32'h3b0;
-    mem[935] = 32'h3b1;
-    mem[936] = 32'h3b2;
-    mem[937] = 32'h3b3;
-    mem[938] = 32'h3b4;
-    mem[939] = 32'h3b5;
-    mem[940] = 32'h3b6;
-    mem[941] = 32'h3b7;
-    mem[942] = 32'h3b8;
-    mem[943] = 32'h3b9;
-    mem[944] = 32'h3ba;
-    mem[945] = 32'h3bb;
-    mem[946] = 32'h3bc;
-    mem[947] = 32'h3bd;
-    mem[948] = 32'h3be;
-    mem[949] = 32'h3bf;
-    mem[950] = 32'h3c0;
-    mem[951] = 32'h3c1;
-    mem[952] = 32'h3c2;
-    mem[953] = 32'h3c3;
-    mem[954] = 32'h3c4;
-    mem[955] = 32'h3c5;
-    mem[956] = 32'h3c6;
-    mem[957] = 32'h3c7;
-    mem[958] = 32'h3c8;
-    mem[959] = 32'h3c9;
-    mem[960] = 32'h3ca;
-    mem[961] = 32'h3cb;
-    mem[962] = 32'h3cc;
-    mem[963] = 32'h3cd;
-    mem[964] = 32'h3ce;
-    mem[965] = 32'h3cf;
-    mem[966] = 32'h3d0;
-    mem[967] = 32'h3d1;
-    mem[968] = 32'h3d2;
-    mem[969] = 32'h3d3;
-    mem[970] = 32'h3d4;
-    mem[971] = 32'h3d5;
-    mem[972] = 32'h3d6;
-    mem[973] = 32'h3d7;
-    mem[974] = 32'h3d8;
-    mem[975] = 32'h3d9;
-    mem[976] = 32'h3da;
-    mem[977] = 32'h3db;
-    mem[978] = 32'h3dc;
-    mem[979] = 32'h3dd;
-    mem[980] = 32'h3de;
-    mem[981] = 32'h3df;
-    mem[982] = 32'h3e0;
-    mem[983] = 32'h3e1;
-    mem[984] = 32'h3e2;
-    mem[985] = 32'h3e3;
-    mem[986] = 32'h3e4;
-    mem[987] = 32'h3e5;
-    mem[988] = 32'h3e6;
-    mem[989] = 32'h3e7;
-    mem[990] = 32'h3e8;
-    mem[991] = 32'h3e9;
-    mem[992] = 32'h3ea;
-    mem[993] = 32'h3eb;
-    mem[994] = 32'h3ec;
-    mem[995] = 32'h3ed;
-    mem[996] = 32'h3ee;
-    mem[997] = 32'h3ef;
-    mem[998] = 32'h3f0;
-    mem[999] = 32'h3f1;
-    mem[1000] = 32'h3f2;
-    mem[1001] = 32'h3f3;
-    mem[1002] = 32'h3f4;
-    mem[1003] = 32'h3f5;
-    mem[1004] = 32'h3f6;
-    mem[1005] = 32'h3f7;
-    mem[1006] = 32'h3f8;
-    mem[1007] = 32'h3f9;
-    mem[1008] = 32'h3fa;
-    mem[1009] = 32'h3fb;
-    mem[1010] = 32'h3fc;
-    mem[1011] = 32'h3fd;
-    mem[1012] = 32'h3fe;
-    mem[1013] = 32'h3ff;
-    mem[1014] = 32'h400;
-    mem[1015] = 32'h401;
-    mem[1016] = 32'h402;
-    mem[1017] = 32'h403;
-    mem[1018] = 32'h404;
-    mem[1019] = 32'h405;
-    mem[1020] = 32'h406;
-    mem[1021] = 32'h407;
-    mem[1022] = 32'h408;
-    mem[1023] = 32'h409;
+    mem[924] = 32'h0;
+    mem[925] = 32'h0;
+    mem[926] = 32'h0;
+    mem[927] = 32'h0;
+    mem[928] = 32'h0;
+    mem[929] = 32'h0;
+    mem[930] = 32'h0;
+    mem[931] = 32'h0;
+    mem[932] = 32'h0;
+    mem[933] = 32'h0;
+    mem[934] = 32'h0;
+    mem[935] = 32'h0;
+    mem[936] = 32'h0;
+    mem[937] = 32'h0;
+    mem[938] = 32'h0;
+    mem[939] = 32'h0;
+    mem[940] = 32'h0;
+    mem[941] = 32'h0;
+    mem[942] = 32'h0;
+    mem[943] = 32'h0;
+    mem[944] = 32'h0;
+    mem[945] = 32'h0;
+    mem[946] = 32'h0;
+    mem[947] = 32'h0;
+    mem[948] = 32'h0;
+    mem[949] = 32'h0;
+    mem[950] = 32'h0;
+    mem[951] = 32'h0;
+    mem[952] = 32'h0;
+    mem[953] = 32'h0;
+    mem[954] = 32'h0;
+    mem[955] = 32'h0;
+    mem[956] = 32'h0;
+    mem[957] = 32'h0;
+    mem[958] = 32'h0;
+    mem[959] = 32'h0;
+    mem[960] = 32'h0;
+    mem[961] = 32'h0;
+    mem[962] = 32'h0;
+    mem[963] = 32'h0;
+    mem[964] = 32'h0;
+    mem[965] = 32'h0;
+    mem[966] = 32'h0;
+    mem[967] = 32'h0;
+    mem[968] = 32'h0;
+    mem[969] = 32'h0;
+    mem[970] = 32'h0;
+    mem[971] = 32'h0;
+    mem[972] = 32'h0;
+    mem[973] = 32'h0;
+    mem[974] = 32'h0;
+    mem[975] = 32'h0;
+    mem[976] = 32'h0;
+    mem[977] = 32'h0;
+    mem[978] = 32'h0;
+    mem[979] = 32'h0;
+    mem[980] = 32'h0;
+    mem[981] = 32'h0;
+    mem[982] = 32'h0;
+    mem[983] = 32'h0;
+    mem[984] = 32'h0;
+    mem[985] = 32'h0;
+    mem[986] = 32'h0;
+    mem[987] = 32'h0;
+    mem[988] = 32'h0;
+    mem[989] = 32'h0;
+    mem[990] = 32'h0;
+    mem[991] = 32'h0;
+    mem[992] = 32'h0;
+    mem[993] = 32'h0;
+    mem[994] = 32'h0;
+    mem[995] = 32'h0;
+    mem[996] = 32'h0;
+    mem[997] = 32'h0;
+    mem[998] = 32'h0;
+    mem[999] = 32'h0;
+    mem[1000] = 32'h0;
+    mem[1001] = 32'h0;
+    mem[1002] = 32'h0;
+    mem[1003] = 32'h0;
+    mem[1004] = 32'h0;
+    mem[1005] = 32'h0;
+    mem[1006] = 32'h0;
+    mem[1007] = 32'h0;
+    mem[1008] = 32'h0;
+    mem[1009] = 32'h0;
+    mem[1010] = 32'h0;
+    mem[1011] = 32'h0;
+    mem[1012] = 32'h0;
+    mem[1013] = 32'h0;
+    mem[1014] = 32'h0;
+    mem[1015] = 32'h0;
+    mem[1016] = 32'h0;
+    mem[1017] = 32'h0;
+    mem[1018] = 32'h0;
+    mem[1019] = 32'h0;
+    mem[1020] = 32'h0;
+    mem[1021] = 32'h0;
+    mem[1022] = 32'h0;
+    mem[1023] = 32'h0;
   end
 
 

tests/extension/thread_/ram_initval/thread_ram_initval.py

@@ -19,7 +19,7 @@ def mkLed():
     datawidth = 32
     addrwidth = 10
     numports = 1
-    initvals = [i + 10 for i in range(2 ** addrwidth)]
+    initvals = [i + 10 for i in range(2 ** addrwidth - 100)]
     myram = vthread.RAM(m, 'myram', clk, rst, datawidth, addrwidth,
                         numports, initvals)
 

veriloggen/types/ram.py

@@ -45,9 +45,9 @@ def mkRAMDefinition(name, datawidth=32, addrwidth=10, numports=2,
         initvals = new_initvals
 
         if 2 ** addrwidth > len(initvals):
-            initvals = list(initvals).extend(
+            initvals.extend(
                 [vtypes.Int(0, datawidth, base=16)
-                 for _ in 2 ** addrwidth - len(initvals)])
+                 for _ in range(2 ** addrwidth - len(initvals))])
 
         m.Initial(
             *[mem[i](initval) for i, initval in enumerate(initvals)]