Merge remote-tracking branch 'origin/copilot/sub-pr-230' into copilot/sub-pr-230

Author: Copilot

tools/README.md

@@ -212,9 +212,9 @@ CSR selector properties:
   - Both accept decimal integer, hex string (`0x...`), or binary string (`0b...`).
 
 Notes on output behavior:
-- JSON output copies the authored `enum` object into the generated `csrs` entry.
+- JSON output preserves the structure of the authored `enum` object in the generated `csrs` entry, but normalizes any multibase numeric values (for example hex `0x...` or binary `0b...`) to plain integers.
 - JSON output stores `ro-mask`/`ro-value` as integers.
-- HTML output preserves the original authored literal text for `ro-mask`/`ro-value` when available (for example `0xF0F0` or `0b1100`).
+- HTML output uses internal underscore-prefixed fields to preserve the original authored literal text for `enum` values and `ro-mask`/`ro-value` when available (for example `0xF0F0` or `0b1100`).
 
 Examples:
 ```yaml

tools/create_params.py

@@ -630,7 +630,6 @@ def parse_multibase_int(value: Any, label: str, csr_name: str) -> int:
 
     representative_name = names[0]
 
-    csr_enum_input: Optional[List[int]] = None
     csr_width_input: Optional[str] = None
     csr_read_only_mask: Optional[int] = None
     csr_read_only_value: Optional[int] = None
@@ -672,12 +671,24 @@ def parse_multibase_int(value: Any, label: str, csr_name: str) -> int:
         if raw_illegal_ignore is not None:
             if not isinstance(raw_illegal_ignore, bool):
                 fatal(f"CSR {representative_name} in {def_filename} has non-boolean illegal-write-ignore: {raw_illegal_ignore!r}")
-            csr_illegal_write_ignore = raw_illegal_ignore
+            if raw_illegal_ignore is not True:
+                fatal(
+                    f"CSR {representative_name} in {def_filename} must set 'illegal-write-ignore' to true "
+                    f"when present (got {raw_illegal_ignore!r})"
+                )
+            csr_illegal_write_ignore = True
 
         if raw_illegal_return is not None:
-            if not isinstance(raw_illegal_return, (int, str)) or isinstance(raw_illegal_return, bool):
-                fatal(f"CSR {representative_name} in {def_filename} has invalid illegal-write-return: {raw_illegal_return!r}; expected integer, hex string, or binary string")
-            csr_illegal_write_return = parse_multibase_int(raw_illegal_return, "illegal-write-return", representative_name)
+            if not isinstance(raw_illegal_return, (str, int)) or isinstance(raw_illegal_return, bool):
+                fatal(
+                    f"CSR {representative_name} in {def_filename} has invalid illegal-write-return "
+                    f"{raw_illegal_return!r}; expected integer, hex string, or binary string"
+                )
+            csr_illegal_write_return = parse_multibase_int(
+                raw_illegal_return,
+                "illegal-write-return",
+                representative_name,
+            )
 
     if has_width:
         raw_width = entry.get("width")