lowRISC / lowRISC/opentitan

[racl_ctrl,rtl] Strangely named recoverable alert

Open
#26,581 0 comments 0 reactions 0 assignees View on GitHub
Component:RTL IP:racl_ctrl
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

### Description

The recoverable alert from `racl_ctrl` is called `recov_ctrl_update_err`. I'm not sure what a "recov_ctrl" is or whether this is a recoverable "ctrl" error but both seem a bit odd.

There's no need to add the block name as a prefix, so I'd suggest just "update_err".

Contributor guide

Open the contributing guide

Research direction

Start by locating the recoverable alert declaration in the racl_ctrl block and check where recov_ctrl_update_err is referenced. Rename it to update_err, update all references, and verify that the racl_ctrl checks pass with no old-name references remaining.

Written by the indexing model from the issue text.

Assessment

Domain
embedded-iot
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.