lowRISC / lowRISC/opentitan

[host,tools] Allow unsupported `opentitantools` args through `test_args`

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

Description

During [PR#30966](https://github.com/lowRISC/opentitan/pull/30966), a change was made to support multiple hyperdebug boards (especially FPGAs). Now `test_args` are also passed to the FPGA setup and cleanup phase which are direct calls to `opentitantool` with this flag.

If `test_args` is containing non-recognized arguments for `opentitantool` (may be test harness specific flags), the Setup and Cleanup phase will fail.

__Action__: Find a way to allow or filter unsupported opentitantool options in `test_args`.

The are already some proposals by @pamaury (https://github.com/lowRISC/opentitan/pull/30966#issuecomment-5254397577).

Contributor guide

Open the contributing guide

Research direction

Start with PR#30966 and its discussion, then trace how test_args reach the FPGA setup and cleanup calls to opentitantool. Done means test-harness-specific or otherwise unsupported opentitantool options no longer make setup or cleanup fail, while relevant options continue to work.

Written by the indexing model from the issue text.

Assessment

Domain
tooling
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.