From 1bc446a069a9c641291ea453f8c2700de36a4f99 Mon Sep 17 00:00:00 2001
From: Eric Kooistra <kooistra@astron.nl>
Date: Fri, 28 Oct 2022 10:50:13 +0200
Subject: [PATCH] Clarify c_fifo_size.

---
 libraries/io/eth/src/vhdl/eth_tester_rx.vhd | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/libraries/io/eth/src/vhdl/eth_tester_rx.vhd b/libraries/io/eth/src/vhdl/eth_tester_rx.vhd
index 59fbd897a5..425f3f39f1 100644
--- a/libraries/io/eth/src/vhdl/eth_tester_rx.vhd
+++ b/libraries/io/eth/src/vhdl/eth_tester_rx.vhd
@@ -63,6 +63,10 @@ ARCHITECTURE str OF eth_tester_rx IS
   CONSTANT c_nof_total_counts     : NATURAL := 3;  -- 0 = nof_sop, 1 = nof_valid, 2 = nof_crc_corrupt
 
   CONSTANT c_empty_w              : NATURAL := 2;  -- for 0, 1, 2, 3 empty octets per word
+
+  -- Rx FIFO size can be much less than rx_block_sz_max, because st_clk >
+  -- eth_clk rate, but with st level tx-rx loopback the Rx FIFO does need
+  -- rx_block_sz_max FIFO size.
   CONSTANT rx_block_sz_max        : NATURAL := c_eth_tester_rx_block_len_max / c_word_sz;  -- = 9004 / 4 = 2251
   CONSTANT c_fifo_size            : NATURAL := true_log_pow2(rx_block_sz_max);  -- = 4096 words
 
-- 
GitLab