Skip to content

tests: drivers: gpio: gpio_nfct: Fix nrf7120dk overlay#28763

Merged
nordicjm merged 1 commit into
nrfconnect:mainfrom
dhsu-nordic:nrf7120-nfct-test
Jun 1, 2026
Merged

tests: drivers: gpio: gpio_nfct: Fix nrf7120dk overlay#28763
nordicjm merged 1 commit into
nrfconnect:mainfrom
dhsu-nordic:nrf7120-nfct-test

Conversation

@dhsu-nordic
Copy link
Copy Markdown
Contributor

The nRF7120 uses a new dedicated UICR binding (nordic,nrf71-uicr) which does not declare the legacy nfct-pins-as-gpios property. On v2 NFCT SoCs (nRF54H, nRF54L, nRF7120), the property has been moved to the NFCT node itself via the nordic,nrf-nfct-v2 binding.

Move nfct-pins-as-gpios from the &uicr node to the &nfct node and disable NFCT, matching the form already used by the nRF54H20 overlay in this test.

The nRF7120 uses a new dedicated UICR binding (nordic,nrf71-uicr)
which does not declare the legacy nfct-pins-as-gpios property.
On v2 NFCT SoCs (nRF54H, nRF54L, nRF7120), the property has been
moved to the NFCT node itself via the nordic,nrf-nfct-v2 binding.

Move nfct-pins-as-gpios from the &uicr node to the &nfct node and
disable NFCT, matching the form already used by the nRF54H20 overlay
in this test.

Signed-off-by: Dhanoo Surasarang <dhanoo.surasarang@nordicsemi.no>
@dhsu-nordic dhsu-nordic requested review from a team as code owners May 13, 2026 14:57
@dhsu-nordic
Copy link
Copy Markdown
Contributor Author

This depends on nrfconnect/sdk-zephyr#4068 which switches the nRF7120 nfct compatible to nordic,nrf-nfct-v2 and extends the binding description to cover nRF7120.

@dhsu-nordic dhsu-nordic requested a review from nordic-krch May 13, 2026 15:00
@NordicBuilder
Copy link
Copy Markdown
Contributor

NordicBuilder commented May 13, 2026

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 6127bde3d323f878e2f6ce42838fc80710223862

more details

sdk-nrf:

PR head: 6127bde3d323f878e2f6ce42838fc80710223862
merge base: 7ed83c23a4422ecb9e01e9459f9a9168fc69bc05
target head (main): 0dc2def48137edc1e0415bc72bb5dc94d6191e29
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 (1)
tests
│  ├── drivers
│  │  ├── gpio
│  │  │  ├── gpio_nfct
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf7120dk_nrf7120_cpuapp.overlay

Outputs:

Toolchain

Version: f0aa129f09
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:f0aa129f09_5ea73affbf

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

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

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

@dhsu-nordic dhsu-nordic added this to the 3.5.0 milestone Jun 1, 2026
@nordicjm nordicjm merged commit 1ba0e38 into nrfconnect:main Jun 1, 2026
21 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants