HLS dynamic MVAU tests passing.

- Changed the datatype for the HLS test case, from [UINT8, INT8] to [INT8, INT8]. Seems the core was not handling mixed datatypes.
- Update to the RTL dynamic_load core (bug fix).

Author: dkorolij

finn-rtllib/dynload/hdl/dynamic_load.sv

@@ -237,14 +237,14 @@ always_comb begin : NSL_PROC_RD
 
     case (state_rd_C)
         ST_RD_0:
-            if(ordy && (state_wr_C == ST_WR_0) ? (curr_sf_C > cons_sfnf_C) : 1'b1) begin
+            if(ordy && ((state_wr_C == ST_WR_0) ? (curr_sf_C > cons_sfnf_C) : 1'b1)) begin
                 if((cons_sfnf_C == N_TLS-1) && (cons_r_C == N_REPS-1)) begin
                     state_rd_N = ST_RD_1;
                 end
             end
 
         ST_RD_1:
-            if(ordy && (state_wr_C == ST_WR_1) ? (curr_sf_C > cons_sfnf_C) : 1'b1) begin
+            if(ordy && ((state_wr_C == ST_WR_1) ? (curr_sf_C > cons_sfnf_C) : 1'b1)) begin
                 if((cons_sfnf_C == N_TLS-1) && (cons_r_C == N_REPS-1)) begin
                     state_rd_N = ST_RD_0;
                 end

tests/fpgadataflow/test_fpgadataflow_mvau.py

@@ -847,7 +847,7 @@ def test_fpgadataflow_rtl_mvau(
 @pytest.mark.parametrize("n_vectors", [128])
 @pytest.mark.parametrize("pe", [32])
 @pytest.mark.parametrize("simd", [16])
-@pytest.mark.parametrize("idt_wdt", [[DataType["UINT8"], DataType["INT8"]]])
+@pytest.mark.parametrize("idt_wdt", [[DataType["INT8"], DataType["INT8"]]])
 @pytest.mark.parametrize("part", ["xcvc1902-vsva2197-2MP-e-S"])
 @pytest.mark.parametrize("clk_ns", [4])
 @pytest.mark.parametrize("pumpedCompute", [False])