Skip to content

Conversation

@dchat-nordic
Copy link
Contributor

Fix the mismatch in API for nrfx_gppi_ep_attach function.

Fix the mismatch in API for nrfx_gppi_ep_attach function.

Signed-off-by: Dominik Chat <[email protected]>
@dchat-nordic dchat-nordic added this to the 3.2.0 milestone Dec 4, 2025
@dchat-nordic dchat-nordic requested a review from a team as a code owner December 4, 2025 11:22
Copilot AI review requested due to automatic review settings December 4, 2025 11:22
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Dec 4, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes an API mismatch in the nrfx_gppi_ep_attach function calls within the Direct Test Mode sample. The function signature was being used with reversed parameter order, where the event address and channel identifier were swapped.

Key Changes:

  • Corrected parameter order in all nrfx_gppi_ep_attach calls to match the proper API signature (event address first, channel second)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dchat-nordic dchat-nordic removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Dec 4, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Dec 4, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: a0f64feb7e605f59c395bd281b6a2703b635268b

more details

sdk-nrf:

PR head: a0f64feb7e605f59c395bd281b6a2703b635268b
merge base: d0f84cb57d7e07a99e2cd487eeb8c0503dd579de
target head (main): d0f84cb57d7e07a99e2cd487eeb8c0503dd579de
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)
samples
│  ├── bluetooth
│  │  ├── direct_test_mode
│  │  │  ├── src
│  │  │  │  │ dtm.c

Outputs:

Toolchain

Version: 43683a87ea
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:43683a87ea_5ea73affbf

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 23
  • ✅ Integration tests
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • 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

@FrancescoSer FrancescoSer requested review from FrancescoSer and removed request for FrancescoSer December 5, 2025 11:09
@carlescufi carlescufi merged commit 76b4c33 into nrfconnect:main Dec 5, 2025
28 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