Clean up parameter types. Fix for issue #676

bhv/cv32e40x_core_log.sv

@@ -39,8 +39,8 @@
 module cv32e40x_core_log import cv32e40x_pkg::*;
 #(
 // todo: log all parameters
-  parameter int NUM_MHPMCOUNTERS =  1,
-  parameter bit ENABLE           =  1
+  parameter int unsigned NUM_MHPMCOUNTERS =  1,
+  parameter bit          ENABLE           =  1
 )
 (
   input logic        clk_i,

bhv/cv32e40x_rvfi.sv

@@ -23,7 +23,7 @@ module cv32e40x_rvfi
   import cv32e40x_rvfi_pkg::*;
   #(
     parameter bit     CLIC  = 0,
-    parameter int     DEBUG = 1,
+    parameter bit     DEBUG = 1,
     parameter a_ext_e A_EXT = A_NONE
   )
   (

bhv/cv32e40x_wrapper.sv

@@ -56,22 +56,22 @@ module cv32e40x_wrapper
   parameter b_ext_e      B_EXT                        = B_NONE,
   parameter m_ext_e      M_EXT                        = M,
   parameter bit          X_EXT                        = 0,
-  parameter int          X_NUM_RS                     = 2,
-  parameter int          X_ID_WIDTH                   = 4,
-  parameter int          X_MEM_WIDTH                  = 32,
-  parameter int          X_RFR_WIDTH                  = 32,
-  parameter int          X_RFW_WIDTH                  = 32,
+  parameter int unsigned X_NUM_RS                     = 2,
+  parameter int unsigned X_ID_WIDTH                   = 4,
+  parameter int unsigned X_MEM_WIDTH                  = 32,
+  parameter int unsigned X_RFR_WIDTH                  = 32,
+  parameter int unsigned X_RFW_WIDTH                  = 32,
   parameter logic [31:0] X_MISA                       = 32'h00000000,
   parameter logic [1:0]  X_ECS_XS                     = 2'b00,
-  parameter int          NUM_MHPMCOUNTERS             = 1,
+  parameter int unsigned NUM_MHPMCOUNTERS             = 1,
   parameter bit          CLIC                         = 0,
-  parameter int          CLIC_ID_WIDTH                = 5,
-  parameter int          CLIC_INTTHRESHBITS           = 8,
+  parameter int unsigned CLIC_ID_WIDTH                = 5,
+  parameter int unsigned CLIC_INTTHRESHBITS           = 8,
   parameter int          DBG_NUM_TRIGGERS             = 1,
   parameter int          PMA_NUM_REGIONS              = 0,
   parameter pma_cfg_t    PMA_CFG[PMA_NUM_REGIONS-1:0] = '{default:PMA_R_DEFAULT},
   parameter bit          CORE_LOG_ENABLE              = 1,
-  parameter int          DEBUG                        = 1,
+  parameter bit          DEBUG                        = 1,
   parameter logic [31:0] DM_REGION_START              = 32'hF0000000,
   parameter logic [31:0] DM_REGION_END                = 32'hF0003FFF
 )

rtl/cv32e40x_clic_int_controller.sv

@@ -28,7 +28,7 @@
 
 module cv32e40x_clic_int_controller import cv32e40x_pkg::*;
 #(
