lowRISC / lowRISC/opentitan

[tlul] Consider refactoring tlul_adapter_reg

Open
#15,822 1 comment 0 reactions 0 assignees View on GitHub
Component:RTL Earlgrey-PROD Triaged IP:tlul Priority:P3 Type:Cleanup Type:FutureRelease
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

If we decide to refactor `tlul_adapter_sram` as part of #7462, we should probably also refactor other related modules such as `tlul_adapter_reg`, and align them.

See also this discussion here: https://github.com/lowRISC/opentitan/pull/15737#pullrequestreview-1160824397

CC @tjaychen

Contributor guide

Open the contributing guide

Research direction

Start by reading issue #7462 and the referenced discussion in pull request #15737, then compare the related tlul_adapter_sram and tlul_adapter_reg modules. The scope depends on the outcome of that refactor discussion; done would mean an agreed and implemented alignment for tlul_adapter_reg.

Written by the indexing model from the issue text.

Assessment

Domain
embedded-iot
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.