Skip to content

tests: drivers: lpuart: Fix two chip test - #28430

Merged
rlubos merged 1 commit into
nrfconnect:mainfrom
nordic-krch:lpuart_test_fix
May 26, 2026
Merged

tests: drivers: lpuart: Fix two chip test#28430
rlubos merged 1 commit into
nrfconnect:mainfrom
nordic-krch:lpuart_test_fix

Conversation

@nordic-krch

Copy link
Copy Markdown
Contributor

Test which run on nrf9160 and nrf52480 was failing as uart driver is now releasing UART pins when in idle. It means that TX pin should have pull-up as otherwise it is floating and can lead to errors in the communication. Test was failing due to that.

Added pull-ups to TX pins in pinctrl sleep state.
Enforced pinctrl sleep state to be present (event when PM device is not used).
Added detection of premature test completion which could lead to false positive. Test is now failing if too few packets are received.

Test which run on nrf9160 and nrf52480 was failing as uart driver
is now releasing UART pins when in idle. It means that TX pin should
have pull-up as otherwise it is floating and can lead to errors in
the communication. Test was failing due to that.

Added pull-ups to TX pins in pinctrl sleep state.
Enforced pinctrl sleep state to be present (event when PM device
is not used).
Added detection of premature test completion which could lead to
false positive. Test is now failing if too few packets are received.

Signed-off-by: Krzysztof Chruściński <krzysztof.chruscinski@nordicsemi.no>
@nordic-krch
nordic-krch requested a review from a team as a code owner April 28, 2026 11:05
@NordicBuilder

NordicBuilder commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 23e0a61e2574431a0c4126af4cff09f45fc46289

more details

sdk-nrf:

PR head: 23e0a61e2574431a0c4126af4cff09f45fc46289
merge base: 0244691e3437bcf1546fed6e2613905cd4aca6e0
target head (main): fd7412f071c483c4108b04cfe49438d242049732
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
│  ├── drivers
│  │  ├── lpuart
│  │  │  ├── boards
│  │  │  │  ├── nrf9160dk_nrf52840.overlay
│  │  │  │  │ nrf9160dk_nrf9160.overlay
│  │  │  ├── prj.conf
│  │  │  ├── src
│  │  │  │  │ main.c

Outputs:

Toolchain

Version: 53ac68dd62
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:53ac68dd62_5ea73affbf

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 9
  • ✅ Integration tests
    • ✅ 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_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-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-wifi

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

@nordic-krch nordic-krch added no-changelog bug backwards-compatible PR doesnt break code, Kconfig, or DTS. I.e no changes needed for customers to port their application labels Apr 29, 2026
@nordic-krch nordic-krch added this to the 3.4.0 milestone May 5, 2026
@rlubos
rlubos merged commit 8a6195e into nrfconnect:main May 26, 2026
24 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backwards-compatible PR doesnt break code, Kconfig, or DTS. I.e no changes needed for customers to port their application no-changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants