Skip to content

Conversation

e-rk
Copy link
Contributor

@e-rk e-rk commented Jun 10, 2025

The clock latency configuration for both nrf-802154 and MPSL is now done in DTS.

@e-rk e-rk requested review from a team as code owners June 10, 2025 14:01
@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Jun 10, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jun 10, 2025

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

Name Old Revision New Revision Diff
nrf-802154 https://github.com/nrfconnect/sdk-nrf-802154/commit/537640fbdd2fc023cc7910d3e59b7bd063c2e65b https://github.com/nrfconnect/sdk-nrf-802154/commit/ab926396fe37113664734181cc4dfbb53fb61863 nrfconnect/[email protected]
nrfxlib nrfconnect/sdk-nrfxlib@b6675b0 nrfconnect/sdk-nrfxlib@8f13312 (main) nrfconnect/[email protected]
zephyr nrfconnect/sdk-zephyr@bcf554e nrfconnect/sdk-zephyr@68ab1cf (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 Jun 10, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 80d256ceb74346862ab65b151dc9432b9a6d02d6
nrfxlib: PR head: 8f13312e5acea1ebc03cd368a1cd3f18026329f8
zephyr: PR head: 68ab1cf88c635a957aad4af56024469b13eb3b3d

more details

sdk-nrf:

PR head: 80d256ceb74346862ab65b151dc9432b9a6d02d6
merge base: 79addd913e5551a590d600680ece097d796c870f
target head (main): caff95f897cd01eb1eb70f83a57c2c98af74d0a3
Diff

nrfxlib:

PR head: 8f13312e5acea1ebc03cd368a1cd3f18026329f8
merge base: b6675b02789cf0c34d7bb9dd9c0694e6dc05bf77
Diff

zephyr:

PR head: 68ab1cf88c635a957aad4af56024469b13eb3b3d
merge base: bcf554e67c331a512c67f91b57dc6d797899e356
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 (60)
dts
│  ├── common
│  │  ├── nordic
│  │  │  ├── nrf54lm20a.dtsi
│  │  │  ├── nrf54lv10a.dtsi
│  │  │  │ nrf7120_enga.dtsi
nrfxlib
│  ├── nrf_802154
│  │  ├── doc
│  │  │  │ CHANGELOG.rst
│  │  ├── sl
│  │  │  ├── include
│  │  │  │  ├── nrf_802154_sl_utils.h
│  │  │  │  ├── platform
│  │  │  │  │  │ nrf_802154_clock.h
│  │  │  ├── sl
│  │  │  │  ├── lib
│  │  │  │  │  ├── nrf54l05_cpuapp
│  │  │  │  │  │  ├── hard-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── soft-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── softfp-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  ├── nrf54l10_cpuapp
│  │  │  │  │  │  ├── hard-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── soft-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── softfp-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  ├── nrf54l10_cpuapp_ns
│  │  │  │  │  │  ├── hard-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── soft-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── softfp-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  ├── nrf54l15_cpuapp
│  │  │  │  │  │  ├── hard-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── soft-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── softfp-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  ├── nrf54l15_cpuapp_ns
│  │  │  │  │  │  ├── hard-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── soft-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── softfp-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  ├── nrf54lm20a_cpuapp
│  │  │  │  │  │  ├── hard-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── soft-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── softfp-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  ├── nrf54lm20a_cpuapp_ns
│  │  │  │  │  │  ├── hard-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── soft-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── softfp-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  ├── nrf54lv10a_cpuapp
│  │  │  │  │  │  ├── hard-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── soft-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── softfp-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  ├── nrf54lv10a_cpuapp_ns
│  │  │  │  │  │  ├── hard-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── soft-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  │  │  │  ├── softfp-float
│  │  │  │  │  │  │  │ libnrf-802154-sl.a
│  │  │  ├── sl_opensource
│  │  │  │  ├── src
│  │  │  │  │  │ nrf_802154_sl_rsch.c
subsys
│  ├── mpsl
│  │  ├── init
│  │  │  │ Kconfig
west.yml
zephyr
│  ├── boards
│  │  ├── nordic
│  │  │  ├── nrf54l15dk
│  │  │  │  ├── Kconfig.defconfig
│  │  │  │  │ nrf54l15dk_common.dtsi
│  ├── drivers
│  │  ├── clock_control
│  │  │  ├── Kconfig.nrf
│  │  │  │ clock_control_nrf.c
│  ├── dts
│  │  ├── arm
│  │  │  ├── nordic
│  │  │  │  ├── nrf51822.dtsi
│  │  │  │  ├── nrf52805.dtsi
│  │  │  │  ├── nrf52810.dtsi
│  │  │  │  ├── nrf52811.dtsi
│  │  │  │  ├── nrf52820.dtsi
│  │  │  │  ├── nrf52832.dtsi
│  │  │  │  ├── nrf52833.dtsi
│  │  │  │  ├── nrf52840.dtsi
│  │  │  │  │ nrf5340_cpuapp_peripherals.dtsi
│  │  ├── bindings
│  │  │  ├── clock
│  │  │  │  ├── nordic,nrf51-hfxo.yaml
│  │  │  │  ├── nordic,nrf52-hfxo.yaml
│  │  │  │  ├── nordic,nrf53-hfxo.yaml
│  │  │  │  ├── nordic,nrf54l-hfxo.yaml
│  │  │  │  │ nordic,nrf54l-lfxo.yaml
│  │  ├── common
│  │  │  ├── nordic
│  │  │  │  ├── nrf54l09.dtsi
│  │  │  │  ├── nrf54l20.dtsi
│  │  │  │  │ nrf54l_05_10_15.dtsi
│  ├── include
│  │  ├── zephyr
│  │  │  ├── drivers
│  │  │  │  ├── clock_control
│  │  │  │  │  │ nrf_clock_control.h
│  ├── modules
│  │  ├── hal_nordic
│  │  │  ├── nrf_802154
│  │  │  │  ├── sl_opensource
│  │  │  │  │  ├── platform
│  │  │  │  │  │  │ nrf_802154_clock_zephyr.c
│  ├── tests
│  │  ├── kernel
│  │  │  ├── timer
│  │  │  │  ├── cycle64
│  │  │  │  │  │ testcase.yaml

Outputs:

Toolchain

Version: 4aa3467a6d
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:4aa3467a6d_776d264d2e

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: 6
  • ❌ Integration tests
    • ✅ test_ble_nrf_config - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-ble_samples - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-chip - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nfc - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nrf-iot_thingy91 - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nrf_crypto - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-rs - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-fem - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-tfm - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-thread-main - Skipped: Job was skipped as it succeeded in a previous run
    • ❌ test-sdk-find-my - Error: Error starting job: null
    • ✅ test-low-level - Skipped: Job was skipped as it succeeded in a previous run
    • ⚠️ test-sdk-dfu
Disabled integration tests
    • desktop52_verification
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-boot
    • 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-ps-main
    • test-fw-nrfconnect-rpc
    • test-sdk-audio
    • 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.

@e-rk e-rk force-pushed the auto-manifest-sdk-zephyr-2927 branch from 06d6e32 to 6cadd90 Compare June 10, 2025 14:28
@e-rk e-rk force-pushed the auto-manifest-sdk-zephyr-2927 branch from 29df69e to e4faaa6 Compare June 11, 2025 12:55
e-rk added 2 commits June 12, 2025 10:18
The clock latency configuration for both nrf-802154 and MPSL is now done
in DTS.

Signed-off-by: Rafał Kuźnia <[email protected]>
The node label hfxo is assumed to be a 32 MHz oscillator on previous SoC
families. To avoid inconsistency, the 64 MHz hfxo node was renamed to
hfxo64m.

Signed-off-by: Rafał Kuźnia <[email protected]>
@e-rk e-rk force-pushed the auto-manifest-sdk-zephyr-2927 branch from e4faaa6 to 80d256c Compare June 12, 2025 08:19
@NordicBuilder NordicBuilder removed the DNM label Jun 12, 2025
@rlubos
Copy link
Contributor

rlubos commented Jun 12, 2025

@nrfconnect/ncs-co-drivers @nrfconnect/ncs-co-build-system PTAL

int "HFCLK ramp-up latency in microsecond, assumed by MPSL to wait for the clock availability"
default 1650 if SOC_SERIES_NRF54LX
default $(dt_node_int_prop_int,/clocks/hfxo,startup-time-us) if $(dt_node_has_prop,/clocks/hfxo,startup-time-us)
default $(dt_node_int_prop_int,/oscillators/hfxo,startup-time-us) if $(dt_node_has_prop,/oscillators/hfxo,startup-time-us)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not use the hfxo node label, actually? I mean:

default $(dt_nodelabel_int_prop,hfxo,startup-time-us) if $(dt_nodelabel_has_prop,hfxo,startup-time-us)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't know that this existed. Could it be done in subsequent PR to unblock manifest updates for others?

@rlubos rlubos merged commit 45496f2 into nrfconnect:main Jun 12, 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-nrf-802154 manifest-nrfxlib manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants