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

Provisioning sketch #453

Draft
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

chrysn
Copy link
Collaborator

@chrysn chrysn commented Oct 4, 2024

Description

Building on #444, this is a first demo of how the device ID could be used to do something with it outside the system.

Aligned with the latest plans for #446, this collects some data at startup time (once storage is available: after having generated a private/public key pair), and reports them. For applications with defmt, the only way I found to pick up data from the output was to use cargo embed, which can tee the reassembled defmt texts to a TCP socket. (I generally think that cargo-embed is something we should look into: It also allows neat things such a having stdout / defmt (even simultaneously for those so inclined) while also having GDB through probe-rs).

Issues/PRs references

Open Questions

Change checklist

  • This is a proper PR and not a sketch.

chrysn and others added 19 commits September 27, 2024 11:43
… accessor

This makes the defmt dependency of riot-rs-embassy-common unconditional
to allow enfocing it in a trait.

The "convenience" wrapper riot_rs::identity::device_identity is added
because a simple single method call would otherwise not only require
users to dig into ::arch, but also to import a trait form -common (which
they may not even depend on).
The riot_rs module can not be in scope here, so intradoc links are not
possible.
Previously, the type was just a never-instanciated-even-though-you-could
type to put things into a trait. Now it *is* the return type, opening up
for further evolution through provided methods.
@chrysn
Copy link
Collaborator Author

chrysn commented Oct 16, 2024

The latest commit uses the --config-file option to probe-rs -- that has been merged, but will require installation from git.

Pausing activity here until device-id (#444) is done; maybe by that time, a cargo install probe-rs will already be the easy way to use it.

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.

1 participant