-  parameter int CLIC_ID_WIDTH = 5
+  parameter int unsigned CLIC_ID_WIDTH = 5
 )
 (
   input  logic                       clk,

rtl/cv32e40x_controller.sv

@@ -35,8 +35,8 @@ module cv32e40x_controller import cv32e40x_pkg::*;
   parameter a_ext_e      A_EXT                  = A_NONE,
   parameter int unsigned REGFILE_NUM_READ_PORTS = 2,
   parameter bit          CLIC                   = 0,
-  parameter int          CLIC_ID_WIDTH          = 5,
-  parameter int          DEBUG                  = 1
+  parameter int unsigned CLIC_ID_WIDTH          = 5,
+  parameter bit          DEBUG                  = 1
 )
 (
   input  logic        clk,                        // Gated clock

rtl/cv32e40x_controller_fsm.sv

@@ -31,10 +31,10 @@
 
 module cv32e40x_controller_fsm import cv32e40x_pkg::*;
 #(
-  parameter bit       X_EXT         = 0,
-  parameter int       DEBUG         = 1,
-  parameter bit       CLIC          = 0,
-  parameter int       CLIC_ID_WIDTH = 5
+  parameter bit          X_EXT         = 0,
+  parameter bit          DEBUG         = 1,
+  parameter bit          CLIC          = 0,
+  parameter int unsigned CLIC_ID_WIDTH = 5
 )
 (
   // Clocks and reset

rtl/cv32e40x_core.sv

@@ -36,24 +36,24 @@ module cv32e40x_core import cv32e40x_pkg::*;
   parameter a_ext_e                     A_EXT                                   = A_NONE,
   parameter b_ext_e                     B_EXT                                   = B_NONE,
   parameter m_ext_e                     M_EXT                                   = M,
-  parameter int                         DEBUG                                   = 1,
+  parameter bit                         DEBUG                                   = 1,
   parameter logic [31:0]                DM_REGION_START                         = 32'hF0000000,
   parameter logic [31:0]                DM_REGION_END                           = 32'hF0003FFF,
   parameter int                         DBG_NUM_TRIGGERS                        = 1,
   parameter int                         PMA_NUM_REGIONS                         = 0,
   parameter pma_cfg_t                   PMA_CFG[PMA_NUM_REGIONS-1:0]            = '{default:PMA_R_DEFAULT},
   parameter bit                         CLIC                                    = 0,
-  parameter int                         CLIC_ID_WIDTH                           = 5,
-  parameter int                         CLIC_INTTHRESHBITS                      = 8,
+  parameter int unsigned                CLIC_ID_WIDTH                           = 5,
+  parameter int unsigned                CLIC_INTTHRESHBITS                      = 8,
   parameter bit                         X_EXT                                   = 0,
-  parameter int                         X_NUM_RS                                = 2,
-  parameter int                         X_ID_WIDTH                              = 4,
-  parameter int                         X_MEM_WIDTH                             = 32,
-  parameter int                         X_RFR_WIDTH                             = 32,
-  parameter int                         X_RFW_WIDTH                             = 32,
+  parameter int unsigned                X_NUM_RS                                = 2,
+  parameter int unsigned                X_ID_WIDTH                              = 4,
+  parameter int unsigned                X_MEM_WIDTH                             = 32,
+  parameter int unsigned                X_RFR_WIDTH                             = 32,
+  parameter int unsigned                X_RFW_WIDTH                             = 32,
   parameter logic [31:0]                X_MISA                                  = 32'h00000000,
   parameter logic [1:0]                 X_ECS_XS                                = 2'b00,
-  parameter int                         NUM_MHPMCOUNTERS                        = 1
+  parameter int unsigned                NUM_MHPMCOUNTERS                        = 1
 )
 (
   // Clock and reset

rtl/cv32e40x_cs_registers.sv

@@ -37,10 +37,10 @@ module cv32e40x_cs_registers import cv32e40x_pkg::*;
   parameter logic [1:0]  X_ECS_XS             =  2'b00, // todo: implement related mstatus bitfields (but only if X_EXT = 1)
   parameter bit          ZC_EXT               = 0,
   parameter bit          CLIC                 = 0,
-  parameter int          CLIC_ID_WIDTH        = 5,
-  parameter int          CLIC_INTTHRESHBITS   = 8,
-  parameter int          NUM_MHPMCOUNTERS     = 1,
-  parameter int          DEBUG                = 1,
+  parameter int unsigned CLIC_ID_WIDTH        = 5,
+  parameter int unsigned CLIC_INTTHRESHBITS   = 8,
+  parameter int unsigned NUM_MHPMCOUNTERS     = 1,
+  parameter bit          DEBUG                = 1,
   parameter int          DBG_NUM_TRIGGERS     = 1, // todo: implement support for DBG_NUM_TRIGGERS != 1
   parameter int unsigned MTVT_ADDR_WIDTH      = 26
 )

rtl/cv32e40x_if_stage.sv

@@ -32,15 +32,15 @@ module cv32e40x_if_stage import cv32e40x_pkg::*;
   parameter a_ext_e      A_EXT           = A_NONE,
   parameter b_ext_e      B_EXT           = B_NONE,
   parameter bit          X_EXT           = 0,
