From 6c9c45f31308b57fe5d4235af94a9848639c37cb Mon Sep 17 00:00:00 2001
From: Florent Kermarrec <florent@enjoy-digital.fr>
Date: Tue, 3 Dec 2019 12:16:50 +0100
Subject: [PATCH] core/controller: cleanup ControllerSettings

---
 litedram/core/controller.py | 19 +++++++++++++++++--
 1 file changed, 17 insertions(+), 2 deletions(-)

diff --git a/litedram/core/controller.py b/litedram/core/controller.py
index 4c03449..2c95f48 100644
--- a/litedram/core/controller.py
+++ b/litedram/core/controller.py
@@ -16,12 +16,27 @@ from litedram.core.multiplexer import Multiplexer
 
 class ControllerSettings(Settings):
     def __init__(self,
-        cmd_buffer_depth    = 8, cmd_buffer_buffered=False,
+        # Command buffers
+        cmd_buffer_depth    = 8,
+        cmd_buffer_buffered = False,
+
+        # Read/Write times
         read_time           = 32,
         write_time          = 16,
+
+        # Bandwidth
         with_bandwidth      = False,
-        with_refresh        = True, refresh_cls=Refresher, refresh_zqcs_freq=1e0, refresh_postponing=1,
+
+        # Refresh
+        with_refresh        = True,
+        refresh_cls         = Refresher,
+        refresh_zqcs_freq   = 1e0,
+        refresh_postponing  = 1,
+
+        # Auto-Precharge
         with_auto_precharge = True,
+
+        # Address mapping
         address_mapping     = "ROW_BANK_COL"):
         self.set_attributes(locals())