From f301b6f13c95a578218703ce4cc5bb3edd8a0c3e Mon Sep 17 00:00:00 2001
From: Max Korbel <max.korbel@intel.com>
Date: Wed, 19 Feb 2025 16:30:05 -0800
Subject: [PATCH] cleanup, add doc

---
 doc/components/memory.md             | 2 ++
 test/sparse_memory_storage_test.dart | 3 ---
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/doc/components/memory.md b/doc/components/memory.md
index abe88456c..cf8c33438 100644
--- a/doc/components/memory.md
+++ b/doc/components/memory.md
@@ -21,3 +21,5 @@ The read path is combinational, so data is provided immediately according to the
 ## Memory Models
 
 The `MemoryModel` has the same interface as a `Memory`, but is non-synthesizable and uses a software-based `SparseMemoryStorage` as a backing for data storage. This is a useful tool for testing systems that have relatively large memories.
+
+The `MemoryStorage` class also provides utilities for reading (`loadMemString`) and writing (`dumpMemString`) verilog-compliant memory files (e.g. for `readmemh`).
diff --git a/test/sparse_memory_storage_test.dart b/test/sparse_memory_storage_test.dart
index ded5cd9e4..3cde4da38 100644
--- a/test/sparse_memory_storage_test.dart
+++ b/test/sparse_memory_storage_test.dart
@@ -138,7 +138,4 @@ void main() {
       expect(memStr, expected);
     });
   });
-
-  // TODO Testplan:
-  //  - radix 2, 16
 }