chipsalliance / chipsalliance/chisel

Naming of ready/valid interface

Open
#1,350 8 comments 0 reactions 0 assignees View on GitHub
API Modification feature request
Dominant language
Scala
Stars
4.8k
Forks
658
Avg merge
18h 59m
Merged PRs (30d)
14

Description

**Type of issue**: feature request

**Impact**: API addition (no impact on existing code), later API modification

I would like to give the ready/valid interface a better name and also a better nome for the data field.

Interface name: It is not clear to me what DecoupledIO means. Who is decoupling from whom? A more logical name for the ready/valid interface would be ReadyValidIO.

Data field: this field is normally (e.g., in Dally) simply named data. Why bits?

I know renaming might be painful, but it has been done. And if it gives in the long run better reading (code understanding) it might be worth it.

Contributor guide

Open the contributing guide

Research direction

Start by locating the DecoupledIO interface and its bits field, then review their usages across the Chisel API. Check the issue discussion before proceeding; done means the proposed names and any compatibility or migration impact are resolved consistently.

Written by the indexing model from the issue text.

Assessment

Tech stack
scala
Domain
tooling
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.