Skip to content
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

Test branch psa #20151

Closed
wants to merge 110 commits into from
Closed

Conversation

sean-madigan
Copy link
Contributor

No description provided.

@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. manifest ble mesh Label for ble mesh PRbot. Add this if PR is related to ble mesh and you need to get review. labels Jan 31, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jan 31, 2025

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 11

Inputs:

Sources:

more details

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (0)

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ❌ Toolchain
  • ❌ Build twister
  • ❌ Integration tests

Note: This message is automatically posted and updated by the CI

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jan 31, 2025

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

Name Old Revision New Revision Diff
dragoon d85f37138db71ea6376260fb3ed5819faa4b97ee develop/DRGN-23817_adapt_for_upmerge_20241220 N/A
find-my https://github.com/nrfconnect/sdk-find-my/commit/719d36b15db3f228b0d83c52f13b97145d971ea4 (main) https://github.com/nrfconnect/sdk-find-my/pull/260 nrfconnect/sdk-find-my#260/files
mcuboot nrfconnect/sdk-mcuboot@3a25855 (main) nrfconnect/sdk-mcuboot#393 nrfconnect/sdk-mcuboot#393/files
nrfxlib nrfconnect/sdk-nrfxlib@1cdbd26 nrfconnect/sdk-nrfxlib#1648 nrfconnect/sdk-nrfxlib#1648/files
sidewalk nrfconnect/sdk-sidewalk@f274ce8 nrfconnect/sdk-sidewalk#673 nrfconnect/sdk-sidewalk#673/files
suit-processor nrfconnect/suit-processor@a499bca nrfconnect/suit-processor@a1a9ef7 nrfconnect/suit-processor@a499bcad..a1a9ef77
zephyr nrfconnect/sdk-zephyr@693769a (main) nrfconnect/sdk-zephyr#2411 nrfconnect/sdk-zephyr#2411/files

DNM label due to: 5 projects with PR revision and 1 unreachable repo

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

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link.

Note: This comment is automatically posted by the Documentation Publish GitHub Action.

@sean-madigan sean-madigan force-pushed the test_branch_psa branch 3 times, most recently from a63bd57 to 8bf9490 Compare February 5, 2025 15:15
anangl and others added 9 commits February 5, 2025 16:26
Add corrections for a bunch of sdk-zephyr revert commits that were
cherry-picked after rebase without updating their references to
reverted commits.
Remove a no longer needed correction for one sdk-trusted-firmware-m
commit.

The added corrections can be removed after the next sdk-zephyr rebase.

Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
Pull in upmerged revisions of zephyr and mcuboot.

Update other modules with related adjustments.

Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
Adapt to the latest changes applied to Zephyr.

Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
The bluetooth shell has been refactored (see
bf897cf941a514e7ea7a5f837d2287e360281f24), so the sourced file no longer
exists. Note that it was actually a bad use of sourcing, as the upstream
options are already part of the Kconfig tree (unless I missed
something).

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
- Upgrade to latest Sphinx
- Fix some references
- Fix Zephyr docset required env vars

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Align with list of inputs in upstream Zephyr.

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
According to the new extension parameters.

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Now the output will be $name/html|xml, so that we can generate
other outputs other than HTML only, e.g. XML.

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Instead of the removed CONFIG_NRF_REGTOOL_GENERATE_BICR.

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
nordicjm and others added 23 commits February 5, 2025 16:26
Fixes an issue with this test wrongly using 0 as the area size

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Output uses logging, which has no guarantee of even being enabled
or supported

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
- `native_sim/native` needs to be used now instead of `native_sim`
- add entries for failing samples and tests that are to be handled
  after the upmerge:
  * sample.mcumgr.smp_svr.ram_load.*
  * nrf_modem_dect_phy_pcc_event
  * secure_storage.psa.its.secure_storage.*
  * sample.net.prometheus
- remove `native_posix`

Signed-off-by: Maciej Perkowski <maciej.perkowski@nordicsemi.no>
Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
So far when building for nRF54L15 static DPPI channels were assigned.
After adding nRF54L15 support in the DPPI driver this code is unused
for nRF54L15 and the common DPPI allocator used hardcoded DPPIC
instance. This commit parameterizes the DDPIC instance number depending
on the SoC and enables the coresponding instances in the ESB samples'
configurations. ESB uses hardcoded instance number per SoC.

TODO: Allow selection of the DPPIC instance in one place e.g. Kconfig
based on the DTS.

Signed-off-by: Michał Grochala <michal.grochala@nordicsemi.no>
- Change posix names to zsock.
- Remove CONFIG_x_NRF_HW_ASYNC usage.
- Correct size for the external flash.

Signed-off-by: Markus Lassila <markus.lassila@nordicsemi.no>
Function has been deprecated in upstream zephyr, so remove
from here.

