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

doc: samples: lwm2m_carrier: Add overlay and doc for SoftBank and LG U+ #18706

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

nordic-hani
Copy link
Contributor

Add overlay and improve the Kconfigs and documentation to calrify the extra configurations needed to successfully use SoftBank and/or LG U+.

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

NordicBuilder commented Nov 7, 2024

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 7dbb149cca3923ed19704ebde67c612fd030e38b

more details

sdk-nrf:

PR head: 7dbb149cca3923ed19704ebde67c612fd030e38b
merge base: 40120ba33acc04654fdf14329d2d0972ce15d1f3
target head (main): 423981109ef39b942e73e2b8e91c64692bc4d566
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 (13)
applications
│  ├── serial_lte_modem
│  │  ├── doc
│  │  │  │ slm_description.rst
│  │  ├── overlay-carrier-lgu.conf
│  │  ├── overlay-carrier-softbank.conf
│  │  ├── sample.yaml
│  │  │ sysbuild-softbank.conf
doc
│  ├── nrf
│  │  ├── libraries
│  │  │  ├── bin
│  │  │  │  ├── lwm2m_carrier
│  │  │  │  │  │ requirements.rst
lib
│  ├── bin
│  │  ├── lwm2m_carrier
│  │  │  │ Kconfig
samples
│  ├── cellular
│  │  ├── lwm2m_carrier
│  │  │  ├── overlay-lgu.conf
│  │  │  ├── overlay-softbank.conf
│  │  │  ├── sample.yaml
│  │  │  ├── sample_description.rst
│  │  │  ├── sysbuild-lgu.conf
│  │  │  │ sysbuild-softbank.conf

Outputs:

Toolchain

Version: b44b7a08c9
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:b44b7a08c9_912848a074

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 254
  • ❌ Integration tests
    • ❌ test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • ✅ test-fw-nrfconnect-nrf-iot_lwm2m
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • 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_mosh
    • test-fw-nrfconnect-nrf-iot_nrf_provisioning
    • 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-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi

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

Add overlay and improve the Kconfigs and documentation to calrify the
extra configurations needed to successfully use SoftBank and/or LG U+.

Signed-off-by: Håvard Vermeer <[email protected]>
@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.

@nordic-hani nordic-hani force-pushed the softbank_dependency branch 3 times, most recently from c1f08f5 to e5986c9 Compare November 14, 2024 10:15
* Add overlay and improve the Kconfigs and documentation to clarify the
  extra configurations needed to successfully use SoftBank and/or LG U+.
* Add missing entry for shell overlay in the sample.yaml.
* Improve requirement documentation to mention the specific setups.

Signed-off-by: Håvard Vermeer <[email protected]>
@nordic-hani nordic-hani marked this pull request as ready for review November 14, 2024 12:08
@nordic-hani nordic-hani requested review from a team as code owners November 14, 2024 12:08
@nordic-hani nordic-hani added this to the 2.9.0 milestone Nov 14, 2024
Comment on lines +312 to +319
See :ref:`slm_carrier_library_support` for more information on how to connect to an operator's device management platform.

* :file:`overlay-carrier-softbank.conf`;:file:`sysbuild-softbank.conf` - Configuration files that adds SoftBank configurations needed by the carrier libray.
Used in conjunction with :file:`overlay-carrier.conf`.
For more information see the :ref:`lwm2m_carrier_dependent` section of the :ref:`liblwm2m_carrier_readme` documentation.
* :file:`overlay-carrier-lgu.conf` - Configuration file that adds LG U+ configurations needed by the carrier libray.
Used in conjunction with :file:`overlay-carrier.conf`.
For more information see the :ref:`lwm2m_carrier_dependent` section of the :ref:`liblwm2m_carrier_readme` documentation.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
See :ref:`slm_carrier_library_support` for more information on how to connect to an operator's device management platform.
* :file:`overlay-carrier-softbank.conf`;:file:`sysbuild-softbank.conf` - Configuration files that adds SoftBank configurations needed by the carrier libray.
Used in conjunction with :file:`overlay-carrier.conf`.
For more information see the :ref:`lwm2m_carrier_dependent` section of the :ref:`liblwm2m_carrier_readme` documentation.
* :file:`overlay-carrier-lgu.conf` - Configuration file that adds LG U+ configurations needed by the carrier libray.
Used in conjunction with :file:`overlay-carrier.conf`.
For more information see the :ref:`lwm2m_carrier_dependent` section of the :ref:`liblwm2m_carrier_readme` documentation.
See :ref:`slm_carrier_library_support` for more information on how to connect to an operator's device management platform.
* :file:`overlay-carrier-softbank.conf` and :file:`sysbuild-softbank.conf` - Configuration files that add SoftBank configurations needed by the carrier library.
Used in conjunction with :file:`overlay-carrier.conf`.
For more information, see the :ref:`lwm2m_carrier_dependent` section of the :ref:`liblwm2m_carrier_readme` documentation.
* :file:`overlay-carrier-lgu.conf` - This configuration file adds LG U+ configurations needed by the carrier library.
Used in conjunction with :file:`overlay-carrier.conf`.
For more information, see the :ref:`lwm2m_carrier_dependent` section of the :ref:`liblwm2m_carrier_readme` documentation.

