Skip to content

Conversation

SebastianBoe
Copy link
Contributor

@SebastianBoe SebastianBoe commented Aug 18, 2025

Misc. ironside work september 3.

test_secdom_samples_public: NCSDK-NONE_fix_uicr_path
test_ble_samples: PR-242
test_crypto: PR-853
test_proprietary: PR-115

@SebastianBoe SebastianBoe requested review from a team as code owners August 18, 2025 10:22
@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Aug 18, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 18, 2025

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@d32b794 (main) nrfconnect/sdk-zephyr#3276 nrfconnect/sdk-zephyr#3276/files

DNM label due to: 1 project with PR revision

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 18, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 92810fa41f3f3c8fce65b41b1d3c24f574b1cf7d
zephyr: PR head: 6de68198b55f221defa71c7607e1e011299a258a

more details

sdk-nrf:

PR head: 92810fa41f3f3c8fce65b41b1d3c24f574b1cf7d
merge base: 56c75ef8557a10fb42932c6bd539e00a2f72d8ed
target head (main): 56c75ef8557a10fb42932c6bd539e00a2f72d8ed
Diff

zephyr:

PR head: 6de68198b55f221defa71c7607e1e011299a258a
merge base: d32b79417c5a4c70f16cbbb579d3610daab5f88f
target head (main): d32b79417c5a4c70f16cbbb579d3610daab5f88f
Diff

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 (35)
CMakeLists.txt
cmake
│  ├── sysbuild
│  │  │ periphconf_migrate.cmake
samples
│  ├── pmic
│  │  ├── native
│  │  │  ├── npm13xx_fuel_gauge
│  │  │  │  │ sample.yaml
│  │  │  ├── npm13xx_one_button
│  │  │  │  │ sample.yaml
sysbuild
│  ├── CMakeLists.txt
│  ├── Kconfig.periphconf
│  │ Kconfig.sysbuild
west.yml
zephyr
│  ├── boards
│  │  ├── nordic
│  │  │  ├── nrf54h20dk
│  │  │  │  │ nrf54h20dk_nrf54h20-memory_map.dtsi
│  ├── scripts
│  │  ├── ci
│  │  │  │ check_compliance.py
│  │  ├── west_commands
│  │  │  ├── runners
│  │  │  │  │ nrf_common.py
│  ├── soc
│  │  ├── nordic
│  │  │  ├── Kconfig.sysbuild
│  │  │  ├── common
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── uicr
│  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  ├── Kconfig
│  │  │  │  │  ├── Kconfig.sysbuild
│  │  │  │  │  ├── gen_periphconf_entries.py
│  │  │  │  │  ├── gen_uicr.py
│  │  │  │  │  ├── gen_uicr
│  │  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  │  ├── Kconfig
│  │  │  │  │  │  ├── prj.conf
│  │  │  │  │  │  ├── zephyr
│  │  │  │  │  │  │  │ CMakeLists.txt
│  │  │  │  │  ├── periphconf
│  │  │  │  │  │  ├── __init__.py
│  │  │  │  │  │  │ builder.py
│  │  │  │  │  │ sysbuild.cmake
│  │  │  ├── ironside
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── bootmode.c
│  │  │  │  ├── include
│  │  │  │  │  ├── nrf_ironside
│  │  │  │  │  │  ├── bootmode.h
│  │  │  │  │  │  ├── cpuconf.h
│  │  │  │  │  │  ├── dvfs.h
│  │  │  │  │  │  ├── tdd.h
│  │  │  │  │  │  │ update.h
│  │  │  │ sysbuild.cmake
│  ├── tests
│  │  ├── drivers
│  │  │  ├── comparator
│  │  │  │  ├── gpio_loopback
│  │  │  │  │  │ testcase.yaml

Outputs:

Toolchain

