Skip to content

Commit

Permalink
Fixes proposed by Athanasios Moschos (https://github.com/0ena). (1) A…
Browse files Browse the repository at this point in the history
…dd "or1k_defines.v" include into or1k_marocchino_monitor.v (2) Move qm_r declaration before 1st usage in pfpu_marocchino_div.v. Both the issues cause error in Modelsim.
  • Loading branch information
bandvig committed Jul 27, 2020
1 parent 8be054f commit d63dbac
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions bench/verilog/or1k_marocchino_monitor.v
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@
// //
////////////////////////////////////////////////////////////////////////

`include "or1k_defines.v"

/* Configure these defines to point to the marocchino instantiation */
`ifndef OR1K_INST
`define OR1K_INST dut.gencpu.or1k_marocchino0
Expand Down
4 changes: 2 additions & 2 deletions rtl/verilog/pfpu_marocchino/pfpu_marocchino_div.v
Original file line number Diff line number Diff line change
Expand Up @@ -187,6 +187,8 @@ module r4div_fract58
// signed digits to tow's complement on the fly converter
// # part Q
reg [N-1:0] q_r;
// # part QM
reg [N-1:0] qm_r;
// # ---
always @(posedge cpu_clk) begin
if (div_start_i)
Expand All @@ -204,8 +206,6 @@ module r4div_fract58
endcase
end
end // @clock
// # part QM
reg [N-1:0] qm_r;
// # ---
always @(posedge cpu_clk) begin
if (div_start_i)
Expand Down

0 comments on commit d63dbac

Please sign in to comment.