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

Cherry-pick to earlgrey_es_sival: [opentitantool] Allow changing which pins to use for JTAG #23422

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

github-actions[bot]
Copy link

This is an automatic cherry-pick of #20934 to branch earlgrey_es_sival.

HyperDebug by default does JTAG on five particular pins.  However, as it
is all bit-banging with no true hardware support, it is able to use
almost any combination of pins.

This PR introduces a transport-specific command to "move" the JTAG
functionality to a different set of pins.

Usage:
`opentitantool --interface=hyperdebug transport set-jtag-pins <TCLK> <TMS> <TDI> <TDO> <TRSTn>`

Signed-off-by: Jes B. Klinke <[email protected]>

Change-Id: Ibf61be39949b6330f4dd1ec01c44bbdc5cc2350c
(cherry picked from commit 86143ec)
@github-actions github-actions bot requested a review from a team as a code owner May 31, 2024 16:27
@github-actions github-actions bot requested review from pamaury and removed request for a team May 31, 2024 16:27
@benjbender
Copy link
Contributor

@jesultra @timothytrippel please merge to earlgrey_es_sival branch.
Thanks.

@benjbender
Copy link
Contributor

@pamaury

Copy link
Contributor

@jesultra jesultra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me (then again, I wrote the original code)

@jesultra jesultra merged commit 424f807 into earlgrey_es_sival Nov 12, 2024
27 checks passed
@a-will a-will deleted the backport-20934-to-earlgrey_es_sival branch November 12, 2024 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants