Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[sival] Add option to skip reset on lc state read. #25651

Merged

Conversation

moidx
Copy link
Contributor

@moidx moidx commented Dec 13, 2024

Add opentitantool argument to skip reset on LcStateRead. This is useful when debugging device alerts as the device will report escalation state as a result of an escalation. Since the escalation status is not persisted across resets, we need to be able to connect to LC_TAP to read without performing a reset. This is only possible in lc states that support continuous PINMUX_TAP polling.

Add `opentitantool` argument to skip reset on `LcStateRead`. This is
useful when debugging device alerts  as the device will report
escalation state as a result of an escalation. Since the escalation
status is not persisted across resets, we need to be able to connect to
LC_TAP to read without performing a reset. This is only possible in lc
states that support continuous PINMUX_TAP polling.

Signed-off-by: Miguel Osorio <[email protected]>
@moidx moidx requested a review from timothytrippel December 13, 2024 23:24
@moidx moidx requested a review from a team as a code owner December 13, 2024 23:24
@moidx moidx removed the request for review from a team December 13, 2024 23:24
@timothytrippel timothytrippel merged commit a155cb2 into lowRISC:earlgrey_1.0.0 Dec 14, 2024
31 checks passed
@moidx moidx deleted the ottool-skip-reset-on-lc-read branch December 14, 2024 00:31
@moidx moidx added the CherryPick:master This PR should be cherry-picked to master label Dec 14, 2024
Copy link

Successfully created backport PR for master:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CherryPick:master This PR should be cherry-picked to master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants