Skip to content

openthread: Align module with Zephyr changes #22574

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 6 commits into from
Closed

openthread: Align module with Zephyr changes #22574

wants to merge 6 commits into from

Conversation

adigie
Copy link
Member

@adigie adigie commented May 30, 2025

  • Synchronize openthread platform with Zephyr
  • Add header for OpenThread module
  • Remove usage of deprecated API
  • Add possibility to initialize OpenThread in POST_KERNEL

ArekBalysNordic and others added 4 commits May 30, 2025 10:42
@ArekBalysNordic @adigie
modules: openthread: Synchronize openthread platform with Zephyr
3e04a4a 
Updated modules/openthread/platform after latest changes in
Zephyr.

Aligned openthread implementation to the new approach.

Signed-off-by: Arkadiusz Balys <[email protected]>
@ArekBalysNordic @adigie
modules: openthread: change log level to PLATFORM
3c0a54d 
Stop using L2 log level in platform files.

Signed-off-by: Arkadiusz Balys <[email protected]>
@ArekBalysNordic @adigie
openthread: Add possibility to initialize OpenThread in POST_KERNEL
4750b87 
The new CONFIG_OPENTHREAD_SYS_INIT Kconfig option allows OpenThread
to be automatically initialised during the Zephyr POST_KERNEL
initialisation stage.

If Zephyr's L2 layer OpenThread implementation is enabled, the
IEEE802.15.4 shim layer initialises OpenThread in the POST_KERNEL
phase. However, since Openthread may work without Zephyr's L2
layer, in this case, no object can initialise it automatically.
This new Kconfig option may help start OpenThread automatically
if the L2 Layer is disabled.

Signed-off-by: Arkadiusz Balys <[email protected]>
Signed-off-by: Adrian Gielniewski <[email protected]>
@adigie
openthread: Add header for OpenThread module
231e996 
* Add header with public API of OpenThread module.
* Add module include directory to Zephyr include directories.
* Use new header in platform implementation.

Signed-off-by: Adrian Gielniewski <[email protected]>
@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels May 30, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 30, 2025 •
edited
Loading

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

Name Old Revision New Revision Diff

✅ All manifest checks OK

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 30, 2025 •
edited
Loading

CI Information

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

Inputs:

Sources:

more details

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 (0)

Outputs:

Toolchain

Version:
Build docker image:

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

  • ❌ Toolchain
  • ❌ Build twister
  • ❌ Integration tests

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

adigie added 2 commits May 30, 2025 12:31
@adigie
openthread: Remove usage of deprecated API
bb5382b 
Use new API from OpenThread module header.

Signed-off-by: Adrian Gielniewski <[email protected]>
@adigie
openthread: Add Kconfigs for packet TX time and carrier functions
70dda37 
Add new Kconfig options to enable platform-specific features in
OpenThread:

- CONFIG_OPENTHREAD_PLATFORM_PKT_TXTIME enables support for packet TX
time when CONFIG_NET_PKT_TXTIME is selected.
- CONFIG_OPENTHREAD_PLATFORM_CARRIER_FUNCTIONS enables modulated and
continuous carrier functions when CONFIG_OPENTHREAD_DIAG and
CONFIG_IEEE802154_CARRIER_FUNCTIONS are enabled.

Update preprocessor conditionals to use the new options instead of
relying directly on CONFIG_NET_PKT_TXTIME and
CONFIG_IEEE802154_CARRIER_FUNCTIONS.

This change improves configurability and allows to reuse the platform
implementation when not using Zephyr networking.

Upstream PR #: 90737

Signed-off-by: Adrian Gielniewski <[email protected]>
@adigie adigie force-pushed the ot-module-header branch from 59ea7b2 to 5d5b56d Compare May 30, 2025 10:37
@github-actions github-actions bot removed the manifest label Jun 2, 2025
@adigie adigie closed this Jun 17, 2025
@adigie adigie deleted the ot-module-header branch June 17, 2025 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@adigie @NordicBuilder @ArekBalysNordic