-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
lib: lte_link_control: Skip parsing of unexpected %NCELLMEAS notif #19724
lib: lte_link_control: Skip parsing of unexpected %NCELLMEAS notif #19724
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: f2e274eaa747046982196c8242d5ce79f99187c9 more detailssdk-nrf:
Github labels
List of changed files detected by CI (4)
Outputs:ToolchainVersion: b77d8c1312 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
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 Publish GitHub Action. |
03f286e
to
8232700
Compare
Added check to skip %NCELLMEAS notification parsing when neighbor cell measurement has not been requested by calling lte_lc_neighbor_cell_measurement(). If the neighbor cell measurement is triggered by sending the AT%NCELLMEAS AT command to the modem, LTE LC does not know how the received notification should be parsed, sometimes leading to a parsing error. Signed-off-by: Tommi Kangas <tommi.kangas@nordicsemi.no>
Added missing time units in structs lte_lc_cell and lte_lc_ncell. Signed-off-by: Tommi Kangas <tommi.kangas@nordicsemi.no>
8232700
to
f2e274e
Compare
Added check to skip
%NCELLMEAS
notification parsing when neighbor cell measurement has not been requested by callinglte_lc_neighbor_cell_measurement()
. If the neighbor cell measurement is triggered by sending theAT%NCELLMEAS
AT command to the modem, LTE LC does not know how the received notification should be parsed, sometimes leading to a parsing error.Added missing time units in structs
lte_lc_cell
andlte_lc_ncell
.