Signed-off-by: Sean Madigan <sean.madigan@nordicsemi.no>
Let HOSTAP_CRYPTO_ALT_LEGACY_PSA select MBEDTLS_MD5_C so that
md5 related calls are available when crypto_alt is used.

Signed-off-by: Ravi Dondaputi <ravi.dondaputi@nordicsemi.no>
In scan_only build we need to override the sysbuild config option
as well. By default it gets set to SB_CONFIG_WIFI_NRF70_SYSTEM_MODE
which enables supplicant. Overriding this with
SB_CONFIG_WIFI_NRF70_SCAN_ONLY fixes this.

Signed-off-by: Ravi Dondaputi <ravi.dondaputi@nordicsemi.no>
The ZMS implementation of the ITS store module is not supported when
partition manager is used because the DT chosen
`secure_storage_its_partition` cannot be used.
This would require using a different, partition manager-specific
configuration.
As partition manager will be replaced this combination is just not
supported.

Signed-off-by: Tomi Fontanilles <tomi.fontanilles@nordicsemi.no>
We cannot render/link Doxygen groups that belong to another project like
Zephyr, see nrfconnect#19341 for
progress on that.

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Add Zephyr tests due to overflow issues.

Refers:
NCSDK-31589
NCSDK-31590
NCSDK-31591

Signed-off-by: Katarzyna Giądła <katarzyna.giadla@nordicsemi.no>
BLE logging size requirements have increased a lot.
We need to reduce logging to fit the application.

Signed-off-by: Maximilian Deubel <maximilian.deubel@nordicsemi.no>
Switched from speed to size optimization for nRF54H20 DK target in
debug configurations to reduce memory footprint.

Jira: NCSDK-31374

Signed-off-by: Mateusz Kapala <mateusz.kapala@nordicsemi.no>
Switched from speed to size optimizations for nRF54L15 DK (nRF54L05
emulated SoC) and enabled LTO in debug and release configurations
to reduce memory footprint.

Jira: NCSDK-31374

Signed-off-by: Mateusz Kapala <mateusz.kapala@nordicsemi.no>
Remove reference of z_impl_sys_rand_get().

Signed-off-by: Eivind Jølsgard <eivind.jolsgard@nordicsemi.no>
Updates required by changes in zephyr upstream for the BT HCI driver.

Signed-off-by: Tommi Rantanen <tommi.rantanen@nordicsemi.no>
Update pm policy event usage after API was changed from
using relative time in cycles to absolute time in ticks.

Signed-off-by: Knut Eldhuset <knut.eldhuset@nordicsemi.no>
This test is not really useful in NCS context (checking if the nRF
Wi-Fi driver can be built successfully is done by other tests and
samples), and it requires the hal_atmel module to be pulled in
(because the WINC1500 driver requires two header files that are
located in that module). And the module weights ~87 MB, so it's
not reasonable to pull it in just for this one test.

Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
Fixes an issue with stream writer by providing the size of the
partition

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Change posix names to zsock in nrf_cloud lib.

Signed-off-by: Maximilian Deubel <maximilian.deubel@nordicsemi.no>
Some sdk-zephyr tests and samples reports issues due to
RAM overflow issues.

Refers:
NCSDK-31667
NCSDK-31671

Signed-off-by: Katarzyna Giądła <katarzyna.giadla@nordicsemi.no>
Applications `applications.nrf_desktop.zrelease` and
`applications.nrf_desktop.zdebug` report RAM overflow.

Therefore, applications on listed platforms are moved
to quaratine.

Refers: NCSDK-31675

Signed-off-by: Katarzyna Giądła <katarzyna.giadla@nordicsemi.no>
Increase `BT_LONG_WQ_STACK_SIZE` after BT switching to PSA API.
Place the increased value in `KConfig.defaults` for now due to
NCSDK-31391 issue.

Signed-off-by: Maciej Baczmanski <maciej.baczmanski@nordicsemi.no>
sean-madigan and others added 4 commits February 5, 2025 16:01
PSA requires large stacks, especially on 54.

Signed-off-by: Sean Madigan <sean.madigan@nordicsemi.no>
Signed-off-by: Frank Audun Kvamtrø <frank.kvamtro@nordicsemi.no>
-This added SYS_INIT will ensure that psa_crypto_init is called
 in kernel *after* Hw_CC3XX is executed to ensure that mutexes are
 ready to be used.

Signed-off-by: Frank Audun Kvamtrø <frank.kvamtro@nordicsemi.no>
This is required by PSA in NCS, so make sure it is selected
NRF_SECURITY_ENABLER.

Signed-off-by: Sean Madigan <sean.madigan@nordicsemi.no>
ENTROPY_GENERATOR is required but was not being selected for
nrf53 bsim for some reason.

Also the heap size required for bsim is larger than on
actual target, reason unknown.

Signed-off-by: Sean Madigan <sean.madigan@nordicsemi.no>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.