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

samples: fast_pair: input_device: Fix build failures on nRF5340 DK #25

Closed

Conversation

MarekPieta
Copy link

Change brings back hci_ipc configuration used for child images to fix build errors on nRF5340 DK.

Jira: NCSDK-27508

rlubos and others added 30 commits May 24, 2024 09:19
Pull upmerged sdk-zephyr and sdk-mcuboot repositories.
Pull fixes in matter and find-my, sidewalk, trusted-firmware-m.

Signed-off-by: Robert Lubos <[email protected]>
PORTING.md is now referenced by other docs, hence make sure it's visible
during doc build.

Signed-off-by: Robert Lubos <[email protected]>
Specifically NET_L2_BT and NET_L2_BT_SHELL.

Signed-off-by: Robert Lubos <[email protected]>
d590bcb removed the
CONFIG_BT_HCI_RAW_RESERVE Kconfig option. Align NCS to the change.

Jira: NCSDK-27508

Signed-off-by: Marek Pieta <[email protected]>
A new calblack event (ZPERF_SESSION_PERIODIC_RESULT) has been added to
zperf, just ignore it int the sample for now to prevent build warnings.

Signed-off-by: Robert Lubos <[email protected]>
lwm2m_engine_set_data_cb_t callback signaure now has an additional
offset parameter.

Signed-off-by: Robert Lubos <[email protected]>
NCS boot banner has to be standalone entity from Zephyr boot banner.

Signed-off-by: Robert Lubos <[email protected]>
MBEDTLS_PSA_CRYPTO_EXTERNAL_RNG symbol has been added as an choice
option in upstream Zephyr, which is not compatible with how nrf_security
interprets the symbol. Try to fix this on the nrf_security side.

Signed-off-by: Robert Lubos <[email protected]>
From the upstream migration guide:

The SMF_CREATE_STATE macro now always takes 5 arguments. The amount
of arguments is now independent of the values of
CONFIG_SMF_ANCESTOR_SUPPORT and CONFIG_SMF_INITIAL_TRANSITION.
If the additional arguments are not used, they have to be set to NULL

Signed-off-by: Robert Lubos <[email protected]>
Align code after 484fe3f was added to
Zephyr.

Jira: NCSDK-27508

Signed-off-by: Marek Pieta <[email protected]>
18c23da removes CONFIG_BT_HCI_VS_EXT.
Align BLE controller configuration with the changes.

Jira: NCSDK-27508

Signed-off-by: Marek Pieta <[email protected]>
Enabling custom fatal error handler globally can be problematic for
sdk-zephyr samples built in NCS context, as they may define their own
handlers, causing linker error. As there's no elegant way to handle that
on the sample side, other than noup patches in sdk-zephyr, disable the
RESET_ON_FATAL_ERROR config by default.

As an alternative for NCS, the config will now be implied by
NCS_SAMPLES_DEFAULTS config, which is exclusive for NCS samples only.

Additionally, enable CONFIG_REBOOT in a few samples, which rely on
reboot functionality but did not enable config explicitly.

Signed-off-by: Robert Lubos <[email protected]>
nRF Desktop application does not use boot banners provided by Zephyr
and NCS.

Jira: NCSDK-27508

Signed-off-by: Marek Pieta <[email protected]>
Having mbed TLS enabled in the system does not automatically mean it's
used for HTTPS communication in memfault. If offloaded sockets are used,
it's not the case regardless of mbed TLS configurations.

Override the default value for MEMFAULT_HTTP_USES_MBEDTLS config in case
NRF_MODEM_LIB is enabled to n. This config simply enables some extra
checks for mbed TLS configuration (whether it fits memfault needs), but
since it's not really used for offloaded sockets, it can generate
false errors. By default, the config is set to y whenever mbed TLS is
enabled, which is oversimplified condition for our case.

Signed-off-by: Robert Lubos <[email protected]>
Changed rram0 dts label name to the cpuapp_rram for nrf54l15pdk
board.

Signed-off-by: Kamil Kasperczyk <[email protected]>
Boot banner must be disabled for nRF2220 and nRF2240 front-end modules
to avoid conflicts between TWIM and UART instances on initialization.

Signed-off-by: Jędrzej Ciupis <[email protected]>
Upstream commit 4760aad changed CPU
idle handler from asm to C, so align with that change.

Signed-off-by: Robert Lubos <[email protected]>
Reduce WPA supplicant stack size to accommodate RAM requirements for
nRF52840.

Signed-off-by: Chaitanya Tata <[email protected]>
nRF54L15 has low RAM, so, till we optimize Wi-Fi RAM requirements,
temporarily quarantine this twister combination.

Signed-off-by: Chaitanya Tata <[email protected]>
Change aligns RRAM configuration for nRF54L15 PDK. The application does
not use cpuflpr core, so whole memory can be used by cpuapp.

Jira: NCSDK-27508

Signed-off-by: Marek Pieta <[email protected]>
OCT-3020
Resolved build issues

Signed-off-by: Kristoffer Rist Skøien <[email protected]>
See comments along the quarantine entries for reasoning.

Signed-off-by: Robert Lubos <[email protected]>
Despite increasing the timeout, sometimes we still see failures in
pushing a net management event, so, increase the queue size to fix such
scenarios.

Fixes SHEL-2773.

Signed-off-by: Chaitanya Tata <[email protected]>
After some reorganization this file is no longer in use.

Signed-off-by: Sebastian Bøe <[email protected]>
OCT-3020
Resolved BIS+ACL tester upmerge issues

Signed-off-by: Kristoffer Rist Skøien <[email protected]>
Pylint requires min version 3 to work

Signed-off-by: Maciej Perkowski <[email protected]>
Enable CONFIG_CRC, which is a dependency for mcumgr UART transport.

Signed-off-by: Robert Lubos <[email protected]>
Revision 0.3.0 is now the default revision of
nrf54l15pdk/nrf54l15/cpuapp board, hence no longer recognisable by
twister if provided explicitly. Update yaml files accordingly to reflect
that.

Signed-off-by: Robert Lubos <[email protected]>
The default watchdog instance for nrf54l15 is now wdt31.

Signed-off-by: Robert Lubos <[email protected]>
The default watchdog instance for nrf54l15 is now wdt31.

Signed-off-by: Robert Lubos <[email protected]>
krish2718 and others added 2 commits May 24, 2024 09:19
This needs a revised memory layout for Wi-Fi to easily extend RAM and
FLAH, so, on hold for now to unblock upmerge.

Fixes SHEL-2777.

Signed-off-by: Chaitanya Tata <[email protected]>
Change brings back hci_ipc configuration used for child images to fix
build errors on nRF5340 DK.

Jira: NCSDK-27508

Signed-off-by: Marek Pieta <[email protected]>
@rlubos rlubos force-pushed the upmerge-16-05-2024 branch from 70fe8c1 to f3c917f Compare May 24, 2024 13:18
@rlubos
Copy link
Owner

rlubos commented May 24, 2024

Cherry-picked into the upmrege branch

@rlubos rlubos closed this May 24, 2024
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.

9 participants