lib: lte_link_control: Send NEIGHBOR_CELL_MEAS event in case of error

[tokangas]

Earlier, no LTE_LC_EVT_NEIGHBOR_CELL_MEAS event was sent in case an error occurred during parsing of the %NCELLMEAS notification. Because of this, the user of the library did not have any way to determine when the operation had been completed. The implementation has been changed so, that an LTE_LC_EVT_NEIGHBOR_CELL_MEAS event is always sent, if lte_lc_neighbor_cell_measurement() has returned 0. In case of an error, an empty LTE_LC_EVT_NEIGHBOR_CELL_MEAS (current_cell set to LTE_LC_CELL_EUTRAN_ID_INVALID) is sent.

Removed Kconfig links from the library header. The reason behind the change is that Breathe is no longer used when generating the documentation, and without Breathe the links do not work and only cause the generated documentation to look bad.

[NordicBuilder]

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 7

Inputs:

Sources:

sdk-nrf: PR head: 1ca47de6a7507d9f2eb97cb02fd9dca0555a789b

more details

sdk-nrf:

PR head: 1ca47de6a7507d9f2eb97cb02fd9dca0555a789b
merge base: b65f69a952107a821e23c0a222a1d2545db8b437
target head (main): b65f69a952107a821e23c0a222a1d2545db8b437
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 (4)

doc
│  ├── nrf
│  │  ├── releases_and_maturity
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst
include
│  ├── modem
│  │  │ lte_lc.h
lib
│  ├── lte_link_control
│  │  ├── modules
│  │  │  │ ncellmeas.c
tests
│  ├── lib
│  │  ├── lte_lc_api
│  │  │  ├── src
│  │  │  │  │ lte_lc_api_test.c

Outputs:

Toolchain

Version: 11349092be
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:11349092be_912848a074

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

• ◻️ Toolchain - Skipped: existing toolchain is used
• ✅ Build twister
  • sdk-nrf test count: 367
• ✅ Integration tests
  • ✅ test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
  • ✅ test-fw-nrfconnect-nrf-iot_samples
  • ✅ test-fw-nrfconnect-nrf-iot_mosh
  • ✅ test-fw-nrfconnect-nrf-iot_positioning
  • ⚠️ test-fw-nrfconnect-fw-update
  • ⚠️ test-fw-nrfconnect-nrf-iot_cloud
  • ⚠️ test-fw-nrfconnect-nrf-iot_lwm2m

Disabled integration tests

• • desktop52_verification
  • doc-internal
  • test_ble_nrf_config
  • test-fw-nrfconnect-apps
  • 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_libmodem-nrf
  • test-fw-nrfconnect-nrf-iot_nrf_provisioning
  • test-fw-nrfconnect-nrf-iot_serial_lte_modem
  • test-fw-nrfconnect-nrf-iot_thingy91
  • 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
  • test-secdom-samples-public

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

[NordicBuilder]

You can find the documentation preview for this PR at this link.

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

[trantanen]

Great improvement!

[tokangas]

@peknis Added a new commit to remove the Kconfig links as discussed, please have a look.