From 386d20a5918a5efdd3cf37d0ab141f352ae509a9 Mon Sep 17 00:00:00 2001
From: Erik Kooistra <kooistra@astron.nl>
Date: Thu, 20 Oct 2016 15:22:38 +0000
Subject: [PATCH] Added remarks on the reorder memories.

---
 libraries/dsp/fft/src/vhdl/fft_r2_wide.vhd | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/libraries/dsp/fft/src/vhdl/fft_r2_wide.vhd b/libraries/dsp/fft/src/vhdl/fft_r2_wide.vhd
index 5581a43862..72ecd6d70d 100644
--- a/libraries/dsp/fft/src/vhdl/fft_r2_wide.vhd
+++ b/libraries/dsp/fft/src/vhdl/fft_r2_wide.vhd
@@ -49,7 +49,7 @@
 --    When use_separate = true then the use_fft_shift must be false, because
 --    fft_shift() only applies to spectra for complex input.
 --
--- Remark:
+-- Remarks:
 -- . This fft_r2_wide does not (yet) support wb_factor = 1 (= only a
 --   fft_r2_pipe instance) or wb_factor = g_fft.nof_points (= only a
 --   fft_r2_par instance). Fixing this is nice to have, but not essential.
@@ -59,6 +59,14 @@
 --   . wb_factor = 1
 --   . wb_factor > 1 AND wb_factor < g_fft.nof_points
 --   . wb_factor = g_fft.nof_points.
+-- . This fft_r2_wide uses the use_reorder in the pipeline FFT, in the parallel
+--   FFT and also has reorder memory in the fft_sepa_wide instance. The reorder
+--   memories in the FFTs can maybe be saved by using only the reorder memory
+--   in the fft_sepa_wide instance. This would require changing the indexing in
+--   fft_sepa_wide instance.
+-- . The reorder memory in the pipeline FFT, parallel FFT and in the
+--   fft_sepa_wide could make reuse of a reorder component from the reorder
+--   library instead of using a dedicated local solution.
 
 library ieee, common_lib, rTwoSDF_lib;
 use IEEE.std_logic_1164.all;
-- 
GitLab