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

[Opentitantool] Add a ftdi transport to the opentitanlib. #25645

Merged
merged 4 commits into from
Jan 7, 2025

Conversation

engdoreis
Copy link
Contributor

@engdoreis engdoreis commented Dec 13, 2024

This PR adds support for the Voyager board by adding a ftdi transport to opentitantool.

The ftdi transport may also be useful for the CW340.

Note: Voyager is a simple board lowRISC and NewAE designed to demonstrate the OpenTitan Earl Grey chip at conferences and other events.

@a-will
Copy link
Contributor

a-will commented Dec 13, 2024

Curious... What is the Voyager board? How is it associated with this project?

@engdoreis
Copy link
Contributor Author

Curious... What is the Voyager board? How is it associated with this project?

Sorry for letting that unexplained. I updated the description.

@nasahlpa
Copy link
Member

nasahlpa commented Jan 2, 2025

just wondering - what is the status of this PR? Please let me know if I can support you @engdoreis, I soon would need to have an opentitantool that works with Voyager.

@jwnrt
Copy link
Contributor

jwnrt commented Jan 2, 2025

FYI @engdoreis, I've changed how Rust dependencies are handled so you will need to rebase and drop the changes to third_party/rust/crates/.... You now only need to change the Cargo.toml and regenerate Cargo.lock (e.g. with cargo update -w --locked).

The new dependencies will be used to implement a new transport that will
use a ftdi chip to bootstrap a opentitan chip.

Signed-off-by: Douglas Reis <[email protected]>
@engdoreis engdoreis force-pushed the opentitantool_voyager branch from 9c5250e to d10ebc3 Compare January 3, 2025 09:47
@engdoreis engdoreis marked this pull request as ready for review January 3, 2025 09:50
@engdoreis engdoreis requested a review from a team as a code owner January 3, 2025 09:50
@engdoreis
Copy link
Contributor Author

@nasahlpa is ready for review now.

thanks @jwnrt for letting me know.

@engdoreis engdoreis changed the title [DRAFT] Opentitantool voyager [Opentitantool] Add a ftdi transport to the opentitanlib. Jan 3, 2025
@engdoreis engdoreis force-pushed the opentitantool_voyager branch from d10ebc3 to 16884ec Compare January 6, 2025 12:26
@engdoreis engdoreis force-pushed the opentitantool_voyager branch from 16884ec to 46f5fc6 Compare January 6, 2025 18:13
Copy link
Contributor

@jwnrt jwnrt left a comment

Choose a reason for hiding this comment

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

This is looking good, thanks.

@engdoreis engdoreis merged commit 15fc5b0 into lowRISC:master Jan 7, 2025
38 checks passed
@engdoreis engdoreis deleted the opentitantool_voyager branch January 7, 2025 10:01
@engdoreis engdoreis added the CherryPick:earlgrey_1.0.0 This PR should be cherry-picked to earlgrey_1.0.0 label Jan 7, 2025
Copy link

github-actions bot commented Jan 7, 2025

Successfully created backport PR for earlgrey_1.0.0:

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

Successfully merging this pull request may close these issues.

4 participants