feat(timer): Remove group from timer csrs.

Used as an example for ungrouped regs, as mentioned in IObundle#83
arturum1 · Nov 24, 2024 · 554b140 · 554b140
1 parent a37d92f
commit 554b140
Show file tree

Hide file tree

Showing 2 changed files with 44 additions and 50 deletions.
diff --git a/py2hwsw/lib/hardware/iob_csrs/scripts/reg_gen.py b/py2hwsw/lib/hardware/iob_csrs/scripts/reg_gen.py
@@ -61,7 +61,7 @@ def build_regs_table(core):
     general_regs_table = find_obj_in_list(core["csrs"], "general")
     if not general_regs_table:
         general_regs_table = iob_csr_group(
-            name="general",
+            name="general_operation",
             descr="General Registers.",
             regs=[],
         )

diff --git a/py2hwsw/lib/hardware/iob_timer/iob_timer.py b/py2hwsw/lib/hardware/iob_timer/iob_timer.py
@@ -114,55 +114,49 @@ def setup(py_params_dict):
                 "instance_description": "Control/Status Registers",
                 "csrs": [
                     {
-                        "name": "timer",
-                        "descr": "TIMER software accessible registers.",
-                        "regs": [
-                            {
-                                "name": "reset",
-                                "type": "W",
-                                "n_bits": 1,
-                                "rst_val": 0,
-                                "log2n_items": 0,
-                                "autoreg": True,
-                                "descr": "Timer soft reset",
-                            },
-                            {
-                                "name": "enable",
-                                "type": "W",
-                                "n_bits": 1,
-                                "rst_val": 0,
-                                "log2n_items": 0,
-                                "autoreg": True,
-                                "descr": "Timer enable",
-                            },
-                            {
-                                "name": "sample",
-                                "type": "W",
-                                "n_bits": 1,
-                                "rst_val": 0,
-                                "log2n_items": 0,
-                                "autoreg": True,
-                                "descr": "Sample time counter value into a readable register",
-                            },
-                            {
-                                "name": "data_low",
-                                "type": "R",
-                                "n_bits": 32,
-                                "rst_val": 0,
-                                "log2n_items": 0,
-                                "autoreg": True,
-                                "descr": "High part of the timer value, which has twice the width of the data word width",
-                            },
-                            {
-                                "name": "data_high",
-                                "type": "R",
-                                "n_bits": 32,
-                                "rst_val": 0,
-                                "log2n_items": 0,
-                                "autoreg": True,
-                                "descr": "Low part of the timer value, which has twice the width of the data word width",
-                            },
-                        ],
+                        "name": "reset",
+                        "type": "W",
+                        "n_bits": 1,
+                        "rst_val": 0,
+                        "log2n_items": 0,
+                        "autoreg": True,
+                        "descr": "Timer soft reset",
+                    },
+                    {
+                        "name": "enable",
+                        "type": "W",
+                        "n_bits": 1,
+                        "rst_val": 0,
+                        "log2n_items": 0,
+                        "autoreg": True,
+                        "descr": "Timer enable",
+                    },
+                    {
+                        "name": "sample",
+                        "type": "W",
+                        "n_bits": 1,
+                        "rst_val": 0,
+                        "log2n_items": 0,
+                        "autoreg": True,
+                        "descr": "Sample time counter value into a readable register",
+                    },
+                    {
+                        "name": "data_low",
+                        "type": "R",
+                        "n_bits": 32,
+                        "rst_val": 0,
+                        "log2n_items": 0,
+                        "autoreg": True,
+                        "descr": "High part of the timer value, which has twice the width of the data word width",
+                    },
+                    {
+                        "name": "data_high",
+                        "type": "R",
+                        "n_bits": 32,
+                        "rst_val": 0,
+                        "log2n_items": 0,
+                        "autoreg": True,
+                        "descr": "Low part of the timer value, which has twice the width of the data word width",
                     },
                 ],
                 "csr_if": "iob",