Skip to content

Commit

Permalink
Bluetooth: Controller: Default to BT_CTLR_SCAN_AUX_USE_CHAINS
Browse files Browse the repository at this point in the history
BT_CTLR_SCAN_AUX_USE_CHAINS is alternative new design with
less RAM usage for supporting Extended Scanning of
simultaneous interleaved Extended Advertising chains.

Switch Extended Scanning to default to
BT_CTLR_SCAN_AUX_USE_CHAINS design in the Controller.

Deprecate use of BT_CTLR_SCAN_AUX_SET.

Signed-off-by: Vinayak Kariappa Chettimada <[email protected]>
  • Loading branch information
cvinayak committed Feb 13, 2025
1 parent c57e888 commit 290d098
Show file tree
Hide file tree
Showing 10 changed files with 42 additions and 24 deletions.
2 changes: 2 additions & 0 deletions doc/releases/release-notes-4.1.rst
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,8 @@ Deprecated APIs and options

* :kconfig:option:`CONFIG_BT_BUF_ACL_RX_COUNT`

* :kconfig:option:`CONFIG_BT_CTLR_SCAN_AUX_SET`

* All HWMv1 board name aliases which were added as deprecated in v3.7 are now removed
(:github:`82247`).

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,9 @@ CONFIG_BT_CTLR_ADV_SYNC_PDU_BACK2BACK=y
CONFIG_BT_CTLR_ADV_DATA_BUF_MAX=6

# Increase the below to receive interleaved advertising chains
CONFIG_BT_CTLR_SCAN_AUX_SET=1
# CONFIG_BT_CTLR_SCAN_AUX_SET=1 # DEPRECATED
CONFIG_BT_CTLR_SCAN_AUX_USE_CHAINS=y
CONFIG_BT_CTLR_SCAN_AUX_CHAIN_COUNT=3

CONFIG_BT_CTLR_ADV_RESERVE_MAX=n
CONFIG_BT_CTLR_SCAN_UNRESERVED=y
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,9 @@ CONFIG_BT_CTLR_ADV_AUX_PDU_BACK2BACK=y
CONFIG_BT_CTLR_ADV_DATA_BUF_MAX=6

# Increase the below to receive interleaved advertising chains
CONFIG_BT_CTLR_SCAN_AUX_SET=1
# CONFIG_BT_CTLR_SCAN_AUX_SET=1 # DEPRECATED
CONFIG_BT_CTLR_SCAN_AUX_USE_CHAINS=y
CONFIG_BT_CTLR_SCAN_AUX_CHAIN_COUNT=3

CONFIG_BT_CTLR_ADV_RESERVE_MAX=n
CONFIG_BT_CTLR_CENTRAL_RESERVE_MAX=n
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,9 @@ CONFIG_BT_CTLR_ADV_SYNC_PDU_BACK2BACK=y
CONFIG_BT_CTLR_ADV_DATA_BUF_MAX=6

# Increase the below to receive interleaved advertising chains
CONFIG_BT_CTLR_SCAN_AUX_SET=1
# CONFIG_BT_CTLR_SCAN_AUX_SET=1 # DEPRECATED
CONFIG_BT_CTLR_SCAN_AUX_USE_CHAINS=y
CONFIG_BT_CTLR_SCAN_AUX_CHAIN_COUNT=3

CONFIG_BT_CTLR_SCAN_UNRESERVED=y
CONFIG_BT_TICKER_NEXT_SLOT_GET_MATCH=y
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -79,9 +79,9 @@ CONFIG_BT_CTLR_ADV_SYNC_PDU_BACK2BACK=y
CONFIG_BT_CTLR_ADV_DATA_BUF_MAX=6

# Increase the below to receive interleaved advertising chains
CONFIG_BT_CTLR_SCAN_AUX_SET=1
# CONFIG_BT_CTLR_SCAN_AUX_USE_CHAINS=y
# CONFIG_BT_CTLR_SCAN_AUX_CHAIN_COUNT=1
# CONFIG_BT_CTLR_SCAN_AUX_SET=1 # DEPRECATED
CONFIG_BT_CTLR_SCAN_AUX_USE_CHAINS=y
CONFIG_BT_CTLR_SCAN_AUX_CHAIN_COUNT=3

CONFIG_BT_CTLR_ADV_RESERVE_MAX=n
CONFIG_BT_CTLR_ADV_ISO_RESERVE_MAX=y
Expand Down
6 changes: 3 additions & 3 deletions samples/bluetooth/hci_uart/overlay-all-bt_ll_sw_split.conf
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,9 @@ CONFIG_BT_CTLR_ADV_SYNC_PDU_BACK2BACK=y
CONFIG_BT_CTLR_ADV_DATA_BUF_MAX=6

# Increase the below to receive interleaved advertising chains
CONFIG_BT_CTLR_SCAN_AUX_SET=1
# CONFIG_BT_CTLR_SCAN_AUX_USE_CHAINS=y
# CONFIG_BT_CTLR_SCAN_AUX_CHAIN_COUNT=1
# CONFIG_BT_CTLR_SCAN_AUX_SET=1 # DEPRECATED
CONFIG_BT_CTLR_SCAN_AUX_USE_CHAINS=y
CONFIG_BT_CTLR_SCAN_AUX_CHAIN_COUNT=3

CONFIG_BT_CTLR_ADV_RESERVE_MAX=n
CONFIG_BT_CTLR_ADV_ISO_RESERVE_MAX=y
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -51,9 +51,9 @@ CONFIG_BT_CTLR_ADV_SYNC_PDU_BACK2BACK=y
CONFIG_BT_CTLR_ADV_DATA_BUF_MAX=6

# Increase the below to receive interleaved advertising chains
CONFIG_BT_CTLR_SCAN_AUX_SET=1
# CONFIG_BT_CTLR_SCAN_AUX_USE_CHAINS=y
# CONFIG_BT_CTLR_SCAN_AUX_CHAIN_COUNT=1
# CONFIG_BT_CTLR_SCAN_AUX_SET=1 # DEPRECATED
CONFIG_BT_CTLR_SCAN_AUX_USE_CHAINS=y
CONFIG_BT_CTLR_SCAN_AUX_CHAIN_COUNT=3

CONFIG_BT_CTLR_ADV_RESERVE_MAX=n
CONFIG_BT_CTLR_ADV_ISO_RESERVE_MAX=y
Expand Down
24 changes: 16 additions & 8 deletions subsys/bluetooth/controller/Kconfig.ll_sw_split
Original file line number Diff line number Diff line change
Expand Up @@ -515,20 +515,13 @@ config BT_CTRL_ADV_ADI_IN_SCAN_RSP
config BT_CTLR_SCAN_AUX_USE_CHAINS
bool "Use new chains based implementation for following advertising chains"
depends on BT_OBSERVER && BT_CTLR_ADV_EXT
default y
help
Use the new chains based implementation of following advertising chains
instead of the old sets based one. The new implementation has a much lighter
footprint, since it limits allocations per advertising chain to the bare
minimum and only uses one ticker

config BT_CTLR_SCAN_AUX_SET
int "LE Extended Scanning Auxiliary Sets"
depends on BT_OBSERVER && BT_CTLR_ADV_EXT && !BT_CTLR_SCAN_AUX_USE_CHAINS
range 1 64
default 1
help
Maximum supported auxiliary channel scan sets.

config BT_CTLR_SCAN_AUX_CHAIN_COUNT
int "LE Extended Scanning Auxiliary Chains"
depends on BT_OBSERVER && BT_CTLR_ADV_EXT && BT_CTLR_SCAN_AUX_USE_CHAINS
Expand All @@ -538,6 +531,21 @@ config BT_CTLR_SCAN_AUX_CHAIN_COUNT
Maximum number of extended/periodic advertising chains that can be
followed simultaneously

config BT_CTLR_SCAN_AUX_SET
int "LE Extended Scanning Auxiliary Sets [DEPRECATED]"
depends on BT_OBSERVER && BT_CTLR_ADV_EXT && !BT_CTLR_SCAN_AUX_USE_CHAINS
range 1 64
default 1
help
Maximum supported auxiliary channel scan sets.

config BT_CTLR_SCAN_AUX_SET_DEPRECATED
# Hidden Kconfig to select DEPRECATED if BT_CTLR_SCAN_AUX_SET is used
bool
depends on BT_OBSERVER && BT_CTLR_ADV_EXT && !BT_CTLR_SCAN_AUX_USE_CHAINS
select DEPRECATED
default y

config BT_CTLR_SCAN_AUX_SYNC_RESERVE_MIN
bool "Use minimal Scan Auxiliary and Periodic Sync PDU time reservation"
depends on (BT_OBSERVER && BT_CTLR_ADV_EXT) || BT_CTLR_SYNC_PERIODIC
Expand Down
6 changes: 3 additions & 3 deletions tests/bsim/bluetooth/audio/overlay-bt_ll_sw_split.conf
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@ CONFIG_BT_CTLR_ADV_SYNC_PDU_BACK2BACK=y
CONFIG_BT_CTLR_ADV_DATA_BUF_MAX=6

# Increase the below to receive interleaved advertising chains
CONFIG_BT_CTLR_SCAN_AUX_SET=1
# CONFIG_BT_CTLR_SCAN_AUX_USE_CHAINS=y
# CONFIG_BT_CTLR_SCAN_AUX_CHAIN_COUNT=1
# CONFIG_BT_CTLR_SCAN_AUX_SET=1 # DEPRECATED
CONFIG_BT_CTLR_SCAN_AUX_USE_CHAINS=y
CONFIG_BT_CTLR_SCAN_AUX_CHAIN_COUNT=3

CONFIG_BT_CTLR_ADV_RESERVE_MAX=n
CONFIG_BT_CTLR_ADV_ISO_RESERVE_MAX=y
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,9 @@ CONFIG_BT_CTLR_ADV_SYNC_PDU_BACK2BACK=y
CONFIG_BT_CTLR_ADV_DATA_BUF_MAX=6

# Increase the below to receive interleaved advertising chains
CONFIG_BT_CTLR_SCAN_AUX_SET=1
# CONFIG_BT_CTLR_SCAN_AUX_SET=1 # DEPRECATED
CONFIG_BT_CTLR_SCAN_AUX_USE_CHAINS=y
CONFIG_BT_CTLR_SCAN_AUX_CHAIN_COUNT=3

CONFIG_BT_CTLR_ADV_RESERVE_MAX=n
CONFIG_BT_CTLR_CENTRAL_RESERVE_MAX=n
Expand Down

0 comments on commit 290d098

Please sign in to comment.