Skip to content

[embassy-usb-dfu] support ed25519 verification #4220

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Gerharddc
Copy link
Contributor

This commit adds the ability to verify that USB DFU updates are correctly signed using ed25519.

This required adding support to embassy-boot for reading from the DFU partition.

The memory map for the stm32wb-dfu also had to be adjusted to accommodate the larger bootloader.

@Gerharddc
Copy link
Contributor Author

Seems the "ci/build" failure is unrelated to this PR but related to some recent changes to embassy-mspm0 in main that have warnings

@i509VCB
Copy link
Member

i509VCB commented May 20, 2025

bender run

@i509VCB
Copy link
Member

i509VCB commented May 20, 2025

Sorry about the issues with embassy-mspm0. If you rebase on main this should all build now.

@Gerharddc
Copy link
Contributor Author

@i509VCB thanks!

@Gerharddc Gerharddc marked this pull request as draft May 23, 2025 09:54
This commit adds the ability to verify that USB DFU updates are correctly signed using ed25519.
This required adding support to embassy-boot for reading from the DFU partition.
@Gerharddc Gerharddc marked this pull request as ready for review May 23, 2025 11:00
@Gerharddc
Copy link
Contributor Author

Gerharddc commented May 23, 2025

Hmmm it seems I just can't get ci/build to pass for this PR. The issue seems unrelated to the code though as it is always all the "rpi-pico teleprobe" tests which are failing and I can't imagine these changes affecting that. Perhaps an issue with the HIL infrastructure?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants