-
Notifications
You must be signed in to change notification settings - Fork 13
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
mniestroj
wants to merge
10
commits into
main
Choose a base branch
from
bump-zephyr
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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 |
mniestroj
force-pushed
the
bump-zephyr
branch
from
October 31, 2024 13:34
fea7f5c
to
2e1e980
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅ |
mniestroj
force-pushed
the
bump-zephyr
branch
2 times, most recently
from
November 2, 2024 14:57
922a83b
to
df5a75d
Compare
mniestroj
force-pushed
the
bump-zephyr
branch
from
November 13, 2024 21:20
df5a75d
to
002b862
Compare
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
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>
sam-golioth
force-pushed
the
bump-zephyr
branch
from
November 14, 2024 19:20
58b3076
to
3c94b03
Compare
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>
sam-golioth
force-pushed
the
bump-zephyr
branch
from
November 14, 2024 20:54
3c94b03
to
74c455c
Compare
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>
sam-golioth
force-pushed
the
bump-zephyr
branch
from
November 14, 2024 21:31
4517f7a
to
31de9cf
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Try newest Zephyr to catch any potential problems.