Skip to content

Changes required for SDK 0.18 for testing using SDK 0.17 #93042

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

keith-packard
Copy link
Contributor

This is the collab-sdk-dev series rebased to a recent main revision and then adapted so that it "should" build using SDK 0.17.

Copy link

github-actions bot commented Jul 11, 2025

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

Name Old Revision New Revision Diff
hal_silabs zephyrproject-rtos/hal_silabs@190a144 zephyrproject-rtos/hal_silabs#123 zephyrproject-rtos/hal_silabs#123/files
mbedtls zephyrproject-rtos/mbedtls@85440ef (zephyr) zephyrproject-rtos/mbedtls#73 zephyrproject-rtos/mbedtls#73/files

DNM label due to: 2 projects with PR revision

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

@keith-packard keith-packard force-pushed the collab-sdk-dev branch 3 times, most recently from 2db5239 to 0ee55f3 Compare July 11, 2025 20:03
@keith-packard keith-packard force-pushed the collab-sdk-dev branch 3 times, most recently from 763b2da to d64055d Compare July 12, 2025 02:57
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Jul 12, 2025
@stephanosio stephanosio added this to the v4.3.0 milestone Jul 15, 2025
stephanosio
stephanosio previously approved these changes Jul 15, 2025
nordicjm
nordicjm previously approved these changes Jul 17, 2025
@stephanosio
Copy link
Member

@keith-packard This needs a rebase.

@nashif nashif assigned stephanosio and unassigned tejlmand Jul 21, 2025
@keith-packard keith-packard dismissed stale reviews from nordicjm and stephanosio via 3f2e3bf July 21, 2025 16:56
@keith-packard keith-packard force-pushed the collab-sdk-dev branch 2 times, most recently from 3f2e3bf to cdf95d7 Compare July 21, 2025 18:13
@zephyrbot zephyrbot added the platform: Silabs Silicon Labs label Jul 21, 2025
@keith-packard
Copy link
Contributor Author

This will now require review from someone at SiLabs, preferably the author of the board support for the affected dev_kit, Martin Hoff martin.hoff@silabs.com

@zephyrbot zephyrbot added the platform: Intel ADSP Intel Audio platforms label Jul 21, 2025
@github-actions github-actions bot added manifest manifest-mbedtls DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Jul 21, 2025
@Martinhoff-maker
Copy link
Contributor

FYI, the commit "boards/siwx917_dk2605a: Adjust code_partition address to fix build" is no more needed IMO since this PR has been reverted #93271. A new PR has been created to fix this issue: #93482.
My suggestion would be to rebase and remove this commit (everything should be ok with the CI).

@keith-packard
Copy link
Contributor Author

This will now require review from someone at SiLabs, preferably the author of the board support for the affected dev_kit, Martin Hoff martin.hoff@silabs.com

A better fix has been merged and the commit in this series dropped.

keith-packard and others added 6 commits July 27, 2025 16:39
Disable the default startup file with -nostartfiles.

Signed-off-by: Keith Packard <keithp@keithp.com>
Zephyr replaces the toolchain version of stdint.h to define uint32_t and
int32_t as int rather than long. This breaks the ARM MVE intrinics which
require uint32_t to be defined as unsigned long.

Define 'ZEPHYR_USE_TOOLCHAIN_STDINT_H_' as that causes zephyr_stdint.h to
be skipped, ensuring that the normal stdint.h types are used instead.

Signed-off-by: Keith Packard <keithp@keithp.com>
Many tests failing for RX, exclude those until the arch stablizes.

See zephyrproject-rtos#89839

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
When the target has only a single CPU, this function cannot ever
succeed. Skip all of the drama and just return -EINVAL. This makes GCC 14
happy as it doesn't get confused about possible out of bounds access of the
soc_cpus_active array.

Signed-off-by: Keith Packard <keithp@keithp.com>
This patch (submitted upstream) avoids an incorrect warning generated by
gcc 14.3 about array bounds. It should not change the generated code at
all.

Signed-off-by: Keith Packard <keithp@keithp.com>
Signed-off-by: Anas Nashif <anas.nashif@intel.com>
This pulls in a trivial type checking error which gcc 14.3 catches
between a function prototype and a definition where a parameter type
is declared as a enum by the prototype and uint8_t by the definition.

Signed-off-by: Keith Packard <keithp@keithp.com>
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: C Library C Standard Library area: C++ area: CMSIS-DSP area: X86 x86 Architecture (32-bit) DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hal_silabs manifest-mbedtls platform: Intel ADSP Intel Audio platforms platform: Silabs Silicon Labs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants