lowRISC / lowRISC/opentitan

[rram_ctrl,dv] Add targeted checks for CsrExclCheck-excluded fields

Open
#31,290 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

Fields like PHY_STATUS.init_done are excluded from the generic predictor-based check since the RAL can't predict them; on_reg_access() needs hand-rolled invariant checks for these instead.

Contributor guide

Open the contributing guide

Research direction

Start in the rram_ctrl,dv verification code and locate on_reg_access(), then inspect how CsrExclCheck-excluded fields such as PHY_STATUS.init_done bypass the generic predictor-based check. Identify the intended invariants for these fields and add targeted checks there. Done means excluded fields are checked without relying on RAL prediction and the relevant verification checks pass.

Written by the indexing model from the issue text.

Assessment

Domain
embedded-iot, testing-qa
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
66/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.