Skip to content

boot: zephyr: Kconfig: Add Zephyr KConfig to build PSA APIs for RSA sigverify#2594

Open
gramshan wants to merge 1 commit intomcu-tools:mainfrom
gramshan:PSA-support-for-RSA
Open

boot: zephyr: Kconfig: Add Zephyr KConfig to build PSA APIs for RSA sigverify#2594
gramshan wants to merge 1 commit intomcu-tools:mainfrom
gramshan:PSA-support-for-RSA

Conversation

@gramshan
Copy link

@gramshan gramshan commented Jan 8, 2026

Adding a Zephyr Kconfig option that exposes mcuboot's PSA API-based RSA signature verification as a configurable option.
Previously, PSA Crypto APIs for RSA signature verification could not be enabled from Zephyr configuration, which prevented building mcuboot with PSA based RSA verification out-of-the-box.


config BOOT_RSA_PSA
bool "RSA signatures with mbedTLS PSA Crypto"
depends on !ADI_TESA
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
depends on !ADI_TESA

no random Kconfigs that do not exist

Copy link
Author

Choose a reason for hiding this comment

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

Removed, apologies for the oversight

@gramshan gramshan force-pushed the PSA-support-for-RSA branch 2 times, most recently from bb95968 to a710b9b Compare January 9, 2026 14:49
…igverify

Adding a Zephyr Kconfig option that exposes mcuboot's PSA API-based
RSA signature verification as a configurable option.
Previously, PSA Crypto APIs for RSA signature verification
 could not be enabled from Zephyr configuration, which prevented
building mcuboot with PSA based RSA verification out-of-the-box.

Signed-off-by: gramshan <ramshankargowri@gmail.com>
Signed-off-by: Gowri-Ramshankar_adi <Gowri.Ramshankar@analog.com>
@gramshan gramshan force-pushed the PSA-support-for-RSA branch from a710b9b to 3010abe Compare January 9, 2026 14:51
d3zd3z
d3zd3z previously requested changes Jan 12, 2026
Copy link
Member

@d3zd3z d3zd3z left a comment

Choose a reason for hiding this comment

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

No actual issue, I'm trying to figure out why people are having difficulties addressing earlier negative reviews.

Copy link
Member

@d3zd3z d3zd3z left a comment

Choose a reason for hiding this comment

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

This should remove my -1.

Copy link
Member

@d3zd3z d3zd3z left a comment

Choose a reason for hiding this comment

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

Trying to see if this removes my -1.

@d3zd3z
Copy link
Member

d3zd3z commented Jan 12, 2026

I'm going to try closing and re-opening this, to see if it fixes the broken review state.

@d3zd3z d3zd3z closed this Jan 12, 2026
@d3zd3z d3zd3z reopened this Jan 12, 2026
Copy link
Member

@d3zd3z d3zd3z left a comment

Choose a reason for hiding this comment

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

Does this remove the -1?

Copy link
Member

@d3zd3z d3zd3z left a comment

Choose a reason for hiding this comment

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

Trying again to remove my -1.

Copy link
Member

@d3zd3z d3zd3z left a comment

Choose a reason for hiding this comment

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

Trying yet again

@d3zd3z d3zd3z dismissed stale reviews from nordicjm and themself January 12, 2026 20:40

Test review

d3zd3z
d3zd3z previously requested changes Jan 12, 2026
Copy link
Member

@d3zd3z d3zd3z left a comment

Choose a reason for hiding this comment

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

Test negative review

@d3zd3z d3zd3z dismissed their stale review January 12, 2026 20:41

Removing test.

select MBEDTLS_RSA_C if MBEDTLS_BUILTIN
select MBEDTLS_PKCS1_V15 if MBEDTLS_BUILTIN
select MBEDTLS_PKCS1_V21 if MBEDTLS_BUILTIN
select MBEDTLS_KEY_EXCHANGE_RSA_ENABLED if MBEDTLS_BUILTIN
Copy link
Collaborator

Choose a reason for hiding this comment

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

Based on #2598 seems like this should go? @valeriosetti

Copy link
Collaborator

Choose a reason for hiding this comment

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

ping @gramshan

@de-nordic de-nordic added the area: zephyr Affects the Zephyr port label Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: zephyr Affects the Zephyr port

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants