Skip to content

Conversation

MarkusLassila
Copy link
Contributor

@MarkusLassila MarkusLassila commented May 14, 2025

Take the hardware-flow-control to use in Serial LTE Modem.

Jira: LRCS-36

@MarkusLassila MarkusLassila requested review from a team as code owners May 14, 2025 12:05
@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 May 14, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 14, 2025

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

Name Old Revision New Revision Diff

All manifest checks OK

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 14, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 2be0fa04b128aab5cccbaca723a2297d2ef4f6ec

more details

sdk-nrf:

PR head: 2be0fa04b128aab5cccbaca723a2297d2ef4f6ec
merge base: ef2192145275a2e0ff86872146829cdfe92259ff
target head (main): ef2192145275a2e0ff86872146829cdfe92259ff
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 (1)
applications
│  ├── serial_lte_modem
│  │  │ overlay-ppp.conf

Outputs:

Toolchain

Version: 1d2c5d0ab5
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:1d2c5d0ab5_8bf7ca4353

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 7
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-nrf-iot_serial_lte_modem
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • 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_thingy91
    • 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-main
    • test-low-level
    • test-sdk-audio
    • 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

@MarkusLassila MarkusLassila force-pushed the manifest-zephyr-ppp-fix branch from b39efba to 5ecd7e2 Compare May 14, 2025 12:12
Copy link

You can find the documentation preview for this PR here.

@MarkusLassila MarkusLassila force-pushed the manifest-zephyr-ppp-fix branch from 5ecd7e2 to 02103e3 Compare May 14, 2025 13:10
@NordicBuilder NordicBuilder removed the DNM label May 14, 2025
Take the hardware-flow-control to use for PPP in Serial LTE Modem.

Signed-off-by: Markus Lassila <[email protected]>
@MarkusLassila MarkusLassila force-pushed the manifest-zephyr-ppp-fix branch from 02103e3 to 2be0fa0 Compare May 15, 2025 05:58
@github-actions github-actions bot removed the manifest label May 15, 2025
@MarkusLassila MarkusLassila changed the title manifest: Update sdk-zephyr revision application: serial_lte_modem: HWFC for PPP May 15, 2025
@MarkusLassila
Copy link
Contributor Author

West.yml manifest already updated beyond the needs of this PR.

@MarkusLassila MarkusLassila removed request for a team May 15, 2025 06:01
@rlubos rlubos merged commit 52e9bd0 into nrfconnect:main May 16, 2025
10 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants