From: Florent Kermarrec <florent@enjoy-digital.fr>
Date: Sat, 21 Mar 2015 16:44:04 +0000 (+0100)
Subject: sdram/module: add description and TODO list
X-Git-Tag: 24jan2021_ls180~2464
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=854058a8db9cb344fade3a270a9df6eea502a14b;p=litex.git

sdram/module: add description and TODO list
---

diff --git a/misoclib/mem/sdram/module.py b/misoclib/mem/sdram/module.py
index 3cafb4a5..d7cefb27 100644
--- a/misoclib/mem/sdram/module.py
+++ b/misoclib/mem/sdram/module.py
@@ -1,3 +1,19 @@
+# SDRAM memory modules library
+#
+# This library avoid duplications of memory modules definitions in targets and
+# ease SDRAM usage. (User can only select an already existing module or create
+# one for its board and contribute to this library)
+#
+# TODO:
+#   Try to share the maximum information we can between modules:
+#    - ex: MT46V32M16 and MT46H32M16 are almost identical (V=DDR, H=LPDDR)
+#    - Modules can have different configuration:
+#        MT8JTF12864 (1GB), MT8JTF25664 (2GB)
+#      but share all others informations, try to create an unique module for all
+#      configurations.
+#    - Modules can have different speedgrades, add support for it (and also add
+#      a check to verify clk_freq is in the supported range)
+
 from math import ceil
 
 from migen.fhdl.std import *
@@ -42,7 +58,8 @@ class IS42S16160(SDRAMModule):
 		"tRFC":		70
 	}
 	def __init__(self, clk_freq):
-		SDRAMModule.__init__(self, clk_freq, self.geom_settings, self.timing_settings)
+		SDRAMModule.__init__(self, clk_freq, self.geom_settings,
+			self.timing_settings)
 
 class MT48LC4M16(SDRAMModule):
 	geom_settings = {
@@ -59,7 +76,8 @@ class MT48LC4M16(SDRAMModule):
 		"tRFC":		66
 	}
 	def __init__(self, clk_freq):
-		SDRAMModule.__init__(self, clk_freq, self.geom_settings, self.timing_settings)
+		SDRAMModule.__init__(self, clk_freq, self.geom_settings,
+			self.timing_settings)
 
 # DDR
 class MT46V32M16(SDRAMModule):
@@ -77,7 +95,8 @@ class MT46V32M16(SDRAMModule):
 		"tRFC":		70
 	}
 	def __init__(self, clk_freq):
-		SDRAMModule.__init__(self, clk_freq, self.geom_settings, self.timing_settings)
+		SDRAMModule.__init__(self, clk_freq, self.geom_settings,
+			self.timing_settings)
 
 # LPDDR
 class MT46H32M16(SDRAMModule):
@@ -95,7 +114,8 @@ class MT46H32M16(SDRAMModule):
 		"tRFC":		72
 	}
 	def __init__(self, clk_freq):
-		SDRAMModule.__init__(self, clk_freq, self.geom_settings, self.timing_settings)
+		SDRAMModule.__init__(self, clk_freq, self.geom_settings,
+			self.timing_settings)
 
 # DDR2
 
@@ -115,4 +135,5 @@ class MT8JTF12864(SDRAMModule):
 		"tRFC":		70
 	}
 	def __init__(self, clk_freq):
-		SDRAMModule.__init__(self, clk_freq, self.geom_settings, self.timing_settings)
+		SDRAMModule.__init__(self, clk_freq, self.geom_settings,
+			self.timing_settings)