bmartini / zynq-axis Goto Github PK

Hi,

Is this driver tested on the linux kernel 5.x version?

Thanks.

I modified the testbench to demonstrate at line 209 as follows

``ifdef TB_VERBOSE
$display("test read");
`endif

    ready <= 1'b1;
    repeat(5) @(posedge clk);

    axi_rdata   <= {32'd8, 32'd7, 32'd6, 32'd5, 32'd4, 32'd3, 32'd2, 32'd1};
    axi_rvalid  <= 1'b1;
    @(posedge clk);
    axi_rdata   <= {32'd9, 32'd8, 32'd7, 32'd6, 32'd5, 32'd4, 32'd3, 32'd2};
    while ( ~axi_rready) @(posedge clk);
    axi_rvalid  <= 1'b1;
    @(posedge clk);
    axi_rdata   <= 'b0;
    axi_rvalid  <= 1'b0;
    **repeat(5) @(posedge clk);
    ready <= 1'b0;
    repeat(5) @(posedge clk);
    ready <= 1'b1;**

    repeat(15) @(posedge clk);

ifdef TB_VERBOSE $display("END"); endif`

If you look at the marker, valid deasserts, but by the AXIS protocol it needs to be held until ready completes the transfer. In this case the data value 0x3 would be missed by the downstream slave.

I'm speculating, but maybe there's a similar error in axis_write_data (even though
I've never seen that interface to the HP port deassert tready)