Version: 2b2cd9579a
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:2b2cd9579a_bba2ea5f2e

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ❌ Build twister
  • ❌ Integration tests
    • ❌ test-sdk-audio
    • ❌ desktop52_verification
    • ❌ test-fw-nrfconnect-apps
    • ❌ test_ble_nrf_config
    • ❌ test-fw-nrfconnect-ble_mesh
    • ❌ test-fw-nrfconnect-ble_samples
    • ❌ test-fw-nrfconnect-chip
    • ❌ test-fw-nrfconnect-nfc
    • ❌ test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • ❌ test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • ❌ test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • ❌ test-fw-nrfconnect-nrf-iot_samples
    • ❌ test-fw-nrfconnect-nrf-iot_lwm2m
    • ❌ doc-internal
    • ❌ test-fw-nrfconnect-nrf-iot_thingy91
    • ❌ test-fw-nrfconnect-nrf_crypto
    • ❌ test-fw-nrfconnect-proprietary_esb
    • ❌ test-fw-nrfconnect-rpc
    • ❌ test-fw-nrfconnect-rs
    • ❌ test-fw-nrfconnect-fem
    • ❌ test-fw-nrfconnect-tfm
    • ❌ test-fw-nrfconnect-thread-main
    • ❌ test-sdk-find-my
    • ❌ test-fw-nrfconnect-nrf_lrcs_mosh
    • ❌ test-fw-nrfconnect-nrf_lrcs_positioning
    • ❌ test-sdk-wifi
    • ❌ test-low-level
    • ❌ test-sdk-pmic-samples
    • ❌ test-sdk-mcuboot
    • ❌ test-sdk-dfu
    • ❌ test-fw-nrfconnect-ps-main
    • ❌ test-secdom-samples-public
    • ⚠️ test-fw-nrfconnect-nrf-iot_cloud

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

@57300 57300 added the CI-all-test Run All integration tests label Aug 24, 2025
Copy link

github-actions bot commented Aug 24, 2025

You can find the documentation preview for this PR here.

@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label Sep 3, 2025
@SebastianBoe SebastianBoe changed the title ironside: Generate periphconf from Zephyr images instead of sysbuild Misc. ironside work september 3. Sep 3, 2025
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Sep 3, 2025
Copy link
Contributor

@nordicjm nordicjm left a comment

Choose a reason for hiding this comment

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

and final 2 commits should be squashed rather than fixups

@SebastianBoe SebastianBoe force-pushed the ps branch 4 times, most recently from 12dac27 to c9a3579 Compare September 10, 2025 09:02
@jonathannilsen jonathannilsen force-pushed the ps branch 2 times, most recently from 8a8e4f2 to d1e7a90 Compare September 15, 2025 07:48
@jonathannilsen jonathannilsen removed the CI-all-test Run All integration tests label Sep 15, 2025
@jonathannilsen jonathannilsen force-pushed the ps branch 2 times, most recently from b4e213c to 628446e Compare September 17, 2025 08:29
@NordicBuilder NordicBuilder added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. and removed doc-required PR must not be merged without tech writer approval. labels Sep 17, 2025
@jonathannilsen jonathannilsen force-pushed the ps branch 3 times, most recently from f281120 to 9db0407 Compare September 18, 2025 05:19
SebastianBoe and others added 3 commits September 18, 2025 10:29
Apply the shield to the app only.

Don't apply it to other images in the sysbuild, i.e. 54h20's UICR
image.

Signed-off-by: Sebastian Bøe <[email protected]>
PERIPHCONF generation based on devicetree is being added in a script
in upstream zephyr. The new script script replaces nrf-regtool and the
temporary "periphconf migration" solution.

Ref: NCSDK-35206

Signed-off-by: Jonathan Nilsen <[email protected]>
Pull in new build system support for generating PERIPHCONF entries
based on devicetree. This commit is deliberately split from the related
build system changes to enable cherry-picking those without the
manifest update.

Ref: NCSDK-35206

Signed-off-by: Jonathan Nilsen <[email protected]>
@SebastianBoe
Copy link
Contributor Author

all commits are now in NCS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. DNM manifest manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.