-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
application: serial_lte_modem: new nRF Cloud service #11573
Conversation
Test specificationCI/Jenkins/NRF
CI/Jenkins/integration
Detailed information of selected test modules Note: This message is automatically posted and updated by the CI |
5c2dede
to
876cea0
Compare
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. |
876cea0
to
8b19916
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doc build fails on the missing link target for nRF Cloud AT commands.
8b19916
to
12dd8a9
Compare
12dd8a9
to
dcb10da
Compare
dcb10da
to
c5f2068
Compare
c5f2068
to
8151760
Compare
Separate nRF Cloud service from the GNSS service to allow include/exclude of nRF Cloud services by configurating prj.conf, and allow more nRF Cloud-specific services be added later. New configuration CONFIG_SLM_NRF_CLOUD for nRF Cloud access. nRF Cloud A-GPS, P-GPS and Location service to be configured separately with lib_nrfcloud configuration items: .Remove CONFIG_SLM_LOCATION, to use CONFIG_NRF_CLOUD_LOCATION. .Remove CONFIG_SLM_AGPS, to use CONFIG_NRF_CLOUD_AGPS. .Remove CONFIG_SLM_PGPS, to use CONFIG_NRF_CLOUD_PGPS. Enable nRF Cloud A-GPS and Location service by default. Signed-off-by: Jun Qing Zou <jun.qing.zou@nordicsemi.no>
@kacperradoszewski |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than the documentation build failing, this looks good.
8151760
to
29b8e8d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Carrier integration changes look good to me
@@ -0,0 +1,850 @@ | |||
/* | |||
* Copyright (c) 2021 Nordic Semiconductor ASA |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2023
@melwee01 Please help to review the docs on behalf of @wiba-nordic |
all required doc changes have been addressed
Separate nRF Cloud service from the GNSS service to allow
include/exclude of nRF Cloud services by configurating prj.conf,
and allow more nRF Cloud-specific services be added later.
New configuration CONFIG_SLM_NRF_CLOUD for nRF Cloud access.
nRF Cloud A-GPS, P-GPS and Location service to be configured seperately with lib_nrfcloud configuration items:
.Remove CONFIG_SLM_LOCATION, to use CONFIG_NRF_CLOUD_LOCATION.
.Remove CONFIG_SLM_AGPS, to use CONFIG_NRF_CLOUD_AGPS.
.Remove CONFIG_SLM_PGPS, to use CONFIG_NRF_CLOUD_PGPS.
Enable nRF Cloud A-GPS and Location service by default.