Skip to content
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

Openthread updates #1445

Merged
merged 8 commits into from
Jan 18, 2024
Merged

Conversation

maciejbaczmanski
Copy link
Member

Pull openthread related commits from zephyr upstream

Copy link
Contributor

@rlubos rlubos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add sauce tags in the commit titles.

@maciejbaczmanski
Copy link
Member Author

maciejbaczmanski commented Jan 15, 2024

fyi: zephyrproject-rtos/zephyr@f32e686 adds new kconfig options for new bootloader reset feature. I'll create separate PR enabling this feature under KRKNWK-18026 (not to stress the manifest PR with a lot of cherry-picked commits).

@edmont
Copy link
Contributor

edmont commented Jan 17, 2024

Missing sauce tag in latest added commit.

@maciejbaczmanski
Copy link
Member Author

Missing sauce tag in latest added commit.

yeah, did it again... fixed now

@maciejbaczmanski
Copy link
Member Author

Twister still fails because of otLinkMetricsInit . Dropping commits:

zephyrproject-rtos/zephyr@e504ab5
zephyrproject-rtos/zephyr@fe9e306

As they are not crucial and are blocking this PR which is needed for other tasks. After proper fix is implemented, commits can be pulled in separate PR

maciejbaczmanski and others added 7 commits January 17, 2024 11:41
Regular OpenThread upmerge to commit `75694d2`.

Move CONFIG_OPENTHREAD_PLATFORM_KEY_REFERENCES_ENABLE
from header file to Kconfig.

Signed-off-by: Maciej Baczmanski <[email protected]>
(cherry picked from commit c2f1ff7)
…ists.

This commit introduces `kconfig_to_ot_option` to simply fye the way of
adding openthread related kconfigs.

Signed-off-by: Przemyslaw Bida <[email protected]>
(cherry picked from commit b6d8d27)
Running TCAT with PSA crypto API  causes stack overflow.
Increased stack size.

Signed-off-by: Maciej Baczmanski <[email protected]>
(cherry picked from commit 11613e0)
Added `OT_PLATFORM_POWER_CALIBRATION` and set to always off
as in Zephyr power calibration is handled by Radio Driver.

Signed-off-by: Maciej Baczmanski <[email protected]>
(cherry picked from commit d0283f9)
This commit implements `otPlatResetToBootloader` in two ways:

- trigger reset to bootloader using boot mode retention API
- trigger reset to bootloader by triggering GPIO pin (applicable
for nRF52840 Dongle)

Signed-off-by: Maciej Baczmanski <[email protected]>
(cherry picked from commit f32e686)
This commit adds new types of keys and algorithm to crypto_psa backend
of openthread.

Added options:
- `OT_CRYPTO_KEY_TYPE_ECDSA`
- `OT_CRYPTO_KEY_ALG_ECDSA`
- `OT_CRYPTO_KEY_USAGE_VERIFY_HASH`

Signed-off-by: Przemyslaw Bida <[email protected]>
(cherry picked from commit 0018204)
Remove two workarounds in OpenThread's PSA crypto backend
that were required when Zephyr used pre-1.5 TF-M version:
1. psa_open_key() is no longer needed to reference
   a persistent key
2. psa_cipher_encrypt() can be used to simplify AES
   encryption

Signed-off-by: Damian Krolik <[email protected]>
(cherry picked from commit 36b7a3e)
When OpenThread application is built with CONFIG_ASSERT
and CONFIG_ASSERT_NO_MSG_INFO, OT_ASSERT() prints a location
that points to the otPlatAssertFail() function instead of
the code that actually failed an assertion.

This is confusing and CONFIG_ASSERT_NO_MSG_INFO sometimes
cannot be disabled because of flash size limitations.

Make otPlatAssertFail() always print the actual assert
location.

Signed-off-by: Damian Krolik <[email protected]>
(cherry picked from commit 7b42e36)
@nordicjm nordicjm merged commit 5d7a30b into nrfconnect:main Jan 18, 2024
11 checks passed
@maciejbaczmanski maciejbaczmanski deleted the pull_ot_commits branch January 18, 2024 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants