lowRISC / lowRISC/opentitan

[top_darjeeling] Don't generate clk_io_root and clk_io_div2_root in clkmgr

Open
#25,987 0 comments 0 reactions 0 assignees View on GitHub
Component:Darjeeling IP:clkmgr
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

### desc

These two clocks are unused and should not be generated in the first place. Right now, we need to waive lint warnings associated with that (see #25982). Since ipgen is already used to generate clkmgr, it should be doable but there is some work in the templates / hjson files.

Contributor guide

Open the contributing guide

Research direction

Start by tracing how ipgen generates clkmgr from its templates and HJSON files, focusing on clk_io_root and clk_io_div2_root. Confirm where these clocks are described and how the generated output is checked. Done means the unused clocks are no longer generated and the related lint waivers are no longer needed.

Written by the indexing model from the issue text.

Assessment

Domain
build-system, embedded-iot
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.