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

bump Zephyr to tip of main (post 4.0.0-rc3) #667

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Conversation

mniestroj
Copy link
Collaborator

Try newest Zephyr to catch any potential problems.

Copy link

github-actions bot commented Oct 31, 2024

Visit the preview URL for this PR (updated for commit 31de9cf):

https://golioth-firmware-sdk-doxygen-dev--pr667-bump-zephyr-ct4pseq8.web.app

(expires Thu, 21 Nov 2024 21:31:59 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a9993e61697a3983f3479e468bcb0b616f9a0578

Copy link

github-actions bot commented Oct 31, 2024

Code Coverage

Code Coverage

Package Line Rate Branch Rate Health
include.golioth 75% 50%
port.linux 53% 28%
port.zephyr 51% 20%
src 69% 31%
Summary 67% (2489 / 3728) 30% (1057 / 3487)

Copy link

codecov bot commented Oct 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

see 5 files with indirect coverage changes

@mniestroj mniestroj force-pushed the bump-zephyr branch 2 times, most recently from 922a83b to df5a75d Compare November 2, 2024 14:57
@mniestroj mniestroj changed the title bump Zephyr to tip of main (post 4.0.0-rc1) bump Zephyr to tip of main (post 4.0.0-rc3) Nov 13, 2024
mniestroj and others added 6 commits November 14, 2024 14:12
Try newest Zephyr to catch any potential problems.

In new Zephyr there is a DTLS timeout respected based on
CONFIG_NET_SOCKETS_CONNECT_TIMEOUT option (previously this was hardcoded to
K_FOREVER). Default value of 3s is too little for 64 MHz CPU on nRF52840
when using certificates. Tests show that DTLS handshake reliably takes
around 24s, so increase timeout to 30s.

Signed-off-by: Marcin Niestroj <m.niestroj@emb.dev>
Passive mode in ESP-AT driver was enabled as a workaround to enable
CONFIG_AT_CIPDINFO_USE and allow receive data with recvfrom() API in DNS
resolver.

Since there is already proper support for assigning remote address (through
CONFIG_AT_CIPDINFO_USE) with active mode, switch back to that.

Signed-off-by: Marcin Niestroj <m.niestroj@emb.dev>
Drop deprecated Kconfig options, which are now in sync with upstream
Zephyr.

Signed-off-by: Marcin Niestroj <m.niestroj@emb.dev>
Signed-off-by: Marcin Niestroj <m.niestroj@emb.dev>
The default runner for the mimxrt1024_evk board has change to `linkserver`,
but we flash the on-board debugger with JLink firmware so we need to use
the `jlink` runner for Twister tests.

Signed-off-by: Sam Friedman <sam@golioth.io>
Zephyr v4.0.0 introduced a separate heap for ESP32. This heap has
minimum size requirements that we cannot satisfy, so intead we disable
the heap and force everything onto the system heap.

Signed-off-by: Sam Friedman <sam@golioth.io>
When using sysbuild, cmake variables passed on the command line are
not automatically available, but we first need to retrieve them with
the `zephyr_get()` CMake function. This also works for non-sysbuild
builds, so we use it unconditionally.

Signed-off-by: Sam Friedman <sam@golioth.io>
Enable the p384 curve for the 9160dk in the HIL tests. This was missed
when the same change was made for the samples.

Signed-off-by: Sam Friedman <sam@golioth.io>
The nrf9160DK now builds with sysbuild by default. We don't need
sysbuild for the HIL tests, so we disable it.

Signed-off-by: Sam Friedman <sam@golioth.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants