On Tue, Mar 26, 2013 at 8:18 PM, Ouabache Designworks
<[email protected]> wrote:
> Stefan,
>
> Here are some lint errors that you might want to clean up
>
>
> ------------------------------------
> mor1kx_ctrl_cappuccino.v:
> mor1kx_ctrl_espresso.v
> should use non-blocking assignments
>
>        always @(posedge clk)
>          begin
>         spr_dmr1 = 0;
>         spr_dmr2 = 0;
>         spr_dsr = 0;
>         spr_drr = 0;
>         du_npc_written = 0;
>          end
>
>

Nice catch.

> ----------------------------------------------
> mor1kx_fetch_cappuccino.v
> should use blocking assignments
>
>    always @(*)
>      if (kill_fetch | pipeline_flush_i)
>        fetch_valid <= 1'b0;
>      else if (bus_access_done | stall_fetch_valid |
>           (except_itlb_miss | except_ipagefault) & !branch_except_occur_i)
>        fetch_valid <= 1'b1;
>      else
>        fetch_valid <= 1'b0;
>

This was already fixed in git HEAD when you wrote this :)

> ------------------------------------------
> `ifndef SYNTHESIS
>
> should always be used anytime there is a translate_off
>
> translate_off should always be used anytime there is a $display
>
>
> ----------------------------------------------------------------

Thanks for the suggestions

Stefan
_______________________________________________
OpenRISC mailing list
[email protected]
http://lists.openrisc.net/listinfo/openrisc

Reply via email to