Like thus? Sorry, not too familiar with the email patches. > From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx> > > Due to what seem to be a bug with variant version returned by some > firmwares the code may set hdev->classify_pkt_type with > btintel_classify_pkt_type when in fact the controller doesn't even > support ISO channels feature but may use the handle range expected from > a controllers that does causing the packets to be reclassified as ISO > causing several bugs. > > Link: https://bugzilla.kernel.org/show_bug.cgi?id=219553 > Link: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/2100565 > Link: https://github.com/StarLabsLtd/firmware/issues/180 > Fixes: f25b7fd36cc3 ("Bluetooth: Add vendor-specific packet classification for ISO data") > Tested-by: Sean McAllen <sean.mcallen@xxxxxxxxxxx> > Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx> > --- > drivers/bluetooth/btintel.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/bluetooth/btintel.c b/drivers/bluetooth/btintel.c > index 55cc1652bfe4..06575a0b9aee 100644 > --- a/drivers/bluetooth/btintel.c > +++ b/drivers/bluetooth/btintel.c > @@ -2670,7 +2670,7 @@ static u8 btintel_classify_pkt_type(struct hci_dev *hdev, struct sk_buff *skb) > * Distinguish ISO data packets form ACL data packets > * based on their connection handle value range. > */ > - if (hci_skb_pkt_type(skb) == HCI_ACLDATA_PKT) { > + if (iso_capable(hdev) && hci_skb_pkt_type(skb) == HCI_ACLDATA_PKT) { > __u16 handle = __le16_to_cpu(hci_acl_hdr(skb)->handle); > > if (hci_handle(handle) >= BTINTEL_ISODATA_HANDLE_BASE) > -- > 2.50.0 On Thu, 10 Jul 2025 at 14:33, Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx> wrote: > > Hi Sean, > > On Thu, Jul 10, 2025 at 9:18 AM Sean Rhodes <sean@starlabs.systems> wrote: > > > > Hi Luiz > > > > Yes, works perfectly with that patch. > > > > Thanks > > > > Sean > > > > On Wed, 9 Jul 2025 at 21:06, Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx> wrote: > >> > >> Hi Sean, > >> > >> On Wed, Jul 9, 2025 at 3:13 PM Luiz Augusto von Dentz > >> <luiz.dentz@xxxxxxxxx> wrote: > >> > > >> > From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx> > >> > > >> > Due to what seem to be a bug with variant version returned by some > >> > firmwares the code may set hdev->classify_pkt_type with > >> > btintel_classify_pkt_type when in fact the controller doesn't even > >> > support ISO channels feature but may use the handle range expected from > >> > a controllers that does causing the packets to be reclassified as ISO > >> > causing several bugs. > >> > > >> > Link: https://bugzilla.kernel.org/show_bug.cgi?id=219553 > >> > Link: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/2100565 > >> > Link: https://github.com/StarLabsLtd/firmware/issues/180 > >> > Fixes: f25b7fd36cc3 ("Bluetooth: Add vendor-specific packet classification for ISO data") > >> > Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx> > >> > --- > >> > drivers/bluetooth/btintel.c | 2 +- > >> > 1 file changed, 1 insertion(+), 1 deletion(-) > >> > > >> > diff --git a/drivers/bluetooth/btintel.c b/drivers/bluetooth/btintel.c > >> > index 55cc1652bfe4..06575a0b9aee 100644 > >> > --- a/drivers/bluetooth/btintel.c > >> > +++ b/drivers/bluetooth/btintel.c > >> > @@ -2670,7 +2670,7 @@ static u8 btintel_classify_pkt_type(struct hci_dev *hdev, struct sk_buff *skb) > >> > * Distinguish ISO data packets form ACL data packets > >> > * based on their connection handle value range. > >> > */ > >> > - if (hci_skb_pkt_type(skb) == HCI_ACLDATA_PKT) { > >> > + if (iso_capable(hdev) && hci_skb_pkt_type(skb) == HCI_ACLDATA_PKT) { > >> > __u16 handle = __le16_to_cpu(hci_acl_hdr(skb)->handle); > >> > > >> > if (hci_handle(handle) >= BTINTEL_ISODATA_HANDLE_BASE) > >> > -- > >> > 2.50.0 > >> > >> Can you try with the changes above? > >> > >> -- > >> Luiz Augusto von Dentz > > Please add Tested-by so we can move to get it merged, I do intend to > send a pull request later this week. > > -- > Luiz Augusto von Dentz
