lowRISC / lowRISC/opentitan

[dv/lc_ctrl] use escalation agent

Open
#18,007 3 comments 0 reactions 0 assignees View on GitHub
Earlgrey-PROD Candidate IP:lc_ctrl Priority:P2
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

### Description

- [ ] In https://cs.opensource.google/opentitan/opentitan/+/master:hw/ip/lc_ctrl/dv/env/seq_lib/lc_ctrl_errors_vseq.sv
`lc_ctrl_errors_vseq.sv` currently manually drive escalation input.
For future release, it can enhance the esc_agent and use the agent to drive.

- [ ] In `tb.sv`: https://cs.opensource.google/opentitan/opentitan/+/master:hw/ip/lc_ctrl/dv/tb.sv;l=108
Add a separate clock and reset for the kmac interface.
This is listed for future release because the current chip-level test already tested in a different clock and reset domain.

Contributor guide

Open the contributing guide

Research direction

Start with hw/ip/lc_ctrl/dv/env/seq_lib/lc_ctrl_errors_vseq.sv to understand how the escalation input is currently driven, then inspect the escalation agent. Review hw/ip/lc_ctrl/dv/tb.sv around line 108 for the KMAC interface clock and reset. Done means the agent drives escalation and the KMAC interface has separate clock and reset signals.

Written by the indexing model from the issue text.

Assessment

Domain
embedded-iot, testing
Issue type
Refactor
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.