Skip to content

Conversation

@ryanjh
Copy link
Contributor

@ryanjh ryanjh commented Apr 2, 2025

The CS complete callbacks provide both status and params. In the case of errors, NULL pointer is passed to the params of callbacks.

@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 2, 2025
@ryanjh ryanjh force-pushed the cs_complete_events_with_status branch from 4e5d1dc to e8fe104 Compare April 2, 2025 11:28
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Apr 2, 2025

CI Information

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

Inputs:

Sources:

hal_nordic: PR head: 8cdcc33c6e5c7618ca4e77ff19a0eab557cc4bd5
sdk-nrf: PR head: 4eb8c2052706aa78167ee3c39d1654240bd2d098
zephyr: PR head: e6ab653d822c1e8a12859b887c8f0d4dfb163e99

more details

hal_nordic:

PR head: 8cdcc33c6e5c7618ca4e77ff19a0eab557cc4bd5
merge base: 7440d573d36953b80cb6fb6438b147acb21bf753
Diff

sdk-nrf:

PR head: 4eb8c2052706aa78167ee3c39d1654240bd2d098
merge base: d9c0c0fd8ba98e1d8bf2643f2c6a2422506ad16b
target head (main): 1e2a4672d46f1c9355e60db3c724efda2dcbb97e
Diff

zephyr:

PR head: e6ab653d822c1e8a12859b887c8f0d4dfb163e99
merge base: f55a8067736a905992ef47d164c87443879b273c
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 (16)
modules
│  ├── hal
│  │  ├── nordic
│  │  │  ├── nrfx
│  │  │  │  ├── drivers
│  │  │  │  │  ├── include
│  │  │  │  │  │  │ nrfx_twim.h
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ nrfx_twim.c
samples
│  ├── bluetooth
│  │  ├── channel_sounding_ras_initiator
│  │  │  ├── src
│  │  │  │  │ main.c
│  │  ├── channel_sounding_ras_reflector
│  │  │  ├── src
│  │  │  │  │ main.c
subsys
│  ├── bluetooth
│  │  ├── services
│  │  │  ├── ras
│  │  │  │  ├── rrsp
│  │  │  │  │  │ ras_rd_buffer.c
west.yml
zephyr
│  ├── include
│  │  ├── zephyr
│  │  │  ├── bluetooth
│  │  │  │  │ conn.h
│  ├── samples
│  │  ├── bluetooth
│  │  │  ├── channel_sounding
│  │  │  │  ├── src
│  │  │  │  │  ├── connected_cs_initiator.c
│  │  │  │  │  │ connected_cs_reflector.c
│  ├── subsys
│  │  ├── bluetooth
│  │  │  ├── host
│  │  │  │  ├── conn.c
│  │  │  │  ├── conn_internal.h
│  │  │  │  ├── cs.c
│  │  │  │  ├── shell
│  │  │  │  │  │ bt.c
│  ├── tests
│  │  ├── bluetooth
│  │  │  ├── host
│  │  │  │  ├── cs
│  │  │  │  │  ├── mocks
│  │  │  │  │  │  ├── conn.c
│  │  │  │  │  │  │ conn.h
│  │ west.yml

Outputs:

Toolchain

Version: 7dbe0e2658
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:7dbe0e2658_8bf7ca4353

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 1270
    • sdk-zephyr test count: 799
  • ✅ Integration tests
    • ✅ test-sdk-audio
    • ✅ desktop52_verification
    • ✅ test_ble_nrf_config
    • ✅ test-fw-nrfconnect-ble_samples
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-fw-nrfconnect-nfc
    • ✅ test-fw-nrfconnect-nrf-iot_thingy91
    • ✅ test-sdk-find-my
    • ✅ test-low-level
    • ✅ test-sdk-dfu
Disabled integration tests
    • doc-internal
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

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

@ryanjh ryanjh marked this pull request as ready for review April 2, 2025 12:05
@ryanjh ryanjh requested a review from a team as a code owner April 2, 2025 12:05
@ryanjh ryanjh requested review from a team as code owners April 2, 2025 12:35
@ryanjh ryanjh force-pushed the cs_complete_events_with_status branch from 90d8996 to 72e569d Compare April 2, 2025 12:55
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Apr 2, 2025

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@f55a806 nrfconnect/sdk-zephyr@e6ab653 (main) nrfconnect/[email protected]

All manifest checks OK

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

@github-actions
Copy link

github-actions bot commented Apr 2, 2025

You can find the documentation preview for this PR here.

@ivaniushkov ivaniushkov self-requested a review April 3, 2025 07:38
@ryanjh ryanjh force-pushed the cs_complete_events_with_status branch from 72e569d to 38e34aa Compare April 3, 2025 07:59
@NordicBuilder NordicBuilder removed the DNM label Apr 3, 2025
@rlubos
Copy link
Contributor

rlubos commented Apr 3, 2025

@ryanjh Please rebase

ryanjh added 2 commits April 3, 2025 10:15
The CS complete callbacks provide both status and params.
In the case of errors, NULL pointer is passed to the params
of callbacks.

Signed-off-by: Chu Ryan <[email protected]>
Report status of Channel Sounding complete events.

Signed-off-by: Chu Ryan <[email protected]>
@ryanjh ryanjh force-pushed the cs_complete_events_with_status branch from 38e34aa to 4eb8c20 Compare April 3, 2025 08:16
@ryanjh ryanjh requested a review from olivier-le-sage April 3, 2025 08:27
@rlubos rlubos merged commit 9b94ad9 into nrfconnect:main Apr 3, 2025
13 of 14 checks passed
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. manifest manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants