From 62137cabadfd9bc2ec14a5bed604559256cda49a Mon Sep 17 00:00:00 2001 From: Dustin Richmond Date: Mon, 22 Feb 2016 13:32:59 -0800 Subject: [PATCH] Fixed a bug in the TX Multiplexer, write notification logic. The write notification buffer was too small. --- fpga/riffa_hdl/tx_multiplexer.v | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/fpga/riffa_hdl/tx_multiplexer.v b/fpga/riffa_hdl/tx_multiplexer.v index 6ab7920..1d88d12 100644 --- a/fpga/riffa_hdl/tx_multiplexer.v +++ b/fpga/riffa_hdl/tx_multiplexer.v @@ -106,7 +106,7 @@ module tx_multiplexer reg rAckRdEn; // Fifo read enable (TXR_SENT) always @(*) begin - _rAckWrEn = (WR_ACK != 0); + _rAckWrEn = (WR_ACK != 0) | (RD_ACK != 0); end always @(posedge CLK) begin @@ -116,8 +116,8 @@ module tx_multiplexer always @(posedge CLK) begin rAckRdEn <= TXR_SENT; - if(rAckRdEn) begin - rAckRdData <= wAckRdData & {C_NUM_CHNL{wAckValid}}; + if(rAckRdEn & wAckValid) begin + rAckRdData <= wAckRdData;// end else begin rAckRdData <= 0; end @@ -128,7 +128,7 @@ module tx_multiplexer fifo #(// Parameters .C_WIDTH (C_NUM_CHNL), - .C_DEPTH (C_DEPTH_PACKETS*3), // This is an extremely conservative estimate... + .C_DEPTH (C_DEPTH_PACKETS*8), // This is an extremely conservative estimate... .C_DELAY (0) /*AUTOINSTPARAM*/) req_ack_fifo