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.

Sign up for GitHub

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

Jump to bottom

doc: Updates to the LTE guide #21280

Merged
merged 2 commits into from
Apr 10, 2025
Merged

doc: Updates to the LTE guide #21280

merged 2 commits into from
Apr 10, 2025

Conversation

divipillai
Copy link
Contributor

@divipillai divipillai commented Apr 1, 2025 •
edited
Loading

Following changes are made to the LTE guide:

  • Renamed it to Cellular.
  • Expanded as subpages.
  • Added the PSM guide.

NCSDK-31323

@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Apr 1, 2025
@divipillai divipillai added CI-disable Disable CI for this PR and removed changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Apr 1, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Apr 1, 2025 •
edited
Loading

CI Information

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

Inputs:

Sources:

more details

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 (0)

Outputs:

Toolchain

Version:
Build docker image:

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

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests

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

@divipillai divipillai force-pushed the NCSDK-31323_psm_guide branch from 7d8c994 to 3c9e4a8 Compare April 1, 2025 14:38
@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 Apr 1, 2025
@divipillai divipillai removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 1, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 1, 2025 •
edited
Loading

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-21280/nrf/app_dev/device_guides/nrf91/index.html
https://ncsdoc.z6.web.core.windows.net/PR-21280/nrf/protocols.html
https://ncsdoc.z6.web.core.windows.net/PR-21280/nrf/protocols/lte/index.html
https://ncsdoc.z6.web.core.windows.net/PR-21280/nrf/protocols/lte/lte_ncs.html
https://ncsdoc.z6.web.core.windows.net/PR-21280/nrf/protocols/lte/overview.html
https://ncsdoc.z6.web.core.windows.net/PR-21280/nrf/protocols/lte/psm.html
https://ncsdoc.z6.web.core.windows.net/PR-21280/nrf/test_and_optimize/optimizing/power_nrf91.html

@divipillai divipillai force-pushed the NCSDK-31323_psm_guide branch from 3c9e4a8 to 5bbce0b Compare April 1, 2025 15:09
@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 Apr 1, 2025
@divipillai divipillai requested a review from simensrostad April 1, 2025 15:17
@divipillai divipillai removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 1, 2025
@divipillai divipillai force-pushed the NCSDK-31323_psm_guide branch from 5bbce0b to 2aa7003 Compare April 2, 2025 11:38
@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 Apr 2, 2025
@divipillai divipillai removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 2, 2025
@divipillai divipillai marked this pull request as ready for review April 2, 2025 11:39
@divipillai divipillai requested review from a team as code owners April 2, 2025 11:39
@divipillai divipillai force-pushed the NCSDK-31323_psm_guide branch from 2aa7003 to 3eb4e18 Compare April 4, 2025 11:46
@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 Apr 4, 2025
@Damian-Nordic Damian-Nordic mentioned this pull request Apr 4, 2025
Draft
@divipillai divipillai added DNM backport v3.0-branch and removed changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Apr 4, 2025
simensrostad
simensrostad reviewed Apr 7, 2025
View reviewed changes
doc/nrf/protocols/lte/overview.rst
SIM card support
================

SIMs that are used with the nRF91 Series devices must support LTE-M, NB-IoT, or both.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we add something about esim, mff2 here. cc @jtguggedal @martiles

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Could cover in a separate task. Created Jira NCSDK-32935.

doc/nrf/protocols/lte/psm.rst

The following tasks are performed in the RRC connected mode:

* Sending uplink (UL) messages from the UE to the eNB.
Copy link
Contributor

Choose a reason for hiding this comment

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

TAU (Tracking Area Update) is missing, @tokangas correct me if im wrong. Looks like it has been wrongly places under the RRC idle section.

Copy link
Contributor

Choose a reason for hiding this comment

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

@divipillai This is a nice reference diagram:
image

Copy link
Contributor

Choose a reason for hiding this comment

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

Nordic created figures that are already created can be found here: https://academy.nordicsemi.com/courses/cellular-iot-fundamentals/lessons/lesson-1-cellular-fundamentals/topic/lesson-1-power-saving-techniques/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The PSM and eDRX figures in the above Devacademy course are already covered in this guide. Only the combined figure of PSM and eDRX is missing.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, @simensrostad is right, TAU is performed in RRC connected mode.

@divipillai divipillai force-pushed the NCSDK-31323_psm_guide branch from 3eb4e18 to 5030662 Compare April 8, 2025 16:04
@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 Apr 8, 2025
@divipillai divipillai removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 8, 2025
@divipillai divipillai force-pushed the NCSDK-31323_psm_guide branch from 5030662 to 611317d Compare April 8, 2025 16:26
@divipillai divipillai requested a review from a team as a code owner April 8, 2025 16:26
@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 Apr 8, 2025
umapraseeda
umapraseeda reviewed Apr 9, 2025
View reviewed changes
tokangas
tokangas reviewed Apr 9, 2025
View reviewed changes
@divipillai
doc: Updates to the LTE guide
d4a4432 
Following changes are made to the LTE guide:

* Renamed it to Cellular.
* Expanded as subpages.
* Added the PSM guide.

Signed-off-by: divya pillai <divya.pillai@nordicsemi.no>
@divipillai
doc: Doc review updates
0aa3b20 
Doc review updates.

Signed-off-by: divya pillai <divya.pillai@nordicsemi.no>
@divipillai divipillai force-pushed the NCSDK-31323_psm_guide branch from 611317d to 0aa3b20 Compare April 9, 2025 15:00
@divipillai divipillai removed the DNM label Apr 10, 2025
@rlubos rlubos merged commit 28bd02f into nrfconnect:main Apr 10, 2025
13 of 14 checks passed
@NordicBuilder NordicBuilder mentioned this pull request Apr 10, 2025
Merged
@divipillai divipillai mentioned this pull request Apr 10, 2025
Draft
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tokangas tokangas tokangas approved these changes

@martiles martiles martiles approved these changes

@simensrostad simensrostad simensrostad approved these changes

@umapraseeda umapraseeda umapraseeda approved these changes

@jtguggedal jtguggedal Awaiting requested review from jtguggedal

@trantanen trantanen Awaiting requested review from trantanen

Assignees
No one assigned
Labels
backport v3.0-branch changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. CI-disable Disable CI for this PR doc only doc-required PR must not be merged without tech writer approval.
Projects
None yet
Milestone
3.0.0
Development

Successfully merging this pull request may close these issues.
7 participants
@divipillai @NordicBuilder @tokangas @martiles @simensrostad @umapraseeda @rlubos