hw: Make external register mapping + naming more explicit (#35)

pulp-platform · Oct 22, 2024 · f395035 · f395035
1 parent 75b865d
commit f395035
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 19 deletions.
diff --git a/hw/chimera_pkg.sv b/hw/chimera_pkg.sv
@@ -35,29 +35,32 @@ package chimera_pkg;
   localparam int ExtCores = _sumVector(ChimeraClusterCfg.NrCores, ExtClusters);
 
   // SoC Config
-  localparam int SnitchBootROM = 1;
+  localparam bit SnitchBootROM = 1;
+  localparam bit TopLevelCfgRegs = 1;
+  localparam bit FLLCfgRegs = 1;
+  localparam bit PadCfgRegs = 1;
 
   // SCHEREMO: Shared Snitch bootrom, one clock gate per cluster, Fll cfg regs, Pad cfg regs
-  localparam int ExtRegNum = SnitchBootROM + 1 + 1 + 1;
+  localparam int ExtRegNum = SnitchBootROM + TopLevelCfgRegs + FLLCfgRegs + PadCfgRegs;
   localparam int ClusterDataWidth = 64;
 
   localparam int SnitchBootROMIdx = 0;
   localparam doub_bt SnitchBootROMRegionStart = 64'h3000_0000;
   localparam doub_bt SnitchBootROMRegionEnd = 64'h3000_1000;
 
-  localparam int TopLevelIdx = 1;
-  localparam doub_bt TopLevelRegionStart = 64'h3000_1000;
-  localparam doub_bt TopLevelRegionEnd = 64'h3000_2000;
+  localparam int TopLevelCfgRegsIdx = 1;
+  localparam doub_bt TopLevelCfgRegsRegionStart = 64'h3000_1000;
+  localparam doub_bt TopLevelCfgRegsRegionEnd = 64'h3000_2000;
 
   // PADs external configuration registers
-  localparam int PadIdx = 2;
-  localparam doub_bt PadRegionStart = 64'h3000_2000;
-  localparam doub_bt PadRegionEnd = 64'h3000_3000;
+  localparam int PadCfgRegsIdx = 2;
+  localparam doub_bt PadCfgRegsRegionStart = 64'h3000_2000;
+  localparam doub_bt PadCfgRegsRegionEnd = 64'h3000_3000;
 
   // FLL external configuration registers
-  localparam int FllIdx = 3;
-  localparam doub_bt FllRegionStart = 64'h3000_3000;
-  localparam doub_bt FllRegionEnd = 64'h3000_4000;
+  localparam int FllCfgRegsIdx = 3;
+  localparam doub_bt FllCfgRegsRegionStart = 64'h3000_3000;
+  localparam doub_bt FllCfgRegsRegionEnd = 64'h3000_4000;
 
 
   localparam aw_bt ClusterNarrowAxiMstIdWidth = 1;
@@ -107,9 +110,14 @@ package chimera_pkg;
     cfg.RegExtNumRules = ExtRegNum;
     cfg.RegExtRegionIdx = {8'h3, 8'h2, 8'h1, 8'h0};  // SnitchBootROM
     cfg.RegExtRegionStart = {
-      FllRegionStart, PadRegionStart, TopLevelRegionStart, SnitchBootROMRegionStart
+      FllCfgRegsRegionStart,
+      PadCfgRegsRegionStart,
+      TopLevelCfgRegsRegionStart,
+      SnitchBootROMRegionStart
+    };
+    cfg.RegExtRegionEnd = {
+      FllCfgRegsRegionEnd, PadCfgRegsRegionEnd, TopLevelCfgRegsRegionEnd, SnitchBootROMRegionEnd
     };
-    cfg.RegExtRegionEnd = {FllRegionEnd, PadRegionEnd, TopLevelRegionEnd, SnitchBootROMRegionEnd};
 
     // ACCEL HART/IRQ CFG
     cfg.NumExtIrqHarts = ExtCores;

diff --git a/hw/chimera_top_wrapper.sv b/hw/chimera_top_wrapper.sv
@@ -211,8 +211,8 @@ module chimera_top_wrapper
   ) i_fll_reg_to_apb (
     .clk_i    (soc_clk_i),
     .rst_ni   (rst_ni),
-    .reg_req_i(reg_slv_req[FllIdx]),
-    .reg_rsp_o(reg_slv_rsp[FllIdx]),
+    .reg_req_i(reg_slv_req[FllCfgRegsIdx]),
+    .reg_rsp_o(reg_slv_rsp[FllCfgRegsIdx]),
     .apb_req_o(apb_fll_req_o),
     .apb_rsp_i(apb_fll_rsp_i)
   );
@@ -226,8 +226,8 @@ module chimera_top_wrapper
   ) i_pad_reg_to_apb (
     .clk_i    (soc_clk_i),
     .rst_ni   (rst_ni),
-    .reg_req_i(reg_slv_req[PadIdx]),
-    .reg_rsp_o(reg_slv_rsp[PadIdx]),
+    .reg_req_i(reg_slv_req[PadCfgRegsIdx]),
+    .reg_rsp_o(reg_slv_rsp[PadCfgRegsIdx]),
     .apb_req_o(apb_req_o),
     .apb_rsp_i(apb_rsp_i)
   );
@@ -241,8 +241,8 @@ module chimera_top_wrapper
   ) i_reg_top (
     .clk_i    (soc_clk_i),
     .rst_ni,
-    .reg_req_i(reg_slv_req[TopLevelIdx]),
-    .reg_rsp_o(reg_slv_rsp[TopLevelIdx]),
+    .reg_req_i(reg_slv_req[TopLevelCfgRegsIdx]),
+    .reg_rsp_o(reg_slv_rsp[TopLevelCfgRegsIdx]),
     .reg2hw   (reg2hw),
     .devmode_i('1)
   );