lowRISC / lowRISC/opentitan

[opentitanlib] Single source of truth for UART baudrates

Open
#20,794 0 comments 0 reactions 0 assignees View on GitHub
SW:opentitantool Type:Cleanup
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

### Description

See https://github.com/lowRISC/opentitan/pull/20706#pullrequestreview-1794578270

We set the initial baud rate for UARTs in OpenTitanLib using constants in the source. These bauds are also specified in the JSON configurations for each target.

There should only really be a single source of truth for these, likely the JSON config.

Contributor guide

Open the contributing guide

Research direction

Compare the UART baudrate constants in OpenTitanLib with the baudrates in each target's JSON configuration. Start by locating both definitions and tracing how the initial UART rate is selected. Done means the baudrate has one authoritative source, the other copy is removed, and all targets still receive the intended initial rate.

Written by the indexing model from the issue text.

Assessment

Tech stack
json
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.