lowRISC / lowRISC/opentitan

[top, padring] Template padring generation or move padring parameterization into padring package

Open
#30,614 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

TOP:earlgrey
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

Description

As pointed out by @glaserf in this comment, we should consider templating the padring or a top-specific padring package file taking care of driving the padring parameters like pad orientation and scan role. Right now, these parameter lists are large and updating them is error prone.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the top-specific padring parameter lists and the padring package boundary, then read the referenced discussion in PR 30545. Determine whether templating the padring or moving parameterization into a padring package is the agreed direction; done means the large parameter lists are no longer error-prone to update.

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
Quiet
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.