modifed the internal buffer of rx streamer to output valid data only when valid_o is HIGH

Before, the output data would be maintained after to valid_o goes LOW. this causes two problems - main problem: if there is only single output word, rx_first_p1_o and rx_last_p1_o are maintained HIGH, they are generally expected to be pulses - side problem: the data is maintained - it is OK, but it is supposed to be valid anyway only when valid_o is HIGH So, now the data_o is set to zeros when valid is not HIGH. this should prevent anyone from relying on the fact that data is maintaned after valid_o goes LOW and it makes rx_first_p1_o and rx_last_p1_o to be pulses

modifed the internal buffer of rx streamer to output valid data only when valid_o is HIGH
Before, the output data would be maintained after to valid_o goes LOW. this causes two problems - main problem: if there is only single output word, rx_first_p1_o and rx_last_p1_o are maintained HIGH, they are generally expected to be pulses - side problem: the data is maintained - it is OK, but it is supposed to be valid anyway only when valid_o is HIGH So, now the data_o is set to zeros when valid is not HIGH. this should prevent anyone from relying on the fact that data is maintaned after valid_o goes LOW and it makes rx_first_p1_o and rx_last_p1_o to be pulses
2e23cac3 · Maciej Lipinski · c6d1206b · 2e23cac3
Commit 2e23cac3 authored Oct 26, 2016 by Maciej Lipinski
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 0 deletions

dropping_buffer.vhd modules/wr_streamers/dropping_buffer.vhd +1 -0

No files found.
--- a/modules/wr_streamers/dropping_buffer.vhd
+++ b/modules/wr_streamers/dropping_buffer.vhd
@@ -89,6 +89,7 @@ begin  -- behavioral
        d_o       <= mem(to_integer(rd_ptr));
        d_valid_o <= '1';
      else
+        d_o       <= (others => '0');
        d_valid_o <= '0';
      end if;
    end if;