[DRAFT] modules: openthread: Implement radio_nrf5 OpenThread platform.

[ArekBalysNordic]

Introduce radio_nrf5 OpenThread radio platform which uses nRF radio driver directly. The Zephyr L2 layer, IEEE802.15.4 shim layer and net packets are not used in this implementation.

To build without Zephyr's L2 layer add:

-DCONFIG_OPENTHREAD=y -DCONFIG_NET_L2_OPENTHREAD=n -DCONFIG_NETWORKING=n

TODO:

• Prepare radio_nrf5.c file.
• Prepare Kconfig to switch between the legacy approach with Zephyr L2 and IEE802.15.4 shim layer and the new one.
• Implement receiving and transmitting through nRF radio driver.
• Implement threading to manage transmission.

Do not review while it is in draft.

[NordicBuilder]

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: bca4b6bf8c9219c5af8f4f32140c2a5417b195a4
zephyr: PR head: 0087a083d76101e611c0c156c212f7bb6b0e9f55

more details

sdk-nrf:

PR head: bca4b6bf8c9219c5af8f4f32140c2a5417b195a4
merge base: d0189a6959cded3ca35737a37359d2fde26b0fef
target head (main): cffb09ccf70792b3c32b4bd38c1e341aec2630c7
Diff

zephyr:

PR head: 0087a083d76101e611c0c156c212f7bb6b0e9f55
merge base: 6131774ed5598fde0d09eb266fec686c95e2f24b
target head (main): 9761288d1c61d676cf5e7501563c199db9007ae0
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 (53)

doc
│  ├── nrf
│  │  ├── protocols
│  │  │  ├── thread
│  │  │  │  │ configuring.rst
modules
│  ├── openthread
│  │  ├── CMakeLists.txt
│  │  ├── Kconfig
│  │  ├── Kconfig.nrf5
│  │  ├── include
│  │  │  │ openthread.h
│  │  ├── openthread.c
│  │  ├── platform
│  │  │  ├── CMakeLists.txt
│  │  │  ├── alarm.c
│  │  │  ├── ble.c
│  │  │  ├── diag.c
│  │  │  ├── entropy.c
│  │  │  ├── platform-zephyr.h
│  │  │  ├── radio.c
│  │  │  ├── radio_nrf5.c
│  │  │  │ settings.c
│  │  │ shell.c
samples
│  ├── openthread
│  │  ├── cli
│  │  │  ├── prj.conf
│  │  │  ├── snippets
│  │  │  │  ├── ci
│  │  │  │  │  │ ci.conf
│  │  │  │  ├── logging
│  │  │  │  │  │ logging.conf
│  │  │  ├── src
│  │  │  │  │ low_power.c
│  │  ├── coap_client
│  │  │  ├── src
│  │  │  │  │ coap_client_utils.c
│  │  ├── coap_server
│  │  │  ├── prj.conf
│  │  │  ├── snippets
│  │  │  │  ├── logging
│  │  │  │  │  │ logging.conf
│  │  │  ├── src
│  │  │  │  │ coap_server.c
│  │  ├── coprocessor
│  │  │  ├── prj.conf
│  │  │  ├── src
│  │  │  │  │ nrf_802154_radio_wrapper.c
│  ├── wifi
│  │  ├── thread_coex
│  │  │  ├── src
│  │  │  │  │ ot_utils.c
subsys
│  ├── caf
│  │  ├── modules
│  │  │  │ net_state_ot.c
│  ├── net
│  │  ├── openthread
│  │  │  ├── rpc
│  │  │  │  ├── common
│  │  │  │  │  │ ot_rpc_lock_net_l2.c
tests
│  ├── subsys
│  │  ├── net
│  │  │  ├── openthread
│  │  │  │  ├── rpc
│  │  │  │  │  ├── server
│  │  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  ├── instance_suite.c
│  │  │  │  │  │  │  │ mocks.c
west.yml
zephyr
│  ├── doc
│  │  │ zephyr.doxyfile.in
│  ├── include
│  │  ├── zephyr
│  │  │  ├── net
│  │  │  │  ├── net_if.h
│  │  │  │  │ openthread.h
│  ├── modules
│  │  ├── openthread
│  │  │  ├── CMakeLists.txt
│  │  │  ├── Kconfig
│  │  │  ├── include
│  │  │  │  │ openthread.h
│  │  │  ├── openthread.c
│  │  │  ├── platform
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── alarm.c
│  │  │  │  ├── ble.c
│  │  │  │  ├── diag.c
│  │  │  │  ├── entropy.c
│  │  │  │  ├── platform-zephyr.h
│  │  │  │  ├── radio.c
│  │  │  │  ├── radio_spinel.cpp
│  │  │  │  │ settings.c
│  │  │  │ shell.c
│  ├── subsys
│  │  ├── net
│  │  │  ├── l2
│  │  │  │  ├── openthread
│  │  │  │  │  ├── openthread.c
│  │  │  │  │  │ openthread_utils.c
│  ├── tests
│  │  ├── subsys
│  │  │  ├── openthread
│  │  │  │  │ radio_stub.c

Outputs:

Toolchain

Version: 4aa3467a6d
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:4aa3467a6d_e85602c25f

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

• ◻️ Toolchain - Skipped: existing toolchain is used
• ✅ Build twister
  • sdk-nrf test count: 16
  • sdk-zephyr test count: 1457
• ❌ Integration tests
  • ✅ desktop52_verification
  • ✅ test-fw-nrfconnect-apps
  • ✅ test-fw-nrfconnect-chip
  • ✅ test-fw-nrfconnect-nfc
  • ❌ test-fw-nrfconnect-nrf-iot_cloud
  • ✅ test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
  • ✅ test-fw-nrfconnect-nrf-iot_thingy91
  • ❌ test-fw-nrfconnect-rs
  • ❌ test-fw-nrfconnect-fem
  • ❌ test-fw-nrfconnect-thread-main
  • ✅ test-sdk-find-my
  • ✅ test-fw-nrfconnect-nrf-iot_mosh
  • ✅ test-fw-nrfconnect-nrf-iot_positioning
  • ❌ test-sdk-wifi
  • ✅ test-low-level
  • ✅ test-fw-nrfconnect-ps-main
  • ⚠️ test-sdk-dfu

Disabled integration tests

• • doc-internal
  • test_ble_nrf_config
  • test-fw-nrfconnect-ble_mesh
  • test-fw-nrfconnect-ble_samples
  • test-fw-nrfconnect-boot
  • test-fw-nrfconnect-nrf-iot_libmodem-nrf
  • test-fw-nrfconnect-nrf-iot_lwm2m
  • test-fw-nrfconnect-nrf-iot_samples
  • test-fw-nrfconnect-nrf-iot_serial_lte_modem
  • test-fw-nrfconnect-nrf_crypto
  • test-fw-nrfconnect-proprietary_esb
  • test-fw-nrfconnect-rpc
  • test-fw-nrfconnect-tfm
  • test-sdk-audio
  • test-sdk-mcuboot
  • test-sdk-pmic-samples
  • test-secdom-samples-public

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

[NordicBuilder]

The following west manifest projects have changed revision in this Pull Request:

Name	Old Revision	New Revision	Diff
zephyr	nrfconnect/sdk-zephyr@6131774	nrfconnect/sdk-zephyr#2792	nrfconnect/sdk-zephyr#2792/files

⛔ DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

[github-actions]

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-21882/nrf/protocols/thread/configuring.html

[LuDuda]

This file should be moved out of OpenThread platform, as it does not implement it. Would move it one level up.

[ArekBalysNordic]

It will be finished in a different PR.