* If the application is using the TLS socket, it must immediately close it when the :c:macro:`LWM2M_CARRIER_EVENT_FOTA_START` event is received.
* The socket is released by the library again upon the next reboot, or in the event of a FOTA error.
* If the application always needs a TLS socket, it can use `Mbed TLS`_.

* When in the Verizon network, the LwM2M carrier library uses both the DTLS sessions made available through the modem.
Therefore, the application cannot run any DTLS or TLS sessions.

Copy link
Contributor

Choose a reason for hiding this comment

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

Remove extra empty lines.

Comment on lines +63 to +64
Carrier Dependent
*****************
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Carrier Dependent
*****************
Carrier dependency
******************

Carrier Dependent
*****************

In order for your device to comply with the carrier's specifications, the following additional requirements apply to your application to use the carrier's LwM2M device management:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
In order for your device to comply with the carrier's specifications, the following additional requirements apply to your application to use the carrier's LwM2M device management:
To ensure your device complies with the carrier's specifications, your application must meet the following additional requirements to use the carrier's LwM2M device management:


.. group-tab:: Verizon

* When in the Verizon network, the LwM2M carrier library uses both the DTLS sessions made available through the modem.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* When in the Verizon network, the LwM2M carrier library uses both the DTLS sessions made available through the modem.
* In the Verizon network, the :ref:`liblwm2m_carrier_readme` library uses both DTLS sessions made available through the modem.

Comment on lines +89 to +90
* If you are using the :ref:`serial_lte_modem` application, this should be set in the file :file:`slm_auto_connect.h, which is included with :kconfig:option:`CONFIG_SLM_AUTO_CONNECT`.
For more information, see :ref:`CONFIG_SLM_AUTO_CONNECT` in the :ref:`serial_lte_modem` documentation.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* If you are using the :ref:`serial_lte_modem` application, this should be set in the file :file:`slm_auto_connect.h, which is included with :kconfig:option:`CONFIG_SLM_AUTO_CONNECT`.
For more information, see :ref:`CONFIG_SLM_AUTO_CONNECT` in the :ref:`serial_lte_modem` documentation.
* If you are using the :ref:`serial_lte_modem` application, set the mode in the :file:`slm_auto_connect.h` file that is included with the :ref:`CONFIG_SLM_AUTO_CONNECT <CONFIG_SLM_AUTO_CONNECT>` Kconfig option.


* The application must support application FOTA.
The :ref:`serial_lte_modem` application and :ref:`lwm2m_carrier` sample contain extra config files to enable the :kconfig:option:`CONFIG_LWM2M_CARRIER_LG_UPLUS` dependencies.
* The appplication must set the LG U+ specific configurations listed in :c:struct:`lwm2m_carrier_lg_uplus_config_t`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* The appplication must set the LG U+ specific configurations listed in :c:struct:`lwm2m_carrier_lg_uplus_config_t`.
* The application must set the LG U+-specific configurations listed in the :c:struct:`lwm2m_carrier_lg_uplus_config_t` structure.

* The application must support application FOTA.
The :ref:`serial_lte_modem` application and :ref:`lwm2m_carrier` sample contain extra config files to enable the :kconfig:option:`CONFIG_LWM2M_CARRIER_LG_UPLUS` dependencies.
* The appplication must set the LG U+ specific configurations listed in :c:struct:`lwm2m_carrier_lg_uplus_config_t`.
If you are unsure about which values to supply during certification, reach out to your carrier or your local Nordic Sales representative.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
If you are unsure about which values to supply during certification, reach out to your carrier or your local Nordic Sales representative.
If you are unsure about which values to supply during certification, reach out to your carrier or your local Nordic Semiconductor sales representative.

@@ -101,6 +101,10 @@ The following files are available:

* :file:`prj.conf` - Standard default configuration file.
* :file:`overlay-shell.conf` - Enables the :ref:`lwm2m_carrier_shell` and :ref:`lib_at_shell`.
* :file:`overlay-softbank.conf`;:file:`sysbuild-softbank.conf` - Enables configurations for SoftBank.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* :file:`overlay-softbank.conf`;:file:`sysbuild-softbank.conf` - Enables configurations for SoftBank.
* :file:`overlay-softbank.conf` and :file:`sysbuild-softbank.conf` - Enable configurations for SoftBank.

@@ -101,6 +101,10 @@ The following files are available:

* :file:`prj.conf` - Standard default configuration file.
* :file:`overlay-shell.conf` - Enables the :ref:`lwm2m_carrier_shell` and :ref:`lib_at_shell`.
* :file:`overlay-softbank.conf`;:file:`sysbuild-softbank.conf` - Enables configurations for SoftBank.
For more information see the :ref:`lwm2m_carrier_dependent` section of the :ref:`liblwm2m_carrier_readme` documentation.
* :file:`overlay-lgu.conf`;:file:`sysbuild-lgu.conf` - Enables configurations for LG U+.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* :file:`overlay-lgu.conf`;:file:`sysbuild-lgu.conf` - Enables configurations for LG U+.
* :file:`overlay-lgu.conf` and :file:`sysbuild-lgu.conf` - Enable configurations for LG U+.

@peknis
Copy link
Contributor

peknis commented Nov 14, 2024

Might be worth a changelog entry.

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. doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants