Skip to content

[collab-nrfx-4.0] Fix for i2c_latency test and mcuboot with external flash #25337

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 3 commits into
base: collab-nrfx-4.0
Choose a base branch
from
Open

[collab-nrfx-4.0] Fix for i2c_latency test and mcuboot with external flash #25337

wants to merge 3 commits into from

Conversation

@nordic-krch
Copy link
Contributor

@nordic-krch nordic-krch commented Oct 31, 2025

Fixes for common CI failures.

test-sdk-nrf: collab-nrfx-4.0

@nordic-krch nordic-krch added the CI-all-test Run All integration tests label Oct 31, 2025
@nordic-krch nordic-krch requested a review from a team as a code owner October 31, 2025 07:10
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 31, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 31, 2025 •
edited
Loading

CI Information

To view the history of this post, click the 'edited' button above
Build number: 3

Inputs:

Sources:

sdk-nrf: PR head: bd05fd6d5f975f72d55ff7a4ed096add5e57de5c

more details

sdk-nrf:

PR head: bd05fd6d5f975f72d55ff7a4ed096add5e57de5c
merge base: e220384c6c4517c958d3cee246129bea39cabcf6
target head (collab-nrfx-4.0): 5ce1b7ffc789411f1fa99ae27b5d2284fe2befa4
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 (9) 
tests
│  ├── drivers
│  │  ├── audio
│  │  │  ├── pdm_loopback
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  ├── i2c
│  │  │  ├── i2c_latency
│  │  │  │  ├── boards
│  │  │  │  │  ├── nrf52840dk_nrf52840.conf
│  │  │  │  │  ├── nrf5340dk_nrf5340_cpuapp.conf
│  │  │  │  │  ├── nrf54h20dk_nrf54h20_cpuapp.conf
│  │  │  │  │  │ nrf54l15dk_nrf54l15_cpuapp.conf
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  ├── modules
│  │  ├── mcuboot
│  │  │  ├── external_flash
│  │  │  │  ├── sysbuild
│  │  │  │  │  ├── mcuboot
│  │  │  │  │  │  │ prj.conf

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 - Skipped: Skipping Build & Test as it succeeded in a previous run: 2
  • ❌ Integration tests
    • ❌ desktop52_verification
    • ❌ test-fw-nrfconnect-apps
    • ❌ test-fw-nrfconnect-ble_samples
    • ❌ test-fw-nrfconnect-nfc
    • ✅ test-fw-nrfconnect-nrf-iot_thingy91 - Skipped: Job was skipped as it succeeded in a previous run
    • ❌ test-fw-nrfconnect-rpc
    • ✅ test-sdk-wifi - Skipped: Job was skipped as it succeeded in a previous run
    • ❌ test-low-level
    • ⚠️ test-sdk-mcuboot

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 31, 2025 •
edited
Loading

Memory footprint analysis revealed the following potential issues

applications.nrf_desktop.zdebug[nrf52840gmouse/nrf52840]: ROM size increased by 3604[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[nrf54l15dk/nrf54l15/cpuapp]: ROM size increased by 800[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.bluetooth.fast_pair.locator_tag.release[nrf52833dk/nrf52833]: ROM size increased by 1896[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 9102[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 5858[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.nrf_desktop.zrelease.usb_next[nrf52840dk/nrf52840]: ROM size increased by 3244[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.find_my.locator_tag.release[nrf54l15dk/nrf54l10/cpuapp]: ROM size increased by 1736[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug.uart[nrf54l15dk/nrf54l15/cpuapp]: ROM size increased by 652[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_4llpmconn[nrf52840dongle/nrf52840]: ROM size increased by 548[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.bluetooth.fast_pair.locator_tag.release[nrf5340dk/nrf5340/cpuapp]: ROM size increased by 1560[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug[nrf52840dk/nrf52840]: ROM size increased by 3300[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease_keyboard[nrf54l15dk/nrf54l05/cpuapp]: ROM size increased by 1140[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.find_my.locator_tag.release[nrf52833dk/nrf52833]: ROM size increased by 592[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug.usb_next[nrf52840gmouse/nrf52840]: ROM size increased by 3584[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_keyboard.uart[nrf54l15dk/nrf54l10/cpuapp]: ROM size increased by 692[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_keyboard.uart[nrf54l15dk/nrf54l15/cpuapp]: ROM size increased by 704[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_keyboard.usb_next[nrf52840dk/nrf52840]: ROM size increased by 3516[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_keyboard[nrf52840dk/nrf52840]: ROM size increased by 3700[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebugwithshell[nrf52840dk/nrf52840]: ROM size increased by 3284[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_mcuboot_qspi[nrf52840dk/nrf52840]: ROM size increased by 3288[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_wwcb[nrf52840dk/nrf52840]: ROM size increased by 3288[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.find_my.switchable_networks.release.dfu_switch.apple[nrf52833dk/nrf52833]: ROM size increased by 1528[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug[nrf52840dongle/nrf52840]: ROM size increased by 548[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease_fast_pair.keyboard[nrf52kbd/nrf52832]: ROM size increased by 2024[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.bluetooth.fast_pair.locator_tag.release[thingy53/nrf5340/cpuapp]: ROM size increased by 1292[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 12744[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 9492[B] - link (cc: @nrfconnect/ncs-ll-ursus)
sample.bluetooth.fast_pair.locator_tag.release[nrf54lm20dk/nrf54lm20a/cpuapp]: ROM size increased by 1144[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug[nrf52820dongle/nrf52820]: ROM size increased by 12304[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_fast_pair.gmouse[nrf52840gmouse/nrf52840]: ROM size increased by 1324[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.find_my.switchable_networks.release.dfu_switch.apple[nrf54l15dk/nrf54l05/cpuapp]: ROM size increased by 740[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.find_my.switchable_networks.release.dfu_switch.google[nrf54l15dk/nrf54l05/cpuapp]: ROM size increased by 764[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.smp_svr_mini_boot[[email protected]/nrf54h20/cpuapp]: ROM size increased by 2204[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-pluto @robertstypa @maciejpietras)
applications.nrf_desktop.zdebug[nrf52833dongle/nrf52833]: ROM size increased by 520[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.bluetooth.fast_pair.locator_tag.release[nrf52dk/nrf52832]: ROM size increased by 1940[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.find_my.locator_tag.release[nrf52dk/nrf52832]: ROM size increased by 756[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.find_my.switchable_networks.release.dfu_switch.google[nrf52833dk/nrf52833]: ROM size increased by 1520[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug[nrf52833dk/nrf52820]: ROM size increased by 12520[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.bluetooth.fast_pair.locator_tag.release[nrf54l15dk/nrf54l05/cpuapp]: ROM size increased by 1024[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[nrf52840dongle/nrf52840]: ROM size increased by 672[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.find_my.switchable_networks.release.dfu_switch[nrf52833dk/nrf52833]: ROM size increased by 3880[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug.usb_next[nrf52840dk/nrf52840]: ROM size increased by 3120[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug[nrf52kbd/nrf52832]: ROM size increased by 4120[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[nrf52820dongle/nrf52820]: ROM size increased by 2920[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[nrf52833dk/nrf52820]: ROM size increased by 3048[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug[nrf52dmouse/nrf52832]: ROM size increased by 3192[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_nrf21540ek[nrf52840dk/nrf52840]: ROM size increased by 3412[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[nrf52kbd/nrf52832]: ROM size increased by 4296[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[nrf54lm20dk/nrf54lm20a/cpuapp]: ROM size increased by 680[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug.uart[nrf54l15dk/nrf54l10/cpuapp]: ROM size increased by 640[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
sample.find_my.switchable_networks.release.dfu_switch[nrf54l15dk/nrf54l05/cpuapp]: ROM size increased by 3108[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug[nrf54l15dk/nrf54l05/cpuapp]: ROM size increased by 896[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_mcuboot_smp[nrf52840dk/nrf52840]: ROM size increased by 3288[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[nrf52840gmouse/nrf52840]: ROM size increased by 3404[B] in comparison to the main[cf2ce9a] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)

Note: This message is automatically posted and updated by the CI (latest/sdk-nrf/PR-25337/2)

nika-nordic
nika-nordic requested changes Oct 31, 2025
View reviewed changes
Copy link
Contributor

@nika-nordic nika-nordic left a comment

Choose a reason for hiding this comment

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

please fix

	zassert_equal(NRFX_SUCCESS, nrfx_twis_init(&twis, &config, i2s_slave_handler),

as well - 0 instead of NRFX_SUCCESS

nordic-krch and others added 3 commits October 31, 2025 13:17
@nordic-krch
tests: drivers: i2c: i2c_latency: Align to changes in nrfx_twis
05a96ed 
CONFIG_NRFX_TWIXxx options has been removed. Aligning test to
not use those.

Signed-off-by: Krzysztof Chruściński <[email protected]>
@nika-nordic @nordic-krch
tests: modules: mcuboot: external_flash: disable gpio interrupts
7dfcf5c 
They are not needed by the test and consumes Flash.

Signed-off-by: Nikodem Kastelik <[email protected]>
@nordic-krch
tests: drivers: audio: pdm_loopback: Align to nrfx_timer update
bd05fd6 
Align test to update nrfx_timer API.

Signed-off-by: Krzysztof Chruściński <[email protected]>
@nordic-krch
Copy link
Contributor Author

nordic-krch commented Oct 31, 2025

@nika-nordic done, also fixed pdm_loopback after nrfx_timer update.

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

Reviewers

@nika-nordic nika-nordic nika-nordic requested changes

@nordic-segl nordic-segl nordic-segl approved these changes

@magp-nordic magp-nordic magp-nordic approved these changes

@nordic-bami nordic-bami nordic-bami approved these changes

@mstasiaknordic mstasiaknordic mstasiaknordic approved these changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. CI-all-test Run All integration tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@nordic-krch @NordicBuilder @nika-nordic @nordic-segl @magp-nordic @nordic-bami @mstasiaknordic