Skip to content
Permalink
Browse files

Bluetooth: controller: split: Fix tx_ack mfifo count

Fix the tx_ack mfifo count to accomodate both data and
control PDUs being acknowledged.

With out this fix, pending maximum number of data plus
control PDUs in LLL on supervision timeout asserted due to
tx_ack mfifo overflow.

Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
  • Loading branch information...
cvinayak authored and carlescufi committed Jun 7, 2019
1 parent 23a6de5 commit 3169556592303f27bc01c6c79225288883d7dbd4
Showing with 4 additions and 3 deletions.
  1. +4 −3 subsys/bluetooth/controller/ll_sw/ull_conn.c
@@ -102,7 +102,8 @@ static void ticker_start_conn_op_cb(u32_t status, void *param);
sizeof(struct pdu_data_llctrl))

static MFIFO_DEFINE(conn_tx, sizeof(struct lll_tx), CONFIG_BT_CTLR_TX_BUFFERS);
static MFIFO_DEFINE(conn_ack, sizeof(struct lll_tx), CONFIG_BT_CTLR_TX_BUFFERS);
static MFIFO_DEFINE(conn_ack, sizeof(struct lll_tx),
(CONFIG_BT_CTLR_TX_BUFFERS + CONN_TX_CTRL_BUFFERS));


static struct {
@@ -1159,8 +1160,8 @@ void ull_conn_tx_demux(u8_t count)
break;
}

conn = ll_conn_get(lll_tx->handle);
if (conn->lll.handle == lll_tx->handle) {
conn = ll_connected_get(lll_tx->handle);
if (conn) {
struct node_tx *tx = lll_tx->node;

tx->next = NULL;

0 comments on commit 3169556

Please sign in to comment.
You can’t perform that action at this time.