arnavsacheti / PeakRDL-BusDecoder / 28907592898

92%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(cpuif): hoist boilerplate into BaseCpuif and merge APB3/APB4 (#66)

Drives down per-protocol cpuif duplication:

1. Hoist shared cpuif scaffolding into BaseCpuif. The interface instance,
   `is_interface`, `signal()`, and `port_declaration` now live on the base,
   driven by class attributes (`flat_interface_cls`, `sv_interface_cls`,
   `slave_name_flat`, `slave_name_sv`, `master_signal_prefix`,
   `use_sv_interface`). Each SV-variant subclass collapses to a one-line
   `use_sv_interface = True`.

2. Replace the per-protocol SV-array fanin overrides with a declarative
   table (`sv_array_fanin_wr` / `sv_array_fanin_rd`). BaseCpuif now applies
   the override automatically and derives `fanin_intermediate_assignments`
   from the same tables. Subclasses only define `_default_fanin_wr` /
   `_default_fanin_rd`.

3. Collapse APB3 and APB4 into a shared `cpuif/apb/` package: one cpuif
   base, one interface module, one template. Variants differ by
   `apb_interface_type`, `has_pprot`, and `has_pstrb`. The `cpuif/apb3/`
   and `cpuif/apb4/` packages remain as thin re-exports for API stability.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Coverage Stats

182 of 182 new or added lines in 9 files covered. (100.0%)

2 existing lines in 1 file now uncovered.

1480 of 1602 relevant lines covered (92.38%)

0.92 hits per line