lowRISC / lowRISC/opentitan

Making top-level parameters localparams

Open
#27,883 0 comments 0 reactions 2 assignees Claimed by @andreaskurth View on GitHub
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

### Description

Many of the top-level parameters are derived from HJSON definitions but could be unintentionally overridden.
We proposed that we should local parameters whenever possible, i.e., for parameters like SRAM size that should not be overridable.
This would make the top hjson essentially the single source of truth for these parameters.
However, some parameters should remain top-level parameters (e.g., external signal widths, FPGA-specific values, etc).

To do this, we must first review all parameters and determine their appropriate exposure level.

(Note: This is coming from the Multi-Top WG discussion on July 1st)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.