Skip to content

Conversation

@asmellby
Copy link
Contributor

@asmellby asmellby commented Nov 7, 2025

Add mcuboot fork to manifest.

Sysbuild defaults to RSA signing. Use ECDSA on Series 2 to be compatible with root of trust.

While MCUboot uses imgtool for signing applications, the bootloader binary itself must be signed by Simplicity Commander to be securely booted by the SE on Series 2 devices. Integrate a custom signing script to sign the bootloader image.

This ensures that west build/west flash work out of the box on devices that enforce secure boot.

Add mcuboot fork to manifest.

Signed-off-by: Aksel Skauge Mellbye <[email protected]>
Sysbuild defaults to RSA signing. Use ECDSA on Series 2 to be
compatible with root of trust.

Signed-off-by: Aksel Skauge Mellbye <[email protected]>
While MCUboot uses imgtool for signing applications, the bootloader
binary itself must be signed by Simplicity Commander to be securely
booted by the SE on Series 2 devices. Integrate a custom signing
script to sign the bootloader image.

This ensures that `west build`/`west flash` work out of the box on
devices that enforce secure boot.

The script assumes that `commander` is available on the path.

Signed-off-by: Aksel Skauge Mellbye <[email protected]>
Build `with_mcuboot` sample app for representative devices.

Signed-off-by: Aksel Skauge Mellbye <[email protected]>
@asmellby asmellby requested a review from a team as a code owner November 7, 2025 10:20
jhedberg
jhedberg previously approved these changes Nov 7, 2025
Use find_program() to locate Simplicity Commander, either from
SLT or a system install location.

Signed-off-by: Aksel Skauge Mellbye <[email protected]>
Add Simplicity Commander to SLT package file.

Signed-off-by: Aksel Skauge Mellbye <[email protected]>
- name: Install packages
shell: bash
run: |
slt install -f ${{ inputs.package-file }} --no-lock
Copy link
Contributor

Choose a reason for hiding this comment

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

What is the benefit of downloading slt rather than directly downloading commander?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

slt enables version management of the installed tools, and has other tools available. If the user uses the Network Analyzer or Energy Profiler tools installed via slt, it would be inconvenient if the SDK uses a different way of getting Commander.

@jerome-pouiller
Copy link
Contributor

scripts/west_commands/sign.py already uses commander. It does not relies on slt to find the commander binary:

  • Do we plan to also support west sign for series-2?
  • Should we update west sign?
  • Should we support slt and direct commander run (two ways for doing the same thing)?

@asmellby
Copy link
Contributor Author

asmellby commented Nov 7, 2025

I'm not sure if west sign makes sense. The CMake build system was changed from calling west sign for imgtool to directly calling imgtool from CMake. The philosophical argument was that west is not a mandatory tool for Zephyr, so CMake shouldn't call it. So we're left with the standalone CLI command only, and it's questionable whether there is any value in simply wrapping commander convert --secureboot.

west flash on the other hand can certainly be updated to be able to discover Commander through SLT. But that's something that needs to happen in the main tree, not here.

When it comes to supporting Commander both standalone and discovered through SLT, IMO the answer is "yes". The SLT variant is the most integrated, but being able to directly point at a specific binary to use makes sense to me as an option for situations where SLT isn't available or there is a strong desire to not use it.

@asmellby asmellby merged commit 10f618d into SiliconLabsSoftware:main Nov 10, 2025
3 checks passed
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.

3 participants