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

Upmerge 2024.12.20 #19720

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

Upmerge 2024.12.20 #19720

wants to merge 22 commits into from

Conversation

anangl
Copy link
Contributor

@anangl anangl commented Jan 2, 2025

Pull in upmerged revisions of zephyr and mcuboot.

@anangl anangl requested a review from a team as a code owner January 2, 2025 07:11
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jan 2, 2025

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

Name Old Revision New Revision Diff
mcuboot nrfconnect/sdk-mcuboot@b82206c (main) nrfconnect/sdk-mcuboot#393 nrfconnect/sdk-mcuboot#393/files
zephyr nrfconnect/sdk-zephyr@2c7f6ef nrfconnect/sdk-zephyr#2411 nrfconnect/sdk-zephyr#2411/files

DNM label due to: 2 projects with PR revision

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

@gmarull
Copy link
Member

gmarull commented Jan 2, 2025

#19200 should be merged first (other repos need the same update, will open PRs)

@gmarull
Copy link
Member

gmarull commented Jan 2, 2025

3efae06 should be added here (I assume upstream BICR tooling is part of this upmerge)

@anangl anangl requested review from a team as code owners January 3, 2025 12:35
@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label Jan 3, 2025
@anangl anangl force-pushed the upmerge_20241220 branch 2 times, most recently from e829967 to 00df919 Compare January 3, 2025 16:43
@anangl anangl requested review from a team as code owners January 3, 2025 16:50
@anangl
Copy link
Contributor Author

anangl commented Jan 3, 2025

3efae06 should be added here (I assume upstream BICR tooling is part of this upmerge)

done

@thst-nordic thst-nordic self-requested a review January 6, 2025 13:27
@anangl anangl requested a review from a team as a code owner January 7, 2025 12:43
@anangl anangl force-pushed the upmerge_20241220 branch 3 times, most recently from f315d52 to 22726df Compare January 10, 2025 08:23
@anangl anangl requested review from a team as code owners January 10, 2025 08:23
@anangl anangl requested review from sachinthegreen, krish2718 and a team as code owners January 10, 2025 09:00
@anangl anangl force-pushed the upmerge_20241220 branch 3 times, most recently from 45aba6a to 164d42f Compare January 11, 2025 13:44
@anangl anangl requested a review from a team as a code owner January 11, 2025 13:44
anangl and others added 14 commits January 13, 2025 14:42
Add corrections for a bunch of sdk-zephyr revert commits that were
cherry-picked after rebase without updating their references to
reverted commits.
Remove a no longer needed correction for one sdk-trusted-firmware-m
commit.

The added corrections can be removed after the next sdk-zephyr rebase.

Signed-off-by: Andrzej Głąbek <[email protected]>
Pull in upmerged revisions of zephyr and mcuboot.

Signed-off-by: Andrzej Głąbek <[email protected]>
Adapt to the latest changes applied to Zephyr.

Signed-off-by: Andrzej Głąbek <[email protected]>
The bluetooth shell has been refactored (see
bf897cf941a514e7ea7a5f837d2287e360281f24), so the sourced file no longer
exists. Note that it was actually a bad use of sourcing, as the upstream
options are already part of the Kconfig tree (unless I missed
something).

Signed-off-by: Gerard Marull-Paretas <[email protected]>
- Upgrade to latest Sphinx
- Fix some references
- Fix Zephyr docset required env vars

Signed-off-by: Gerard Marull-Paretas <[email protected]>
Align with list of inputs in upstream Zephyr.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
According to the new extension parameters.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
Now the output will be $name/html|xml, so that we can generate
other outputs other than HTML only, e.g. XML.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
Instead of the removed CONFIG_NRF_REGTOOL_GENERATE_BICR.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
`bt-hci-bus` property values are now specified in lowercase and without
the `BT_HCI_BUS_` prefix.

Signed-off-by: Andrzej Głąbek <[email protected]>
The ESB_DYNAMIC_INTERRUPTS option enables code that uses direct dynamic
interrupts mechanism, so it should select the DYNAMIC_DIRECT_INTERRUPTS
option together with its dependency DIRECT_INTERRUPTS, not depend on
those, otherwise some unobvious dependency loops may appear (like one
with MBOX; through NRF_RPC, MPSL_CX_SOFTWARE_RPC, NRF_FEM_ONLY, and
MBOX_ANDES_PLIC_SW).

Signed-off-by: Andrzej Głąbek <[email protected]>
This dependency introduced as a fix for cmake warnings doesn't
seem to be needed anymore but it leads to a dependency loop with
BT_CTLR_ADV_EXT_SUPPORT.
This commit basically reverts 43c4519.

Signed-off-by: Andrzej Głąbek <[email protected]>
After the latest update of the "nordic,npm1300-charger" binding,
1000 mA is the maximum value for the dischg-limit-microamp property.

Signed-off-by: Andrzej Głąbek <[email protected]>
Remove entries for the following files that were recently deleted
from the hostap module:
- crypto_mbedtls-bignum.c
- crypto_mbedtls-ec.c
- crypto_mbedtls.c
- tls_mbedtls.c

Signed-off-by: Andrzej Głąbek <[email protected]>
As the BT_LL_CHOICE choice was removed upstream, BT_LL_SOFTDEVICE
needs to be a regular option now (and not depending on BT_CTLR
anymore). Also it should select HAS_BT_CTLR as it is supposed for
all LL implemenations. See Zephyr PR 82157.

Signed-off-by: Andrzej Głąbek <[email protected]>
The BT_LL_CHOICE choice was removed upstream and the BT_CTLR option was
replaced with HAS_BT_CTLR. See Zephyr PR 82157.

Signed-off-by: Andrzej Głąbek <[email protected]>
Bluetooth LL is now selected via devicetree so remove the obsolete
Kconfig stuff. See Zephyr PR 82157.

Signed-off-by: Andrzej Głąbek <[email protected]>
net_pkt_set_txtime() was deprecated for two Zephyr releases and has been
finally removed. net_pkt_set_timestamp_ns() is to be used instead.

Signed-off-by: Andrzej Głąbek <[email protected]>
Such configuration would lead to a recursive logging loop and a special
check catching it was introduced, so now it generates a build warning.
See Zephyr PR 78449.

Signed-off-by: Andrzej Głąbek <[email protected]>
…_uart

Follow the renaming applied in upstream Zephyr when common CDC ACM UART
configuration was introduced. See Zephyr PR 81308.

Signed-off-by: Andrzej Głąbek <[email protected]>
The BT_LL_SW_SPLIT option is now automatically enabled based on
the corresponding node ("zephyr,bt-hci-ll-sw-split" compatible)
being enabled in devicetree (and this is done by the bt-ll-sw-split
snippet).
The BT_CTLR option is deprecated in favor of HAS_BT_CTLR which is
selected accordingly by LL implementations.

Signed-off-by: Andrzej Głąbek <[email protected]>
@anangl anangl requested a review from a team as a code owner January 14, 2025 09:49
Add an option that will make NRF_SECURITY enabled when this is required
for builds involving BT.
This is a temporary solution, just to unblock building of many samples
and tests. It is supposed to be eventually replaced with something more
suitable.

Signed-off-by: Andrzej Głąbek <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DNM doc-required PR must not be merged without tech writer approval. manifest manifest-mcuboot manifest-zephyr
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants