lowRISC / lowRISC/opentitan

[chip dv] Deprecate chip_common_pkg::chip_io_e and chip_common_pkg::chip_peripheral_e

Open
#14,935 0 comments 0 reactions 0 assignees View on GitHub
Component:DV Earlgrey-PROD Candidate Milestone:V3 Priority:P3 TOP:earlgrey Type:Cleanup
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

These enums are hand-written, which means it could go out of date if the design changes.

We ought to instead use `top_earlgrey_pkg::m|dio_pad_e` or autogenerate additional enums as needed.

Contributor guide

Open the contributing guide

Research direction

Locate the declarations and usages of chip_common_pkg::chip_io_e and chip_common_pkg::chip_peripheral_e, then inspect the available top_earlgrey_pkg enum or enum-generation path. Done means the hand-written enums are no longer the source of truth and the required replacements remain synchronized with design changes.

Written by the indexing model from the issue text.

Assessment

Domain
embedded-iot
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.