Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

manifest: sdk-zephyr: apply modem cherry picks #14042

Closed

Conversation

tomi-font
Copy link
Contributor

To get the generic modem driver support for nRF91 series (w/ SLM).

@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 Feb 16, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Feb 16, 2024

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@a8dec43 nrfconnect/sdk-zephyr@c5c7fb3 (main) nrfconnect/[email protected]

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Feb 16, 2024

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
desktop52_verification X
test-fw-nrfconnect-ble X
test-fw-nrfconnect-ble_samples X
test-fw-nrfconnect-chip X
test-fw-nrfconnect-fem X
test-fw-nrfconnect-nfc X
test-fw-nrfconnect-nrf-iot_cloud X
test-fw-nrfconnect-nrf-iot_mosh X
test-fw-nrfconnect-nrf-iot_positioning X
test-fw-nrfconnect-nrf-iot_samples X
test-fw-nrfconnect-nrf-iot_thingy91 X
test-fw-nrfconnect-nrf-iot_zephyr_lwm2m X
test-fw-nrfconnect-nrf_crypto X
test-fw-nrfconnect-rpc X
test-fw-nrfconnect-rs X
test-fw-nrfconnect-thread X
test-fw-nrfconnect-zigbee X
test-sdk-audio X
test-sdk-find-my X
test-sdk-sidewalk X
test-sdk-wifi X

Detailed information of selected test modules

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

@SeppoTakalo SeppoTakalo added this to the 2.6.0 milestone Feb 16, 2024
@PerMac
Copy link
Contributor

PerMac commented Feb 16, 2024

@tomi-font I think you've picked a wrong PR in the manifest. Description here doesn't match the modified entry.

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@tomi-font tomi-font force-pushed the manifest_revision_update branch from 22de49a to a73549a Compare February 16, 2024 09:41
@tomi-font
Copy link
Contributor Author

@PerMac You are right. Thanks for the heads-up!

@tomi-font tomi-font force-pushed the manifest_revision_update branch 2 times, most recently from 487c5bf to 159aefb Compare February 16, 2024 13:03
@tomi-font tomi-font removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Feb 19, 2024
Copy link
Contributor

@MarkusLassila MarkusLassila left a comment

Choose a reason for hiding this comment

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

Needed for Serial LTE Modem.

west.yml Outdated
@@ -61,7 +61,7 @@ manifest:
# https://developer.nordicsemi.com/nRF_Connect_SDK/doc/latest/zephyr/guides/modules.html
- name: zephyr
repo-path: sdk-zephyr
revision: 962eb7320fdb76ffe265d8bab7330ad472cb6f28
revision: pull/1503/head
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's give it a chance.
revision: c5c7fb3967905dcaef1035ac8d5b234176a2cc0a
nrfconnect/sdk-zephyr@c5c7fb3

To get the generic modem driver support for nRF91 series (w/ SLM).

Signed-off-by: Tomi Fontanilles <[email protected]>
@tomi-font tomi-font force-pushed the manifest_revision_update branch from 159aefb to d057f14 Compare February 20, 2024 09:39
@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 Feb 20, 2024
@NordicBuilder NordicBuilder removed the DNM label Feb 20, 2024
@tomi-font tomi-font removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Feb 20, 2024
@jfischer-no
Copy link
Contributor

@tomi-font Sorry, #14006 is merged, zephyr-revision will be updated in #13768

@jfischer-no
Copy link
Contributor

#13768 is merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants