-
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
modules: add OpenThread platform implementation #20757
modules: add OpenThread platform implementation #20757
Conversation
Copy OpenThread platform implementation from upstream Zephyr, based on commit 10eb60b48111d529ab30bae2a344226accb3ded6 to enable development of Nordic-specific platform implementation. Signed-off-by: Maciej Baczmanski <maciej.baczmanski@nordicsemi.no>
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: b4fd08670e97444caf35f68e4ff19c395eab6cc3 more detailssdk-nrf:
Github labels
List of changed files detected by CI (27)
Outputs:ToolchainVersion: acee3b0b2b Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
You can find the documentation preview for this PR here. |
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.
Please take care of compliance.
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.
LGTM
444af19
to
77efbdb
Compare
@nrfconnect/ncs-co-build-system please review |
Commands are parsed by OpenThread CLI. There is always a null terminator added https://github.com/openthread/openthread/blob/main/src/core/utils/parse_cmdline.cpp#L66 This fail is a false positive |
77efbdb
to
8cab50a
Compare
8cab50a
to
6735a6e
Compare
- Start using NCS implementation of OT platform - align license allowlist - remove sourcing files that have not been copied from upstream - update CODEOWNERS - clean compliance issues - update test spec Signed-off-by: Maciej Baczmanski <maciej.baczmanski@nordicsemi.no>
6735a6e
to
b4fd086
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.
LGTM
@nrfconnect/ncs-co-build-system PR is ready for merge. please review. |
Copy OpenThread platform implementation from upstream Zephyr, based on commit 10eb60b48111d529ab30bae2a344226accb3ded6 to enable development of Nordic-specific platform implementation.