Skip to content

Conversation

@nordic-krch
Copy link
Contributor

timer_expired variable was missing volatile keyword which can lead to forever loop.

@nordic-krch nordic-krch requested a review from a team as a code owner March 5, 2025 10:59
@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 Mar 5, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 5, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 6c433fb209d5b030acda09fe50638e86a3514d9f
zephyr: PR head: 5715c89be1b39b4ecf6ec11e761be9c31c35f807

more details

sdk-nrf:

PR head: 6c433fb209d5b030acda09fe50638e86a3514d9f
merge base: e07457e1616d65c48716756602c60a5ffaa1d1bf
target head (main): e07457e1616d65c48716756602c60a5ffaa1d1bf
Diff

zephyr:

PR head: 5715c89be1b39b4ecf6ec11e761be9c31c35f807
merge base: 163004a98b1a59583840215e25f1d876d09d2df5
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 (4)
tests
│  ├── benchmarks
│  │  ├── multicore
│  │  │  ├── idle_stm
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf54h20dk_nrf54h20_cpuapp.overlay
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
west.yml
zephyr
│  ├── subsys
│  │  ├── logging
│  │  │  ├── frontends
│  │  │  │  │ log_frontend_stmesp.c

Outputs:

Toolchain

Version:
Build docker image:

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

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests
    • ◻️ test-fw-nrfconnect-nrf-iot_thingy91
    • ◻️ test-low-level
    • ◻️ test-sdk-dfu
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • 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_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-fw-nrfconnect-zigbee
    • test-sdk-audio
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi
    • test-secdom-samples-public

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 6, 2025

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@163004a nrfconnect/sdk-zephyr@5715c89 (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 Mar 6, 2025

You can find the documentation preview for this PR here.

timer_expired variable was missing volatile keyword
which can lead to forever loop.

Signed-off-by: Krzysztof Chruściński <[email protected]>
Enable runtime PM for UARTE in the devicetree instead of main.
Enabling in main() can lead to a failure since UART is used before
main() and using PM API puts device into suspend state while UART
is active.

Signed-off-by: Krzysztof Chruściński <[email protected]>
Update zephyr with fix in STM logging.

Signed-off-by: Krzysztof Chruściński <[email protected]>
@sonarqubecloud
Copy link

@rlubos rlubos merged commit b6ccec9 into nrfconnect:main Mar 10, 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.

7 participants