| Message ID | 20240606114321.30515-2-chris.lu@mediatek.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | Bluetooth: btusb: MediaTek ISO data transmission | expand |
Hi Chris, On Thu, Jun 6, 2024 at 7:43 AM Chris Lu <chris.lu@mediatek.com> wrote: > > Add function hci_iso_hdr to get skb->data when packet type is ISO. > > Signed-off-by: Chris Lu <chris.lu@mediatek.com> > Signed-off-by: Sean Wang <sean.wang@mediatek.com> > --- > include/net/bluetooth/hci.h | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h > index e372a88e8c3f..b9f8f91f6c7f 100644 > --- a/include/net/bluetooth/hci.h > +++ b/include/net/bluetooth/hci.h > @@ -2898,6 +2898,11 @@ static inline struct hci_sco_hdr *hci_sco_hdr(const struct sk_buff *skb) > return (struct hci_sco_hdr *) skb->data; > } > > +static inline struct hci_iso_hdr *hci_iso_hdr(const struct sk_buff *skb) > +{ > + return (struct hci_iso_hdr *) skb->data; > +} We might as well check the skb->len before attempting to cast skb->data or better yet use skb_pull_data but I guess that is not possible since you don't want to consume the skb? > /* Command opcode pack/unpack */ > #define hci_opcode_pack(ogf, ocf) ((__u16) ((ocf & 0x03ff)|(ogf << 10))) > #define hci_opcode_ogf(op) (op >> 10) > -- > 2.18.0 >
diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h index e372a88e8c3f..b9f8f91f6c7f 100644 --- a/include/net/bluetooth/hci.h +++ b/include/net/bluetooth/hci.h @@ -2898,6 +2898,11 @@ static inline struct hci_sco_hdr *hci_sco_hdr(const struct sk_buff *skb) return (struct hci_sco_hdr *) skb->data; } +static inline struct hci_iso_hdr *hci_iso_hdr(const struct sk_buff *skb) +{ + return (struct hci_iso_hdr *) skb->data; +} + /* Command opcode pack/unpack */ #define hci_opcode_pack(ogf, ocf) ((__u16) ((ocf & 0x03ff)|(ogf << 10))) #define hci_opcode_ogf(op) (op >> 10)