-  parameter int          X_ID_WIDTH      = 4,
+  parameter int unsigned X_ID_WIDTH      = 4,
   parameter int          PMA_NUM_REGIONS = 0,
   parameter pma_cfg_t    PMA_CFG[PMA_NUM_REGIONS-1:0] = '{default:PMA_R_DEFAULT},
   parameter int unsigned MTVT_ADDR_WIDTH = 26,
   parameter bit          CLIC            = 1'b0,
-  parameter int          CLIC_ID_WIDTH   = 5,
+  parameter int unsigned CLIC_ID_WIDTH   = 5,
   parameter bit          ZC_EXT          = 0,
   parameter m_ext_e      M_EXT           = M_NONE,
-  parameter int          DEBUG           = 1,
+  parameter bit          DEBUG           = 1,
   parameter logic [31:0] DM_REGION_START = 32'hF0000000,
   parameter logic [31:0] DM_REGION_END   = 32'hF0003FFF
 )

rtl/cv32e40x_load_store_unit.sv

@@ -29,11 +29,11 @@ module cv32e40x_load_store_unit import cv32e40x_pkg::*;
 #(
   parameter a_ext_e      A_EXT = A_NONE,
   parameter bit          X_EXT = 0,
-  parameter int          X_ID_WIDTH = 4,
+  parameter int unsigned X_ID_WIDTH = 4,
   parameter int          PMA_NUM_REGIONS = 0,
   parameter pma_cfg_t    PMA_CFG[PMA_NUM_REGIONS-1:0] = '{default:PMA_R_DEFAULT},
   parameter int          DBG_NUM_TRIGGERS = 1,
-  parameter int          DEBUG           = 1,
+  parameter bit          DEBUG = 1,
   parameter logic [31:0] DM_REGION_START = 32'hF0000000,
   parameter logic [31:0] DM_REGION_END   = 32'hF0003FFF
 )

rtl/cv32e40x_mpu.sv

@@ -31,7 +31,7 @@ module cv32e40x_mpu import cv32e40x_pkg::*;
       parameter type         BUS_RESP_TYPE                = data_resp_t,
       parameter int          PMA_NUM_REGIONS              = 0,
       parameter pma_cfg_t    PMA_CFG[PMA_NUM_REGIONS-1:0] = '{default:PMA_R_DEFAULT},
-      parameter int          DEBUG                        = 1,
+      parameter bit          DEBUG                        = 1,
       parameter logic [31:0] DM_REGION_START              = 32'hF0000000,
       parameter logic [31:0] DM_REGION_END                = 32'hF0003FFF)
   (

rtl/cv32e40x_wb_stage.sv

@@ -36,7 +36,7 @@
 
 module cv32e40x_wb_stage import cv32e40x_pkg::*;
 #(
-    parameter int DEBUG = 1
+    parameter bit DEBUG = 1
 )
 (
   input  logic          clk,            // Not used in RTL; only used by assertions

rtl/if_xif.sv

@@ -30,11 +30,11 @@
 
 interface if_xif import cv32e40x_pkg::*;
 #(
-  parameter int          X_NUM_RS        =  2,  // Number of register file read ports that can be used by the eXtension interface
-  parameter int          X_ID_WIDTH      =  4,  // Width of ID field.
-  parameter int          X_MEM_WIDTH     =  32, // Memory access width for loads/stores via the eXtension interface
-  parameter int          X_RFR_WIDTH     =  32, // Register file read access width for the eXtension interface
-  parameter int          X_RFW_WIDTH     =  32, // Register file write access width for the eXtension interface
+  parameter int unsigned X_NUM_RS        =  2,  // Number of register file read ports that can be used by the eXtension interface
+  parameter int unsigned X_ID_WIDTH      =  4,  // Width of ID field.
+  parameter int unsigned X_MEM_WIDTH     =  32, // Memory access width for loads/stores via the eXtension interface
+  parameter int unsigned X_RFR_WIDTH     =  32, // Register file read access width for the eXtension interface
+  parameter int unsigned X_RFW_WIDTH     =  32, // Register file write access width for the eXtension interface
   parameter logic [31:0] X_MISA          =  '0, // MISA extensions implemented on the eXtension interface
   parameter logic [ 1:0] X_ECS_XS        =  '0  // Default value for mstatus.XS
 );

sva/cv32e40x_controller_fsm_sva.sv

@@ -29,7 +29,7 @@ module cv32e40x_controller_fsm_sva
   import uvm_pkg::*;
   import cv32e40x_pkg::*;
   #(  parameter bit X_EXT     = 1'b0,
