Skip to content

Conversation

e-rk
Copy link
Contributor

@e-rk e-rk commented Sep 19, 2025

No description provided.

@e-rk e-rk requested review from a team as code owners September 19, 2025 08:43
@NordicBuilder NordicBuilder added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Sep 19, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Sep 19, 2025

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@d792280 nrfconnect/sdk-zephyr@9d15425 (main) nrfconnect/[email protected]

All manifest checks OK

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Sep 19, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 50aed3ba8ee4775f0ca8798dc7fa1807bc7ae983
zephyr: PR head: 9d15425c577ccc4e261fe68e29b8273ce5d468bf

more details

sdk-nrf:

PR head: 50aed3ba8ee4775f0ca8798dc7fa1807bc7ae983
merge base: 930585144e8ecf29be36cca5513823509810947c
target head (main): 930585144e8ecf29be36cca5513823509810947c
Diff

zephyr:

PR head: 9d15425c577ccc4e261fe68e29b8273ce5d468bf
merge base: d792280a1c8639ac103ce0dd094f8d5dd4e5377d
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 (5)
modules
│  ├── openthread
│  │  ├── platform
│  │  │  │ radio_nrf5.c
samples
│  ├── peripheral
│  │  ├── 802154_phy_test
│  │  │  ├── src
│  │  │  │  │ rf_proc.c
west.yml
zephyr
│  ├── drivers
│  │  ├── i2c
│  │  │  │ i2c_nrfx_twis.c
│  │  ├── ieee802154
│  │  │  │ ieee802154_nrf5.c

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
    • sdk-nrf test count: 7
    • sdk-zephyr test count: 1746
  • ✅ Integration tests
    • ✅ test-sdk-audio
    • ✅ test-fw-nrfconnect-thread-main
    • ✅ test-low-level
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_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • 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-sdk-dfu
    • test-sdk-find-my
    • 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

Copy link

You can find the documentation preview for this PR here.

Copy link
Contributor

@ArekBalysNordic ArekBalysNordic left a comment

Choose a reason for hiding this comment

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

It's ok for now as a workaround, but please create a task for migrating all affected functions to return an error code instead of a bool. Or we can leave here, but add some asserts that check the error code.

Update sdk-zephyr to add nrf-802154 API migration code.

Signed-off-by: Rafał Kuźnia <[email protected]>
The transmit functions will return an error code, instead of a boolean
value. To prepare for this, the API calls are temporarily implemented in
two variants, for old and new API declarations.

The presence of new API will be detected by the use of
NRF_802154_TX_FUNCTIONS_RETURN_ERROR_CODE macro, which will be
unconditionally defined by a newer nrf-802154 driver.

Signed-off-by: Rafał Kuźnia <[email protected]>
The transmit functions will return an error code, instead of a boolean
value. To prepare for this, the API calls are temporarily implemented in
two variants, for old and new API declarations.

The presence of new API will be detected by the use of
NRF_802154_TX_FUNCTIONS_RETURN_ERROR_CODE macro, which will be
unconditionally defined by a newer nrf-802154 driver.

Signed-off-by: Rafał Kuźnia <[email protected]>
@rlubos rlubos merged commit 395afce into nrfconnect:main Sep 24, 2025
14 of 15 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