bluetooth: host: downstream bluetooth/buf.h API change #19412

PavelVPV · 2024-12-11T09:03:55Z

This PR downstreams zephyrproject-rtos/zephyr#81646 PR and the following PRs as dependencies:

BT_BUF_ACL_RX_COUNT is removed in samples as deprecated in zephyrproject-rtos/zephyr#81747

test-sdk-nrf: DRGN-23511_remove_bt_buf_acl_rx_count

NordicBuilder · 2024-12-11T09:05:02Z

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

Name	Old Revision	New Revision	Diff
sidewalk	nrfconnect/sdk-sidewalk@`1a811c3`	nrfconnect/sdk-sidewalk@`f274ce8`	nrfconnect/[email protected]
zephyr	nrfconnect/sdk-zephyr@`ba1d06f`	nrfconnect/sdk-zephyr@`7f8d857` (`main`)	nrfconnect/[email protected]

✅ All manifest checks OK

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

NordicBuilder · 2024-12-11T09:10:47Z

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 23

Inputs:

Sources:

sidewalk: PR head: f274ce8ab97d1504de4c6b372bb98d4dc41bacc5
sdk-nrf: PR head: dc92ed9c38255e6600f45561e8f6b3966480322c
zephyr: PR head: 7f8d857bf1f57b07e832d52263beaa23c91fef24

more details

sidewalk:

PR head: f274ce8ab97d1504de4c6b372bb98d4dc41bacc5
merge base: 1a811c356150a4ccc58d644d110524c33342e3a1
Diff

sdk-nrf:

PR head: dc92ed9c38255e6600f45561e8f6b3966480322c
merge base: a8f2725972edcde86068c20577db77b05887881d
target head (main): a8f2725972edcde86068c20577db77b05887881d
Diff

zephyr:

PR head: 7f8d857bf1f57b07e832d52263beaa23c91fef24
merge base: ba1d06f7241c1a899b43bdef7b731fa244646a2f
Diff

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 (198)

applications
│  ├── matter_bridge
│  │  ├── Kconfig
│  │  ├── boards
│  │  │  │ nrf5340dk_nrf5340_cpuapp.conf
│  │  ├── overlay-bt_max_connections_app.conf
│  │  ├── sysbuild
│  │  │  ├── ipc_radio
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf5340dk_nrf5340_cpunet.conf
│  │  │  │  ├── overlay-bt_max_connections_net.conf
│  │  │  │  │ prj.conf
samples
│  ├── bluetooth
│  │  ├── peripheral_with_multiple_identities
│  │  │  │ prj.conf
│  │  ├── scanning_while_connecting
│  │  │  │ prj.conf
sidewalk
│  ├── .github
│  │  ├── workflows
│  │  │  ├── compare_config.yml
│  │  │  ├── on-commit.yml
│  │  │  ├── on-pr.yml
│  │  │  ├── on-pr_dnm.yml
│  │  │  ├── on-pr_labeler.yml
│  │  │  ├── on-pr_nrf_manifest_update_PR.yml
│  │  │  ├── on-pr_publish_comment.yml
│  │  │  ├── on-push-tag.yml
│  │  │  ├── on_docker_change.yml
│  │  │  ├── publish_documentation.yml
│  │  │  ├── run_tests.yml
│  │  │  ├── run_tests_dut.yml
│  │  │  ├── samples_build.yml
│  │  │  │ validate_code_style.yml
│  ├── doc
│  │  ├── additional_resources.rst
│  │  ├── compatibility_matrix.rst
│  │  ├── links.rst
│  │  │ testing.rst
│  ├── samples
│  │  ├── sid_end_device
│  │  │  ├── Kconfig.defconfig
│  │  │  │ Kconfig.sysbuild
│  ├── subsys
│  │  ├── sal
│  │  │  ├── sid_pal
│  │  │  │  ├── src
│  │  │  │  │  │ CMakeLists.txt
│  ├── test_config.yaml
│  ├── tests
│  │  ├── functional
│  │  │  ├── critical_region
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── crypto
│  │  │  │  │ prj.conf
│  │  │  ├── mfg_storage
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── spi_bus
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf54l15dk_nrf54l15_cpuapp_ns.conf
│  │  │  │  │ testcase.yaml
│  │  │  ├── storage
│  │  │  │  ├── Kconfig
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf54l15dk_nrf54l15_cpuapp_ns.overlay
│  │  │  │  │ testcase.yaml
│  │  │  ├── temperature
│  │  │  │  │ testcase.yaml
│  │  │  ├── time
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  │ prj.conf
│  │  ├── integration
│  │  │  ├── crypto
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── Kconfig.defconfig
│  │  │  │  ├── boards
│  │  │  │  │  ├── native_sim.conf
│  │  │  │  │  │ nrf5340dk_nrf5340_cpuapp_ns.conf
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── mfg
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── boards
│  │  │  │  │  ├── nrf54l15dk_nrf54l10_cpuapp.overlay
│  │  │  │  │  ├── nrf54l15dk_nrf54l15_cpuapp.overlay
│  │  │  │  │  │ nrf54l15dk_nrf54l15_cpuapp_ns.overlay
│  │  │  │  ├── pm_static.yml
│  │  │  │  ├── pm_static_nrf54l15dk_nrf54l10_cpuapp.yml
│  │  │  │  ├── pm_static_nrf54l15dk_nrf54l15_cpuapp.yml
│  │  │  │  ├── pm_static_nrf54l15dk_nrf54l15_cpuapp_ns.yml
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── spi
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── boards
│  │  │  │  │  ├── nrf52840dk_nrf52840.overlay
│  │  │  │  │  ├── nrf5340dk_nrf5340_cpuapp.overlay
│  │  │  │  │  ├── nrf54l15dk_nrf54l10_cpuapp.overlay
│  │  │  │  │  ├── nrf54l15dk_nrf54l15_cpuapp.overlay
│  │  │  │  │  ├── nrf54l15dk_nrf54l15_cpuapp_ns.conf
│  │  │  │  │  │ nrf54l15dk_nrf54l15_cpuapp_ns.overlay
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── storage
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  ├── manual
│  │  │  ├── simple_bootloader
│  │  │  │  │ Kconfig.sysbuild
│  │  ├── unit_tests
│  │  │  ├── mfg_parsers
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf54l15dk_nrf54l10_cpuapp.overlay
│  │  │  │  │ testcase.yaml
│  │  │  ├── pal_assert
│  │  │  │  │ testcase.yaml
│  │  │  ├── pal_ble_adapter
│  │  │  │  │ testcase.yaml
│  │  │  ├── pal_crypto
│  │  │  │  │ testcase.yaml
│  │  │  ├── pal_log
│  │  │  │  │ testcase.yaml
│  │  │  ├── pal_mfg_storage
│  │  │  │  │ testcase.yaml
│  │  │  ├── pal_storage_kv
│  │  │  │  │ testcase.yaml
│  │  │  ├── pal_sw_interrupts
│  │  │  │  │ testcase.yaml
│  │  │  ├── pal_temperature
│  │  │  │  │ testcase.yaml
│  │  │  ├── pal_timer
│  │  │  │  │ testcase.yaml
│  │  │  ├── pal_uptime
│  │  │  │  │ testcase.yaml
│  │  │  ├── scratch_buffer
│  │  │  │  │ testcase.yaml
│  │  │  ├── sid_ace_alloc
│  │  │  │  │ testcase.yaml
│  │  │  ├── sid_ble_adapter_callbacks
│  │  │  │  │ testcase.yaml
│  │  │  ├── sid_ble_advert
│  │  │  │  │ testcase.yaml
│  │  │  ├── sid_ble_connection
│  │  │  │  │ testcase.yaml
│  │  │  ├── sid_ble_service
│  │  │  │  │ testcase.yaml
│  │  │  ├── sid_dut_shell
│  │  │  │  │ testcase.yaml
│  │  │  ├── sid_pal_gpio
│  │  │  │  │ testcase.yaml
│  │  │  ├── state_notifier
│  │  │  │  │ testcase.yaml
│  │  │  ├── tlv
│  │  │  │  │ testcase.yaml
│  │  ├── validation
│  │  │  ├── crypto
│  │  │  │  │ testcase.yaml
│  │  │  ├── storage_kv
│  │  │  │  │ testcase.yaml
│  │  │  ├── timer
│  │  │  │  │ testcase.yaml
│  ├── tests_new
│  │  ├── integration
│  │  │  ├── assert
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── critical_region
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── crypto_keys
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── Kconfig.defconfig
│  │  │  │  ├── boards
│  │  │  │  │  ├── nrf52840dk_nrf52840.conf
│  │  │  │  │  │ nrf5340dk_nrf5340_cpuapp_ns.conf
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── delay
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── boards
│  │  │  │  │  │ native_sim.conf
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── interrupts
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── mfg
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  ├── temperature
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── timer
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  ├── main.c
│  │  │  │  │  │ sid_timer_stub.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── uptime
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
west.yml
zephyr
│  ├── doc
│  │  ├── releases
│  │  │  ├── migration-guide-4.1.rst
│  │  │  │ release-notes-4.1.rst
│  ├── include
│  │  ├── zephyr
│  │  │  ├── bluetooth
│  │  │  │  ├── buf.h
│  │  │  │  ├── l2cap.h
│  │  │  │  │ testing.h
│  ├── samples
│  │  ├── bluetooth
│  │  │  ├── central_hr
│  │  │  │  │ prj_minimal.conf
│  │  │  ├── central_multilink
│  │  │  │  │ prj.conf
│  │  │  ├── hci_uart_async
│  │  │  │  │ prj.conf
│  │  │  ├── mesh
│  │  │  │  ├── boards
│  │  │  │  │  │ bbc_microbit.conf
│  │  │  ├── peripheral_hr
│  │  │  │  │ prj_minimal.conf
│  │  │  ├── peripheral_identity
│  │  │  │  │ prj.conf
│  ├── subsys
│  │  ├── bluetooth
│  │  │  ├── common
│  │  │  │  ├── Kconfig
│  │  │  │  │ dummy.c
│  │  │  ├── host
│  │  │  │  ├── buf.c
│  │  │  │  ├── classic
│  │  │  │  │  │ rfcomm.c
│  │  │  │  ├── hci_core.c
│  │  │  │  ├── hci_raw.c
│  │  │  │  ├── iso.c
│  │  │  │  ├── iso_internal.h
│  │  │  │  │ l2cap.c
│  ├── tests
│  │  ├── bluetooth
│  │  │  ├── buf
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml
│  │  │  ├── common
│  │  │  │  ├── testlib
│  │  │  │  │  ├── include
│  │  │  │  │  │  ├── testlib
│  │  │  │  │  │  │  │ addr.h
│  │  │  ├── tester
│  │  │  │  ├── boards
│  │  │  │  │  ├── frdm_rw612.conf
│  │  │  │  │  │ rd_rw612_bga.conf
│  │  ├── bsim
│  │  │  ├── bluetooth
│  │  │  │  ├── host
│  │  │  │  │  ├── l2cap
│  │  │  │  │  │  ├── compile.sh
│  │  │  │  │  │  ├── einprogress
│  │  │  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  │  │  ├── compile.sh
│  │  │  │  │  │  │  ├── prj.conf
│  │  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  │  ├── data.h
│  │  │  │  │  │  │  │  ├── dut.c
│  │  │  │  │  │  │  │  ├── main.c
│  │  │  │  │  │  │  │  │ tester.c
│  │  │  │  │  │  │  ├── test_scripts
│  │  │  │  │  │  │  │  │ run.sh
│  │  │  │  │  │  ├── reassembly
│  │  │  │  │  │  │  ├── dut
│  │  │  │  │  │  │  │  │ prj.conf
│  │  │  │  │  │  ├── stress
│  │  │  │  │  │  │  ├── prj.conf
│  │  │  │  │  │  │  ├── prj_nofrag.conf
│  │  │  │  │  │  │  │ prj_syswq.conf
│  │  │  │  │  ├── misc
│  │  │  │  │  │  ├── conn_stress
│  │  │  │  │  │  │  ├── central
│  │  │  │  │  │  │  │  │ prj.conf
│  │  │  │  │  │  ├── hfc_multilink
│  │  │  │  │  │  │  ├── dut
│  │  │  │  │  │  │  │  ├── prj.conf
│  │  │  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  │  │  │ dut.c
│  │  │  │  ├── ll
│  │  │  │  │  ├── multiple_id
│  │  │  │  │  │  │ prj.conf

Outputs:

Toolchain

Version: b77d8c1312
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:b77d8c1312_912848a074

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

◻️ Toolchain - Skipped: existing toolchain is used
✅ Build twister
- sdk-nrf test count: 958
- sdk-zephyr test count: 6418
✅ Integration tests
- ✅ test-sdk-audio
- ✅ desktop52_verification
- ✅ test_ble_nrf_config
- ✅ test-fw-nrfconnect-ble_mesh
- ✅ test-fw-nrfconnect-ble_samples
- ✅ test-fw-nrfconnect-chip
- ✅ test-fw-nrfconnect-nfc
- ✅ test-fw-nrfconnect-nrf-iot_thingy91
- ✅ test-sdk-find-my
- ✅ test-sdk-sidewalk
- ✅ test-low-level
- ✅ test-sdk-dfu

Disabled integration tests

- test-fw-nrfconnect-apps
- test-fw-nrfconnect-boot
- test-fw-nrfconnect-fem
- test-fw-nrfconnect-nrf-iot_libmodem-nrf
- test-fw-nrfconnect-nrf-iot_lwm2m
- test-fw-nrfconnect-nrf-iot_mosh
- test-fw-nrfconnect-nrf-iot_nrf_provisioning
- test-fw-nrfconnect-nrf-iot_positioning
- test-fw-nrfconnect-nrf-iot_samples
- test-fw-nrfconnect-nrf-iot_serial_lte_modem
- test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
- test-fw-nrfconnect-nrf_crypto
- test-fw-nrfconnect-ps
- test-fw-nrfconnect-rpc
- test-fw-nrfconnect-rs
- test-fw-nrfconnect-tfm
- test-fw-nrfconnect-thread
- test-fw-nrfconnect-zigbee
- test-sdk-mcuboot
- test-sdk-pmic-samples
- test-sdk-wifi
- test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

NordicBuilder · 2024-12-11T09:21:31Z

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publish GitHub Action.

PavelVPV · 2024-12-17T07:39:38Z

@nrfconnect/ncs-co-build-system , @nrfconnect/ncs-dragoon , @nrfconnect/ncs-code-owners , @nrfconnect/ncs-matter please review.

Update sdk-zephyr revision to pull in nrfconnect/sdk-zephyr#2353 Signed-off-by: Pavel Vasilyev <[email protected]>

BT_BUF_ACL_RX has been deprecated in zephyrproject-rtos/zephyr#81747 It is now by default equal to BT_MAX_CONN + 1. Signed-off-by: Pavel Vasilyev <[email protected]>

Pull in CONFIG_BT_BUF_ACL_RX_COUNT change in sdk-sidewalk. Signed-off-by: Pavel Vasilyev <[email protected]>

github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Dec 11, 2024

NordicBuilder added manifest-zephyr DNM labels Dec 11, 2024

PavelVPV mentioned this pull request Dec 12, 2024

ble: Remove CONFIG_BT_BUF_ACL_RX_COUNT nrfconnect/sdk-sidewalk#664

Merged

5 tasks

NordicBuilder added the manifest-sidewalk label Dec 12, 2024

PavelVPV force-pushed the hci_driver_async_buf_get_downstream branch from 22649f9 to 469d1e5 Compare December 13, 2024 10:56

PavelVPV marked this pull request as ready for review December 16, 2024 08:48

PavelVPV requested review from a team as code owners December 16, 2024 08:48

rugeGerritsen approved these changes Dec 17, 2024

View reviewed changes

nordicjm approved these changes Dec 17, 2024

View reviewed changes

alwa-nordic approved these changes Dec 17, 2024

View reviewed changes

Damian-Nordic approved these changes Dec 18, 2024

View reviewed changes

PavelVPV force-pushed the hci_driver_async_buf_get_downstream branch from 469d1e5 to ac307d6 Compare December 20, 2024 11:13

NordicBuilder removed the DNM label Dec 20, 2024

PavelVPV force-pushed the hci_driver_async_buf_get_downstream branch from ac307d6 to 046103f Compare December 20, 2024 19:42

rlubos force-pushed the hci_driver_async_buf_get_downstream branch from 046103f to ebcb71c Compare January 2, 2025 10:57

ktaborowski mentioned this pull request Jan 7, 2025

Revert "ble: Remove CONFIG_BT_BUF_ACL_RX_COUNT" nrfconnect/sdk-sidewalk#670

Merged

PavelVPV added 3 commits January 7, 2025 11:19

manifest: Update sdk-zephyr revision

aa5aa6b

Update sdk-zephyr revision to pull in nrfconnect/sdk-zephyr#2353 Signed-off-by: Pavel Vasilyev <[email protected]>

samples: Remove BT_BUF_ACL_RX_COUNT

34913ec

BT_BUF_ACL_RX has been deprecated in zephyrproject-rtos/zephyr#81747 It is now by default equal to BT_MAX_CONN + 1. Signed-off-by: Pavel Vasilyev <[email protected]>

manifest: Update sidewalk revision

dc92ed9

Pull in CONFIG_BT_BUF_ACL_RX_COUNT change in sdk-sidewalk. Signed-off-by: Pavel Vasilyev <[email protected]>

PavelVPV force-pushed the hci_driver_async_buf_get_downstream branch from ebcb71c to dc92ed9 Compare January 7, 2025 10:21

rlubos merged commit 4cb4cd1 into nrfconnect:main Jan 7, 2025
13 checks passed

PavelVPV mentioned this pull request Jan 7, 2025

Revert "Revert "ble: Remove CONFIG_BT_BUF_ACL_RX_COUNT"" nrfconnect/sdk-sidewalk#671

Merged

5 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

bluetooth: host: downstream bluetooth/buf.h API change #19412

bluetooth: host: downstream bluetooth/buf.h API change #19412

PavelVPV commented Dec 11, 2024 •

edited

Loading

NordicBuilder commented Dec 11, 2024 •

edited

Loading

NordicBuilder commented Dec 11, 2024 •

edited

Loading

NordicBuilder commented Dec 11, 2024

PavelVPV commented Dec 17, 2024

bluetooth: host: downstream bluetooth/buf.h API change #19412

bluetooth: host: downstream bluetooth/buf.h API change #19412

Conversation

PavelVPV commented Dec 11, 2024 • edited Loading

NordicBuilder commented Dec 11, 2024 • edited Loading

NordicBuilder commented Dec 11, 2024 • edited Loading

CI Information

Inputs:

Sources:

Github labels

Outputs:

Toolchain

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

NordicBuilder commented Dec 11, 2024

PavelVPV commented Dec 17, 2024

PavelVPV commented Dec 11, 2024 •

edited

Loading

NordicBuilder commented Dec 11, 2024 •

edited

Loading

NordicBuilder commented Dec 11, 2024 •

edited

Loading