lowRISC / lowRISC/opentitan

[dv] The TCK frequence in jtag_if is not configured explicitly for top-level tests

Open
#24,603 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

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

Description

### Description

At the moment, the frequency comes from `jtag_if.sv` itself. Making this frequency more sensible for GLS meant we needed to add an ifdef (issue #24580). To make this a bit clearer, we should probably:
1. Add something to the top-level test UVM configuration to explicitly set a JTAG frequency.
2. Make the frequency silly if it hasn't been configured, but overridden by something sensible for GLS.
3. "For extra credit": Come up with some way to configure this sensibly for GLS to avoid needing to encode facts about the technology target for Earlgrey into `jtag_if.sv`(!)

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 reading jtag_if.sv and the top-level test UVM configuration mentioned in the issue, then review issue #24580 for the GLS context. Determine how the TCK frequency is currently selected and what configuration mechanisms are available. Done means top-level tests set it explicitly, an unconfigured value is clearly unsuitable, and GLS can override it without hard-coding Earlgrey technology facts.

Written by the indexing model from the issue text.

Assessment

Domain
embedded-iot, testing-qa
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.