[opentitantool] Convenience for external programs #25358
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jesultra
added
SW:opentitantool
CherryPick:earlgrey_es_sival
Some transports/debuggers expose the SPI ports in a way that can be
accessed by the flashrom program, without going through opentitanlib.
This CL introduces a new `flashrom-args` command under `spi`:
```
opentitantool --conf ... --usb-serial 2042355F3236 spi --bus=SPI1 flashrom-args
{
programmer: "raiden_debug_spi:serial=2042355F3236,target=2"
}
```
This can be used in cases where the flashrom program better knows how to
communicate with e.g. the SPI flash chip mounted on the OpenTitan
development shield, in order to quickly load some data into it as part
of setup before testing code on the OpenTitan chip.
Signed-off-by: Jes B. Klinke <[email protected]>
Change-Id: I9511b449f303a5f0cd901dac19ce9654209b3fd1
Some transports/debuggers expose the serial ports in a way that can be
accessed by other programs, without necessarily going through
opentitanlib.
This CL introduces a new `uart` top level command and `os-device` under
it, (in hindsigt, `console` and `rescue` probably beloned under the same
top-level, in order to share some flag processing.)
The idea is that `opentitantool` can be queried, using a set of
configuration file with aliases and command line usb serial number, and
will print the path of the underlying OS device corresponding to the
given port of the given debugger:
```
opentitantool --conf ... --usb-serial ... uart --uart=EC os-device
{
path: "/dev/ttyUSB6"
}
```
This can be used in cases where processing of output from the OpenTitan
chip takes place in external scripts or programs. Such cases currently
will have to use an `opentitantool console` process to pass data between
the serial device and its stdin/stdout pipes to the actual script doing
the actual processing, which is wasteful.
Change-Id: Id4dfff38fbc74ec65366d3a7e49a6844ae52c8ee
Signed-off-by: Jes B. Klinke <[email protected]>
jesultra
force-pushed
the
uart_and_flashrom
branch
from
47d686b
to
1d12737
Compare
pamaury
approved these changes
Nov 25, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add two new commands to
opentitantool:spi flashrom-arganduart os-device, which can be used for heling external programs access HyperDebug (or some other backend devices) without going throughopentitanlib, while still using theusb-serialand port aliases from our configuration files to identify the debugger and port.