Google Git
Sign in
opensecura / 3p / lowrisc / opentitan / af9be8c510a17a9ad4248d345307c72c94368873
commitaf9be8c510a17a9ad4248d345307c72c94368873[log] [tgz]
authorJes B. Klinke <jbk@chromium.org>Sun Feb 20 17:32:00 2022 -0800
committerTimothy Trippel <5633066+timothytrippel@users.noreply.github.com>Wed Feb 23 10:21:04 2022 -0800
treef33979810b85bd13a8bcf37bb540a840a2a09a26
parent78972f7a4fc2ad28adf5445a6e2e965bf57c433e [diff]
[opentitantool] Gracefully handle requests for unsupported interfaces

As we plan to introduce a session proxy process, it is not desirable
that the persistent session process could panic!() if a single command
line invocation asks for e.g. I2C, though the particular USB backend
does not support I2C at all.

We already have a TransportError value to be used if asking for e.g. a
UART instance which does not exist: InvalidInstance.  It would be sent
via the TCP protocol and presented by the command line invocation.

This change introduces a new InvalidInterface.  And modifies the
default implementation in the Transport trait to return that instead
of panicking, in case the transport does not support UART, I2C or
whatever kind of interface it is being asked about.

Signed-off-by: Jes B. Klinke <jbk@chromium.org>
Change-Id: I7c0ff6e66a87ccd444be603be0075663d99cc280
2 files changed
tree: f33979810b85bd13a8bcf37bb540a840a2a09a26
  1. .github/
  2. ci/
  3. doc/
  4. hw/
  5. rules/
  6. site/
  7. sw/
  8. test/
  9. third_party/
  10. util/
  11. .bazelignore
  12. .bazelrc
  13. .bazelversion
  14. .clang-format
  15. .dockerignore
  16. .flake8
  17. .gitignore
  18. .style.yapf
  19. .svlint.toml
  20. .svls.toml
  21. _index.md
  22. apt-requirements.txt
  23. azure-pipelines.yml
  24. BUILD
  25. check_tool_requirements.core
  26. CLA
  27. COMMITTERS
  28. CONTRIBUTING.md
  29. LICENSE
  30. meson-config.txt
  31. meson.build
  32. meson_init.sh
  33. meson_options.txt
  34. python-requirements.txt
  35. README.md
  36. tool_requirements.py
  37. toolchain.txt
  38. topgen-reg-only.core
  39. topgen.core
  40. WORKSPACE
  41. yum-requirements.txt
README.md

OpenTitan

OpenTitan logo

About the project

OpenTitan is an open source silicon Root of Trust (RoT) project. OpenTitan will make the silicon RoT design and implementation more transparent, trustworthy, and secure for enterprises, platform providers, and chip manufacturers. OpenTitan is administered by lowRISC CIC as a collaborative project to produce high quality, open IP for instantiation as a full-featured product. See the OpenTitan site and OpenTitan docs for more information about the project.

About this repository

This repository contains hardware, software and utilities written as part of the OpenTitan project. It is structured as monolithic repository, or “monorepo”, where all components live in one repository. It exists to enable collaboration across partners participating in the OpenTitan project.

Documentation

The project contains comprehensive documentation of all IPs and tools. You can access it online at docs.opentitan.org.

How to contribute

Have a look at CONTRIBUTING and our documentation on project organization and processes for guidelines on how to contribute code to this repository.

Licensing

Unless otherwise noted, everything in this repository is covered by the Apache License, Version 2.0 (see LICENSE for full text).