Skip to content

CI branch to integrate the latest vanilla TF-M with Mbed OS #94

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

Conversation

LDong-Arm
Copy link
Contributor

@LDong-Arm LDong-Arm commented Mar 17, 2021

This PR

  • enables the IPC suites for Musca S1 (whose support was supported recently)
    Note: temporarily disabled again, as it can't run due to an issue in MRAM fast-read...
  • switches to fetch vanilla TF-M
  • updates the list of test files
  • disables unsupported IRQ tests (not supported for by TF-M in PSA mode anymore)

Notes:

@LDong-Arm LDong-Arm force-pushed the tfm_latest_integration_check branch 4 times, most recently from b0843be to 96e1615 Compare March 17, 2021 14:01
@LDong-Arm LDong-Arm requested a review from evedon March 19, 2021 12:11
@LDong-Arm LDong-Arm force-pushed the tfm_latest_integration_check branch from 96e1615 to 241037d Compare March 22, 2021 11:13
The following commit in trusted-firmware-m

    b774fecd SPM: Remove *TFM* IRQ APIs for IPC Model

means that we need to disable IRQ tests in the regression suite.
TF-M redirects output to serial by declaring its own `FILE __stdout`
and disables the toolchain's default version of this symbol using
the flag `-nostdlib`. But stdlib is enabled and required by Mbed OS,
so we need to disable the one from TF-M's libplatform_ns to avoid
symbol duplication.
Musca B1's path has been renamed in TF-M so we need to update
`targets.json` in Mbed OS accordingly. As a CI-only branch, we
limit the scope to Musca S1 only.
@LDong-Arm LDong-Arm force-pushed the tfm_latest_integration_check branch from 241037d to ac1bc20 Compare March 22, 2021 11:24
Copy link

@evedon evedon left a comment

Choose a reason for hiding this comment

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

Looks good

@LDong-Arm LDong-Arm merged commit 6af5699 into ARMmbed:tfm_latest_integration_check Mar 22, 2021
@LDong-Arm LDong-Arm deleted the tfm_latest_integration_check branch June 7, 2021 13:04
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