lowRISC / lowRISC/opentitan

[conn,tcl] The "reset" line in the connectivity check is backwards

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

Description

### Description

In `conn.tcl`, the line
```tcl
reset -expr {POR_N}
```
tells Jasper that the chip should be considered in reset (and thus the connectivity properties don't need to hold) if `POR_N` is true.

This will need sorting out properly when we start getting the connectivity checks working again.

Contributor guide

Open the contributing guide

Research direction

Start with conn.tcl and inspect how Jasper interprets the reset -expr {POR_N} line in the connectivity checks. Confirm the intended reset polarity and determine the proper correction once the connectivity checks are working again; done means the reset condition no longer disables checks when POR_N is true.

Written by the indexing model from the issue text.

Assessment

Domain
testing-qa
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.