Skip to content

Conversation

tomi-font
Copy link
Contributor

Bring updated tf-m-tests repo plus commits touching TF-M integration/samples.

Copy link

github-actions bot commented Aug 18, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
psa-arch-tests zephyrproject-rtos/psa-arch-tests@2cadb02 (main) zephyrproject-rtos/psa-arch-tests#15 zephyrproject-rtos/psa-arch-tests#15/files
tf-m-tests zephyrproject-rtos/tf-m-tests@c712761 zephyrproject-rtos/tf-m-tests#15 zephyrproject-rtos/tf-m-tests#15/files

DNM label due to: 2 projects with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

Copy link
Contributor

@frkv frkv left a comment

Choose a reason for hiding this comment

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

LGTM pending testing

ceolin and others added 7 commits August 20, 2025 15:25
Update tests to use tfm-2.2.0.

Signed-off-by: Flavio Ceolin <[email protected]>
(cherry picked from commit 1ff7d1c)
TEST_PSA_API requires TFM_PROFILE to be set to "profile_large".

Signed-off-by: Flavio Ceolin <[email protected]>
(cherry picked from commit 8caef68)
…rkaround

Add a workaround which sets the isolation level to 2 in CMake.
The TF-M tests require the large TF-M profile because it supports
the full list of crypto algorithms needed, not because of the
solation level.

For the TF-M tests the isolation level is irrelevant so we
set it to 2 here so that we don't exclude the platforms which
don't support the isolation level 3.

I communicated with a TF-M maintainer and he informed me that this
workaround will be included in the TF-M 2.2 release branch later
so this is a short lived workaround that can be reverted when the
relevant commit from the TF-M branch is cherry-picked.

Signed-off-by: Georgios Vasilakis <[email protected]>
(cherry picked from commit 0081207)
Starting in TF-M 2.2.0 there is a build option to treat warnings as
errors. Lets use it.

Signed-off-by: Flavio Ceolin <[email protected]>
(cherry picked from commit ca7d99a)
z_arm_save_fp_context and z_arm_restore_fp_context
save and restore fpu context regardless of the
CONFIG_FPU_SHARING setting.

This modification is required to support suspend to ram use cases
where save and restore of FPU state is needed to ensure proper bahaviour
after wakeup.

Signed-off-by: Michele Sardo <[email protected]>
(cherry picked from commit 5981dc8)
…mentation

Following changes to arch/arm/core/cortex_m/fpu.c,
the dependency on CONFIG_FPU_SHARING is moved into this file.

Signed-off-by: Michele Sardo <[email protected]>
(cherry picked from commit 7bc1756)
Fixup to the TF-M 2.2.0 update.

Signed-off-by: Tomi Fontanilles <[email protected]>

Upstream PR #: 94729
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants