Skip to content

nimble/transport/common/hci_h4: add HCI H4 packet callback #2247

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
jwise wants to merge 1 commit into
base: master
Choose a base branch
from
+31 −0
Open

nimble/transport/common/hci_h4: add HCI H4 packet callback #2247

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions nimble/transport/common/hci_h4/include/nimble/transport/hci_h4.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,17 @@ struct hci_h4_allocators {
extern const struct hci_h4_allocators hci_h4_allocs_from_ll;
extern const struct hci_h4_allocators hci_h4_allocs_from_hs;

/** Callback invoked on every frame that the H4 parser sees; it is this
* callback's responsibility to dispatch packets to the transport layer.
*/
typedef int (hci_h4_frame_cb)(uint8_t pkt_type, void *data);

/** Callback invoked at the start of every packet that the H4 parser sees,
* to allow a transport to parse custom types of packets that the H4 parser
* may not otherwise understand (for instance, TI eHCILL packets).
*/
typedef uint16_t(hci_h4_packet_cb)(const uint8_t *buf, uint16_t len);

struct hci_h4_sm {
uint8_t state;
uint8_t pkt_type;
Expand All @@ -59,12 +68,19 @@ struct hci_h4_sm {

const struct hci_h4_allocators *allocs;
hci_h4_frame_cb *frame_cb;
hci_h4_packet_cb *packet_cb;
};

void hci_h4_sm_init(struct hci_h4_sm *h4sm,
const struct hci_h4_allocators *allocs,
hci_h4_frame_cb *frame_cb);

/* We do not include this in sm_init to avoid breaking the API for other
* users, but ideally if you need to call this, you will call it immediately
* once, right after sm_init.
*/
void hci_h4_sm_set_packet_cb(struct hci_h4_sm *h4sm, hci_h4_packet_cb *packet_cb);

int hci_h4_sm_rx(struct hci_h4_sm *h4sm, const uint8_t *buf, uint16_t len);

#endif /* _HCI_H4_H_ */
15 changes: 15 additions & 0 deletions nimble/transport/common/hci_h4/src/hci_h4.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -283,6 +283,15 @@ hci_h4_sm_rx(struct hci_h4_sm *h4sm, const uint8_t *buf, uint16_t len)
rc = 0;
switch (h4sm->state) {
case HCI_H4_SM_W4_PKT_TYPE:
if (h4sm->packet_cb) {
/* perhaps the chipset / transport driver wants to inspect
* start-of-packet and consume it on its own? */
uint16_t consumed = h4sm->packet_cb(ib.buf, ib.len);
if (consumed) {
hci_h4_ib_consume(&ib, consumed);
continue;
}
}
hci_h4_frame_start(h4sm, ib.buf[0]);
hci_h4_ib_consume(&ib, 1);
h4sm->state = HCI_H4_SM_W4_HEADER;
Expand Down Expand Up @@ -337,3 +346,9 @@ hci_h4_sm_init(struct hci_h4_sm *h4sm, const struct hci_h4_allocators *allocs,
h4sm->allocs = allocs;
h4sm->frame_cb = frame_cb;
}

void
hci_h4_sm_set_packet_cb(struct hci_h4_sm *h4sm, hci_h4_packet_cb *packet_cb)
{
h4sm->packet_cb = packet_cb;
}
Loading