Skip to content

[nrf noup] cmake: mcuboot: SHA512/pure image signing #3074

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

nordic-mik7
Copy link
Contributor

Update signing script to use proper arguments for imgtool when SHA512 or pure signature is needed.

Upstream PR #: 93573

@nordic-mik7 nordic-mik7 force-pushed the dev/sha512_ironboard branch from 2769fac to bcb8673 Compare July 23, 2025 12:30
@nordic-mik7 nordic-mik7 changed the title [nrf fromlist] cmake: mcuboot: SHA512/pure image signing [nrf noup] cmake: mcuboot: SHA512/pure image signing Jul 23, 2025
@@ -152,6 +152,13 @@ function(zephyr_mcuboot_tasks)
set(imgtool_args --align ${write_block_size} ${imgtool_args})
endif()

# Set proper hash calculation algorithm for signing
if(CONFIG_MCUBOOT_BOOTLOADER_USES_SHA512 AND NOT CONFIG_MCUBOOT_BOOTLOADER_SIGNATURE_TYPE_PURE)
Copy link
Contributor

Choose a reason for hiding this comment

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

If found better, you may change the order of ifs.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@nordic-mik7 nordic-mik7 force-pushed the dev/sha512_ironboard branch from bcb8673 to 8601d86 Compare July 24, 2025 12:11
Update signing script to use proper arguments for imgtool when SHA512
or pure signature is needed.

Signed-off-by: Michal Kozikowski <michal.kozikowski@nordicsemi.no>
@nordic-mik7 nordic-mik7 force-pushed the dev/sha512_ironboard branch from 8601d86 to ca746d8 Compare July 24, 2025 12:25
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.

2 participants