-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
cmake: mcuboot: Use imgtool instead of west for signing #78983
base: main
Are you sure you want to change the base?
Conversation
82d79f4
to
d3d3c9e
Compare
cc @butok |
The Zephyr "west sign" documentation should be updated https://docs.zephyrproject.org/latest/develop/west/sign.html |
d3d3c9e
to
3c5a83a
Compare
Updated |
@tejlmand , kindly review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some observations related to the use of devicetree CMake functions which should be addressed or replied if this is handled / guaranteed elsewhere in the build system.
Rest looks fine.
Adds a Kconfig which can be selected when building for firmware updater mode to select if this is the application or the firmware updater image that is being built Signed-off-by: Jamie McCrae <[email protected]>
Uses imgtool directly to sign images instead of calling west sign, this also removes the MCUBOOT_CMAKE_WEST_SIGN_PARAMS Kconfig option as this has no effect Signed-off-by: Jamie McCrae <[email protected]>
Removes the Kconfig extension function around a Kconfig that has been removed Signed-off-by: Jamie McCrae <[email protected]>
Fixes a compliance error Signed-off-by: Jamie McCrae <[email protected]>
Deprecates signing for imgtool using west sign Signed-off-by: Jamie McCrae <[email protected]>
Adds a note that the build system now uses imgtool directly Signed-off-by: Jamie McCrae <[email protected]>
Adds details on the deprecation of this feature Signed-off-by: Jamie McCrae <[email protected]>
Adds a signing page under build, which contains information on how to sign images from the build system, this removes the old details for using west sign with imgtool from the west sign page Signed-off-by: Jamie McCrae <[email protected]>
6e43321
3c5a83a
to
6e43321
Compare
|
||
# If single slot mode, or if in firmware updater mode and this is the firmware updater image, | ||
# use slot 0 information | ||
if(NOT CONFIG_MCUBOOT_BOOTLOADER_MODE_SINGLE_APP AND (NOT CONFIG_MCUBOOT_BOOTLOADER_MODE_FIRMWARE_UPDATER OR CONFIG_MCUBOOT_APPLICATION_FIRMWARE_UPDATER)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor nit, quite a long line.
Fixes #78044