Skip to content

Trim old formula #22172

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

Closed
wants to merge 34 commits into from
Closed

Trim old formula #22172

wants to merge 34 commits into from

Conversation

e-rk
Copy link
Contributor

@e-rk e-rk commented May 6, 2025

No description provided.

tmon-nordic and others added 30 commits May 6, 2025 12:56
Run migrate_posix_kconfigs.py to rename deprecated POSIX API Kconfig
options.
GETOPT was replaced by POSIX_C_LIB_EXT which default to n. Remove the
obsolete default value change and use the replacement POSIX_C_LIB_EXT.

Signed-off-by: Tomasz Moń <[email protected]>
Signed-off-by: Johann Fischer <[email protected]>
Update version to upmerged sdk-zephyr and sdk-mcuboot.

Signed-off-by: Johann Fischer <[email protected]>
Update Matter SDK revision.

Signed-off-by: Adrian Gielniewski <[email protected]>
The combination of AF_PACKET and IPPROTO_RAW is not correct as packet
sockets should only be used with IEEE 802.3 protocol numbers. As the
support for this combination will likely be soon removed from Zephyr,
switch to use DGRAM packet socket instead.

Using ETH_P_ALL is fine in this case, as OpenThread interface only
provides IPv6 packets anyway.

Signed-off-by: Robert Lubos <[email protected]>
The combination of AF_PACKET and IPPROTO_RAW is not correct as packet
sockets should only be used with IEEE 802.3 protocol numbers. As the
support for this combination will likely be soon removed from Zephyr,
switch to use DGRAM packet socket instead.

As PPP L2 only supports IP/IPv6 packets, specify the L2 protocol type
field accordingly so that the L2 can interpret the packet data
correctly.

Signed-off-by: Robert Lubos <[email protected]>
The combination of AF_PACKET and IPPROTO_RAW is not correct as packet
sockets should only be used with IEEE 802.3 protocol numbers. As the
support for this combination will likely be soon removed from Zephyr,
switch to use DGRAM packet socket instead.

As PPP L2 only supports IP/IPv6 packets, specify the L2 protocol type
field accordingly so that the L2 can interpret the packet data
correctly.

Signed-off-by: Robert Lubos <[email protected]>
last_erased_page_start_offset has been removed by
zephyrproject-rtos/zephyr#82326.

Need to change accordingly to not break the build.

Signed-off-by: Maximilian Deubel <[email protected]>
This has been renamed upstream.

Signed-off-by: Timothy Keys <[email protected]>
BT_ISO_CHAN_TYPE_CONNECTED has been refactored into
BT_ISO_CHAN_TYPE_CENTRAL and BT_ISO_CHAN_TYPE_PERIPHERAL.

Signed-off-by: Timothy Keys <[email protected]>
Signature was changed to have const bt_conn upstream.

Signed-off-by: Timothy Keys <[email protected]>
This is needed after changes in upstream zephyr.

Signed-off-by: Timothy Keys <[email protected]>
Adjust for new net_linkaddr definition.

Signed-off-by: Maximilian Deubel <[email protected]>
Moved OpenThread-related Kconfigs from L2 layer to
modules/openthread. All of those configs were not strictly related
to the L2 layer, so they fits better to OpenThread module.
This operation allows using OpenThread Kconfigs even if L2 layer is
disabled.

Enabling NET_L2_OPENTHREAD also configures those configs by
selecting the OPENTHREAD kconfig, so there is no change regarding
backwards compatibility. The only change is that the Kconfigs
related to Thread were moved to the modules space, so their
location in the menuconfig also changed.

Once it is done, a choice for setting different L2 implementations
seems to be redundant.

Signed-off-by: Arkadiusz Balys <[email protected]>
Signed-off-by: Adrian Gielniewski <[email protected]>
Due to a change in the crypto library, the expected JWT signatures
have changed.

Signed-off-by: Maximilian Deubel <[email protected]>
The signature of bt_conn_get_remote_info changed upstream to use const conn *,
but in this usecase conn * should not be const.

Signed-off-by: Timothy Keys <[email protected]>
BT_GATT_CCC_INITIALIZER was renamed upstream. It is now called
BT_GATT_CCC_MANAGED_USER_DATA_INIT.

Signed-off-by: Timothy Keys <[email protected]>
Upstream refactor means it is no longer needed by default.

Signed-off-by: Timothy Keys <[email protected]>
CURL and IPERF3 depend on POSIX_THREADS not being selected but
POSIX_API selects POSIX_BASE_DEFINITIONS and subsequently POSIX_THREADS,
resulting in a dependency conflict.

There is no justification provided by the responsible team as to what
the exact problem was with  PTHREAD_IPC (now POSIX_THREADS) and whatever
it still is or is not.

Signed-off-by: Johann Fischer <[email protected]>
Removed use of deprecated CONFIG_UART_NRFX_UARTE_LEGACY_SHIM=n Kconfig
option. This option was earlier needed as a workaround, but it is no
longer required.

Signed-off-by: Tommi Kangas <[email protected]>
Removed use of deprecated CONFIG_UART_NRFX_UARTE_LEGACY_SHIM=n Kconfig
option. This option was earlier needed as a workaround, but it is no
longer required.

Signed-off-by: Tommi Kangas <[email protected]>
Fix pipe implementation. Adds a simple implementaation in the test
to avoid further issues.

Signed-off-by: Eivind Jølsgard <[email protected]>
Fix pipe implementation. Adds a simple implementaation in the test
to avoid further issues.

Signed-off-by: Eivind Jølsgard <[email protected]>
Due to a problem with the zephyr header, we need to include
another header to not have compiler warnings about enum mcumgr_err_t.

Signed-off-by: Maximilian Deubel <[email protected]>
Get rid of the -Wno- flags passed for compiling the Oberon PSA Crypto code.
The -Wno-stringop-* ones cause issues with the LLVM toolchain because
they are unknown to Clang.

Compiling without any of those -Wno- flags with both GCC and Clang seems to
work just fine without warning, so get rid of those warning deactivations.

Signed-off-by: Tomi Fontanilles <[email protected]>
Align with a change in upstream to samples names.

Signed-off-by: Maciej Perkowski <[email protected]>
Reduce log level and reduce CDC ACM buffer size
to avoid flash and ram overflow on thingy91/nrf52840 board.

Signed-off-by: Audun Korneliussen <[email protected]>
Property 'max-frequency' is going to be removed from
"nordic,nrf-timer" binding.

Signed-off-by: Sebastian Głąb <[email protected]>
The API to the Zbus add an observer has an added argument.

Signed-off-by: Graham Wacey <[email protected]>
Update the callback API after its change in Zephyr.

Signed-off-by: Pawel Dunaj <[email protected]>
pdunaj and others added 4 commits May 6, 2025 12:57
Ensure argument has proper size before being used.

Signed-off-by: Pawel Dunaj <[email protected]>
Fix test cases after the coap_client_option_initial_block2() was
moved from inline function to library C file.

Signed-off-by: Juha Ylinen <[email protected]>
Fixed mock generation to work with updated net_if.

Signed-off-by: Tommi Kangas <[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 6, 2025
Copy link

github-actions bot commented May 6, 2025

Since quarantine was modified, please make sure you are following the process described in Quarantine Process.

@NordicBuilder
Copy link
Contributor

CI Information

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

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

@NordicBuilder
Copy link
Contributor

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

Name Old Revision New Revision Diff
matter nrfconnect/sdk-connectedhomeip@73537dd (master) nrfconnect/sdk-connectedhomeip#603 nrfconnect/sdk-connectedhomeip#603/files
mcuboot nrfconnect/sdk-mcuboot@9d9d524 (main) nrfconnect/sdk-mcuboot#425 nrfconnect/sdk-mcuboot#425/files
zephyr nrfconnect/sdk-zephyr@893c333 (main) nrfconnect/sdk-zephyr#2840 nrfconnect/sdk-zephyr#2840/files

DNM label due to: 3 projects with PR revision

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. DNM manifest manifest-matter manifest-mcuboot manifest-zephyr
Projects
None yet
Development

Successfully merging this pull request may close these issues.