Skip to content

Commit

Permalink
A variety of data width fixes. Use more wildcards in sensitivity list…
Browse files Browse the repository at this point in the history
…s to shut up warnings in Vivado.
  • Loading branch information
zwabbit committed Apr 19, 2016
1 parent 45d6e66 commit ba98a6e
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 12 deletions.
2 changes: 1 addition & 1 deletion src/verilog/rtl/decode/reg_field_encoder.v
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ assign sgpr_address = sgpr_base + in[6:0];
assign vgpr_address = vgpr_base + in[7:0];
assign negative_constant = (~{4'b0,in[5:0]}) + 10'b1;

always @(in or sgpr_base or vgpr_base)
always @(*)
begin
casex(in)
//invalid operand
Expand Down
2 changes: 1 addition & 1 deletion src/verilog/rtl/fpga/compute_unit_fpga.v
Original file line number Diff line number Diff line change
Expand Up @@ -636,7 +636,7 @@ always @( posedge S_AXI_ACLK ) begin
singleVectorWrData28 <= S_AXI_WDATA;
singleVectorWrDataMask <= singleVectorWrDataMask | 64'h0000000010000000;
end
9'h511D: begin
9'h11D: begin
singleVectorWrData29 <= S_AXI_WDATA;
singleVectorWrDataMask <= singleVectorWrDataMask | 64'h0000000020000000;
end
Expand Down
9 changes: 1 addition & 8 deletions src/verilog/rtl/issue/mem_wait.v
Original file line number Diff line number Diff line change
Expand Up @@ -24,20 +24,13 @@ decoder_6b_40b_en issue_value_decoder
);


decoder_6b_40b_en retire_sgpr_value_decoder
decoder_6b_40b_en retire_lsu_value_decoder
(
.addr_in(lsu_done_wfid),
.out(decoded_lsu_retire_value),
.en(lsu_done)
);

decoder_6b_40b_en retire_vgpr_value_decoder
(
.addr_in(f_vgpr_lsu_wr_done_wfid),
.out(decoded_vgpr_retire_value),
.en(f_vgpr_lsu_wr_done)
);

dff_set_en_rst mem_wait[`WF_PER_CU-1:0]
(
.q(mem_waiting_wf),
Expand Down
4 changes: 2 additions & 2 deletions src/verilog/rtl/rfa/rfa.v
Original file line number Diff line number Diff line change
Expand Up @@ -54,11 +54,11 @@ module rfa(/*AUTOARG*/

// If lsu requests writes, it bypasses the priority encoder
// but if salu request writes, it bypasses both
assign entry_valid = salu_req ? {'b0, simf3_queue_entry_valid, simf2_queue_entry_valid,
assign entry_valid = salu_req ? {8'd0, simf3_queue_entry_valid, simf2_queue_entry_valid,
simf1_queue_entry_valid, simf0_queue_entry_valid,
simd3_queue_entry_valid, simd2_queue_entry_valid,
simd1_queue_entry_valid, simd0_queue_entry_valid} &
{16{~salu_req}}:{'b0, simf3_queue_entry_valid, simf2_queue_entry_valid,
{16{~salu_req}}:{8'd0, simf3_queue_entry_valid, simf2_queue_entry_valid,
simf1_queue_entry_valid, simf0_queue_entry_valid,
simd3_queue_entry_valid, simd2_queue_entry_valid,
simd1_queue_entry_valid, simd0_queue_entry_valid} &
Expand Down

0 comments on commit ba98a6e

Please sign in to comment.