bartoszek/zephyr
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

samples: Bluetooth: hci_ipc: __ASSERT on fail to deliver complete message

Browse Source 
The hci_ipc sample does not support fragmentation of IPC-messages.
Give an ASSERT if only parts of an IPC-message is delivered.
This would happen for instance when the underlying transport mechanism
cannot hold the complete message.

Background:
The open-amp backend uses RPMSG_BUFFER_SIZE=512

Most applications and controllers use HCI-packets which are at most
255 bytes. In some cases a controller may deliver HCI-packets beyond
255 bytes.

This may for instance be the case if the controller delivers large ISO
SDUs as a single HCI-packet.

Signed-off-by: Johan Stridkvist <johan.stridkvist@nordicsemi.no>
This commit is contained in:
Johan Stridkvist 2025-06-13 07:56:07 +02:00 committed by Henrik Brix Andersen
parent 639cb9b3bc
commit df0db6a7a3
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
samples/bluetooth/hci_ipc/src/main.c
View File

@ -252,7 +252,7 @@ static void hci_ipc_send(struct net_buf *buf, bool is_fatal_err)
* call to k_yield is against it.
*/
if (is_fatal_err) {
LOG_ERR("IPC service send error: %d", ret);
LOG_ERR("ipc_service_send error: %d", ret);
} else {
/* In the POSIX ARCH, code takes zero simulated time to execute,
* so busy wait loops become infinite loops, unless we
@ -267,7 +267,8 @@ static void hci_ipc_send(struct net_buf *buf, bool is_fatal_err)
}
} while (ret < 0);
LOG_INF("Sent message of %d bytes.", ret);
LOG_INF("ipc_service_send sent %d/%u bytes", ret, buf->len);
__ASSERT_NO_MSG(ret == buf->len);
net_buf_unref(buf);
}