From 540632a77a970cb58c13eb5c27fd34522a627781 Mon Sep 17 00:00:00 2001 From: pawelkudlakaldec <58807617+pawelkudlakaldec@users.noreply.github.com> Date: Mon, 18 Jan 2021 10:36:22 +0100 Subject: [PATCH] dromajo_ram: Fix IEEE compliance (#584) According to IEEE 1800-2017 (9.2.2.4 Sequential logic always_ff procedure): "Variables on the left-hand side of assignments within an always_ff procedure, including variables from the contents of a called function, shall not be written to by any other process." Thus there is a proposal of changing always_ff with always because Mem_DP is driven by two processes: initial and always_ff what is forbidden. --- src/dromajo_ram.sv | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/dromajo_ram.sv b/src/dromajo_ram.sv index 2983801d7..55ecb81e3 100644 --- a/src/dromajo_ram.sv +++ b/src/dromajo_ram.sv @@ -71,7 +71,7 @@ module dromajo_ram end end - always_ff @(posedge Clk_CI) begin + always @(posedge Clk_CI) begin if(CSel_SI) begin if(WrEn_SI) begin if(BEn_SI[0]) Mem_DP[Addr_DI][7:0] <= WrData_DI[7:0];