Bluetooth: Controller: Kconfig: Allow setting event length to 0

In some cases we really do not want to allocate more time than needed for a single packet pair. This can now be achieved by setting BT_CTLR_SDC_MAX_CONN_EVENT_LEN_DEFAULT to 0. Signed-off-by: Rubin Gerritsen <[email protected]>
DatGizmo · Nov 23, 2023 · 72e0f9a · 72e0f9a
1 parent 2cd0e2a
commit 72e0f9a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/subsys/bluetooth/controller/Kconfig b/subsys/bluetooth/controller/Kconfig
@@ -137,11 +137,15 @@ config BT_CTLR_SDC_MAX_CONN_EVENT_LEN_DEFAULT
 	int "Default max connection event length [us]"
 	default 10000 if BT_ISO && !BT_CTLR_SDC_MAX_CONN_EVENT_LEN_DEFAULT_OVERRIDE
 	default 7500 if !BT_CTLR_SDC_MAX_CONN_EVENT_LEN_DEFAULT_OVERRIDE
-	range 1250 4000000
+	range 0 4000000
 	help
 	  The time set aside for connections on every connection interval in
 	  microseconds. The event length and the connection interval are the
 	  primary parameters for setting the throughput of a connection.
+	  The event length may be set to a value that is shorter than the time needed
+	  for a single packet pair on a given PHY.
+	  In that case the controller will reserve time for receiving 27 bytes and transmitting
+	  the number of bytes configured with BT_CTLR_MIN_VAL_OF_MAX_ACL_TX_PAYLOAD_DEFAULT.
 
 config BT_CTLR_SDC_CENTRAL_ACL_EVENT_SPACING_DEFAULT
 	int "Default central ACL event spacing [us]"

diff --git a/subsys/bluetooth/controller/hci_driver.c b/subsys/bluetooth/controller/hci_driver.c
@@ -1242,7 +1242,7 @@ static int hci_driver_open(void)
 	if (CONFIG_BT_CTLR_MIN_VAL_OF_MAX_ACL_TX_PAYLOAD_DEFAULT != 27) {
 		sdc_hci_cmd_vs_min_val_of_max_acl_tx_payload_set_t params = {
 			.min_val_of_max_acl_tx_payload =
-				CONFIG_config BT_CTLR_MIN_VAL_OF_MAX_ACL_TX_PAYLOAD_DEFAULT
+				CONFIG_BT_CTLR_MIN_VAL_OF_MAX_ACL_TX_PAYLOAD_DEFAULT
 		};
 		err = sdc_hci_cmd_vs_min_val_of_max_acl_tx_payload_set(&params);
 		if (err) {