diff --git a/applications/lofar2/libraries/sdp/src/vhdl/sdp_bdo_pkg.vhd b/applications/lofar2/libraries/sdp/src/vhdl/sdp_bdo_pkg.vhd index 33df8e23bce60049b8e93a075628e96df6a7e400..d19cf03a6bc05de9ef2f36ef7f76b668869b5d9e 100644 --- a/applications/lofar2/libraries/sdp/src/vhdl/sdp_bdo_pkg.vhd +++ b/applications/lofar2/libraries/sdp/src/vhdl/sdp_bdo_pkg.vhd @@ -59,16 +59,12 @@ package sdp_bdo_pkg is -- nof_destinations > 1 and how many block RAM resources are available for -- the reordering. Therefore c_sdp_bdo_reorder_nof_blocks_max = 16 is a -- suitable compromise value. - -- Conclusion: + -- Conclusion (14 feb 2024): -- . Choose c_sdp_bdo_reorder_nof_blocks_max = 4 to save block RAM resources, -- and because it is acceptable to have smaller packets in case of more -- than 4 destinations. Note that the reorder buffer is dual page. -- . Choose nof_bdo_destinations_max = 8 in design revision, because it is - -- enough for CEP and to save logic resources and to ease timing closure. - -- Remark: - -- . The c_sdp_bdo_reorder_nof_blocks_max must be <= - -- nof_bdo_destinations_max, because otherwise this probably caused - -- Quartus synthesis to yield an segmentation error. + -- enough for CEP, and to save logic resources and to ease timing closure. constant c_sdp_bdo_reorder_nof_blocks_max : natural := 4; constant c_sdp_bdo_reorder_nof_blocks_w : natural := ceil_log2(c_sdp_bdo_reorder_nof_blocks_max + 1); constant c_sdp_bdo_reorder_nof_ch_max : natural := c_sdp_bdo_reorder_nof_blocks_max *