-      parameter int DEBUG     = 0,
+      parameter bit DEBUG     = 1'b0,
       parameter bit CLIC      = 1'b0
   )
 (

sva/cv32e40x_core_sva.sv

@@ -27,7 +27,7 @@ module cv32e40x_core_sva
   import cv32e40x_pkg::*;
   #(
     parameter a_ext_e A_EXT = A_NONE,
-    parameter int     DEBUG = 1,
+    parameter bit     DEBUG = 1,
     parameter int     PMA_NUM_REGIONS = 0,
     parameter bit     CLIC = 0
   )

sva/cv32e40x_cs_registers_sva.sv

@@ -27,7 +27,7 @@ module cv32e40x_cs_registers_sva
   import cv32e40x_pkg::*;
 #(
     parameter bit CLIC  = 0,
-    parameter int DEBUG = 1
+    parameter bit DEBUG = 1
   )
 
   (

sva/cv32e40x_decoder_sva.sv

@@ -28,7 +28,7 @@ module cv32e40x_decoder_sva
   import cv32e40x_pkg::*;
 #(
   parameter a_ext_e A_EXT     = A_NONE,
-  parameter int     CLIC      = 0
+  parameter bit     CLIC      = 0
 )
 (
   input logic           clk,

sva/cv32e40x_if_stage_sva.sv

@@ -24,7 +24,7 @@ module cv32e40x_if_stage_sva
   import uvm_pkg::*;
   import cv32e40x_pkg::*;
 #(
-    parameter int CLIC = 0
+    parameter bit CLIC = 0
 )
 (
   input  logic          clk,

sva/cv32e40x_load_store_unit_sva.sv

@@ -24,7 +24,7 @@ module cv32e40x_load_store_unit_sva
   #(
     parameter bit     X_EXT = 0,
     parameter         DEPTH = 0,
-    parameter int     DEBUG = 1,
+    parameter bit     DEBUG = 1,
     parameter a_ext_e A_EXT = A_NONE
   )
   (input logic       clk,

sva/cv32e40x_mpu_sva.sv

@@ -24,13 +24,13 @@
 ////////////////////////////////////////////////////////////////////////////////
 
 module cv32e40x_mpu_sva import cv32e40x_pkg::*; import uvm_pkg::*;
-  #(  parameter int PMA_NUM_REGIONS              = 0,
+  #(  parameter int          PMA_NUM_REGIONS              = 0,
       parameter pma_cfg_t    PMA_CFG[PMA_NUM_REGIONS-1:0] = '{default:PMA_R_DEFAULT},
       parameter int unsigned IS_INSTR_SIDE = 0,
       parameter type         CORE_RESP_TYPE = inst_resp_t,
       parameter type         CORE_REQ_TYPE  = obi_inst_req_t,
       parameter a_ext_e      A_EXT = A_NONE,
-      parameter int          DEBUG = 1,
+      parameter bit          DEBUG = 1,
       parameter logic [31:0] DM_REGION_START = 32'hF0000000,
       parameter logic [31:0] DM_REGION_END   = 32'hF0003FFF)
   (

sva/cv32e40x_rvfi_sva.sv

@@ -29,7 +29,7 @@ module cv32e40x_rvfi_sva
   import cv32e40x_rvfi_pkg::*;
 #(
     parameter bit     CLIC  = 0,
-    parameter int     DEBUG = 1,
+    parameter bit     DEBUG = 1,
     parameter a_ext_e A_EXT = A_NONE
 )
 (

sva/cv32e40x_wb_stage_sva.sv

@@ -27,7 +27,7 @@ module cv32e40x_wb_stage_sva
   import uvm_pkg::*;
   import cv32e40x_pkg::*;
   #(
-    parameter int DEBUG  = 1
+    parameter bit DEBUG  = 1
   )
 (
   input logic           clk,

sva/cv32e40x_write_buffer_sva.sv

@@ -27,7 +27,7 @@ module cv32e40x_write_buffer_sva
   import cv32e40x_pkg::*;
   import uvm_pkg::*;
   #(
-     parameter int PMA_NUM_REGIONS = 0,
+     parameter int       PMA_NUM_REGIONS = 0,
      parameter pma_cfg_t PMA_CFG[PMA_NUM_REGIONS-1:0] = '{default:PMA_R_DEFAULT}
   )
   (input logic          clk,