Skip to content

Commit 407cfcb

Browse files
committed
Merge branch 'valek-docs-dp_uram_impl_note' into 'devel'
docs(dp-uram-xilinx): add TODOs for future improvements See merge request ndk/ndk-fpga!325
2 parents 17eafd1 + c8cc081 commit 407cfcb

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

comp/base/mem/dp_uram_xilinx/dp_uram_xilinx_arch.vhd

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,15 @@
99
--
1010
-- $Id$
1111
--
12-
-- TODO:
13-
--
12+
-- TODO: This implementation should be refactored and polished:
13+
-- 1. The DEVICE generic set to BEHAVIORAL is not used anywhere in the NDK platform since
14+
-- this is already provided by behavioral architectures like DP_BRAM_BEHAV and can be
15+
-- easily obtained within Vivado's language templates.
16+
-- 2. The simulation should be replaced by CocoTB or UVM verification to properly test this
17+
-- fundamental component.
18+
-- 3. A byte-enable feature should be connected and made available on the WEA and WEB
19+
-- ports since URAM macro xpm_memory_tdpram does support this by setting BYTE_WRITE_WIDTH_A
20+
-- and BYTE_WRITE_WIDTH_B to 8.
1421
--
1522
library IEEE;
1623
use IEEE.std_logic_1164.all;

0 commit comments

Comments
 (0)