-
Notifications
You must be signed in to change notification settings - Fork 1.4k
tests: tfm: add nrf7120 ns tests #25772
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
base: main
Are you sure you want to change the base?
tests: tfm: add nrf7120 ns tests #25772
Conversation
CI InformationTo view the history of this post, click the 'edited' button above Inputs:Sources:trusted-firmware-m: PR head: 082c81977976ac0992be2a95e4299c31b6637e50 more detailstrusted-firmware-m:
sdk-nrf:
Github labels
List of changed files detected by CI (42)Outputs:ToolchainVersion: f911d4f4e7 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
de6930c to
576b798
Compare
|
Depend on psa-arch-tests target support: ARM-software/psa-arch-tests#416 |
rob-robinson-14
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Has this been verified with Malago pinsetA (which exposes uart20)?
Please also note we will need https://github.com/rob-robinson-14/sdk-nrf/tree/malago-fpga-pinset commits included too (to make use of new FPGA)- perhaps worth cherry-pick/ rebase of these commits.
samples/crypto/persistent_key_usage/boards/nrf7120pdk_nrf7120_cpuapp_ns.conf
Outdated
Show resolved
Hide resolved
samples/crypto/psa_tls/boards/nrf7120pdk_nrf7120_cpuapp_ns.conf
Outdated
Show resolved
Hide resolved
tests/tfm/tfm_regression_test/boards/nrf7120pdk_nrf7120_cpuapp_ns.conf
Outdated
Show resolved
Hide resolved
66e4310 to
9d657e8
Compare
|
You can find the documentation preview for this PR here. Preview links for modified nRF Connect SDK documents: https://ncsdoc.z6.web.core.windows.net/PR-25772/nrf/releases_and_maturity/releases/release-notes-changelog.html |
umapraseeda
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need a changelog entry?
Not for nrf7120? Because we not yet officially release this SoC |
|
The following west manifest projects have changed revision in this Pull Request:
⛔ DNM label due to: 2 projects with PR revision Note: This message is automatically posted and updated by the Manifest GitHub Action. |
8bdbde2 to
c545533
Compare
c545533 to
1729405
Compare
tomi-font
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Skipped the psa_tls sample?
| #define TFM_GPIOTE_N_IRQ_SIGNAL TFM_GPIOTE0_IRQ_SIGNAL | ||
|
|
||
| #elif defined(CONFIG_SOC_SERIES_NRF54LX) | ||
| #elif defined(CONFIG_SOC_SERIES_NRF54LX) || defined(CONFIG_SOC_SERIES_NRF71X) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we use the Lumos platform Kconfig option instead?
nordicjm
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
placing a block on this until the PR has been merged which prevents this board target from using partition manager has been merged, and this PR has been rebased and CI passes showing it runs fine without PM
a1227b1 to
f14e53e
Compare
add nrf7120dk ns board test for crypto samples and tests tfm, nrf7120 uses uart20 for non-secure application printing. Signed-off-by: Travis Lam <[email protected]>
Point to sdk-psa-arch-tests update for nrf7120 Point to sdk-trusted-firmware-m update for nrf7120 uart20 switch Signed-off-by: Travis Lam <[email protected]>
f14e53e to
e70fdca
Compare
test to reenable partition maanger to pass CI Signed-off-by: Travis Lam <[email protected]>
| depends on SOC_SERIES_NRF52X || SOC_SERIES_NRF53X || SOC_SERIES_NRF91X || \ | ||
| SOC_NRF54L05 || SOC_NRF54L10 || SOC_NRF54L15 || SOC_NRF54LM20A || \ | ||
| SOC_NRF54LV10A_ENGA || SOC_NRF54LS05B_ENGA | ||
| SOC_NRF54LV10A_ENGA || SOC_SERIES_NRF71X || SOC_NRF54LS05B_ENGA |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not allowed as per Shantha's instructions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hold on this PR until DTS way for TFM memory partitioning is supported.
Memory footprint analysis revealed the following potential issuesapplications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 12744[B] - link (cc: @nrfconnect/ncs-ll-ursus) Note: This message is automatically posted and updated by the CI (latest/sdk-nrf/PR-25772/18) |
add ns board test for crypto samples and tests tfm, nrf7120 uses uart20 for non-secure application printing.