lowRISC / lowRISC/opentitan

[pwrmgr,dv] Exercise all rom_ctrl inputs

Open
#25,261 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

https://github.com/lowRISC/opentitan/pull/25169 adds the ability to parametrize the number of rom_ctrl inputs. Earlgrey has 1 rom_ctrl but Darjeeling has 2.

While the aforementioned PR templates the number of rom_ctrl inputs in the RTL, the DV part only templates the minimum, i.e., the interface becomes an array. However DV does not yet exercise the other rom_ctrl inputs.

This issue is to track the effort to ipgen also the DV part to properly test rom_ctrls > 1.

/cc @andreaskurth @matutem

Contributor guide

Open the contributing guide

Research direction

Start by reading PR #25169 and the issue’s description of the parametrized rom_ctrl inputs in RTL and DV. Trace the ipgen path for the DV interface, then ensure configurations with more than one rom_ctrl input are exercised; done means Darjeeling’s two inputs are properly tested.

Written by the indexing model from the issue text.

Assessment

Domain
testing
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.