Skip to content

Commit

Permalink
bootloader: bl_crypto: Change wrong comments
Browse files Browse the repository at this point in the history
The comments for no SHA256 and no secp256r1 wrongly stated that
these are disabled, this however is not true, the fields are
accessed and must still be present, therefore explain that whilst
they might not be checked, they are still required to be present

Signed-off-by: Jamie McCrae <[email protected]>
  • Loading branch information
nordicjm committed Jan 20, 2025
1 parent 8bdf8af commit 6eb637f
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions subsys/bootloader/bl_crypto/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -73,8 +73,10 @@ config SB_CRYPTO_CLIENT_ECDSA_SECP256R1
Using EXT_APIs from fw_info.

config SB_CRYPTO_NO_ECDSA_SECP256R1
bool "Disable secp256r1 support"
bool "secp256r1 without validation"
select SB_ECDSA_SECP256R1
help
Requires that a secp256r1 signature be present in the image but does no validation.

config SB_CRYPTO_PSA_ED25519
bool "PSA ed25519 support"
Expand Down Expand Up @@ -137,7 +139,9 @@ config SB_CRYPTO_CLIENT_SHA256
Using EXT_APIs from fw_info.

config SB_CRYPTO_NO_SHA256
bool "Disable SHA256 support"
bool "SHA256 without validation"
help
Requires that a SHA256 hash be present in the image but does no validation.

config SB_CRYPTO_NONE
bool "None"
Expand Down

0 comments on commit 6eb637f

Please sign in to comment.