Messages in this thread Patch in this message | | | From | Hilda Wu <> | Subject | RE: [PATCH] Bluetooth: btusb: Add support USB ALT 3 for WBS | Date | Fri, 14 May 2021 02:31:45 +0000 |
| |
Hi Marcel,
Thank you! I will propose a patch v2.
Regards, Hilda
-----Original Message----- From: Marcel Holtmann <marcel@holtmann.org> Sent: Thursday, May 13, 2021 11:13 PM To: Hilda Wu <hildawu@realtek.com> Cc: Johan Hedberg <johan.hedberg@gmail.com>; Luiz Augusto von Dentz <luiz.dentz@gmail.com>; Bluetooth Kernel Mailing List <linux-bluetooth@vger.kernel.org>; open list <linux-kernel@vger.kernel.org>; Max Chou <max.chou@realtek.com>; alex_lu@realsil.com.cn; KidmanLee <kidman@realtek.com> Subject: Re: [PATCH] Bluetooth: btusb: Add support USB ALT 3 for WBS
Hi Hilda,
> Because mSBC frames do not need to be aligned to the SCO packet > boundary. Using USB ALT 3 let HCI payload >= 60 bytes, let mSBC data > satisfy 60 Bytes avoid payload unaligned situation and fixed some > headset no voise issue. > > USB Alt 3 supported also need HFP support transparent MTU in 72 Bytes. > > Signed-off-by: hildawu <hildawu@realtek.com>
please use real name here.
> --- > drivers/bluetooth/btusb.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c > index 6f253378e893..872034e7a232 100644 > --- a/drivers/bluetooth/btusb.c > +++ b/drivers/bluetooth/btusb.c > @@ -1752,6 +1752,13 @@ static void btusb_work(struct work_struct *work) > * which work with WBS at all. > */ > new_alts = btusb_find_altsetting(data, 6) ? 6 : 1; > + /* Because mSBC frames do not need to be aligned to the > + * SCO packet boundary. If support the Alt 3, use the > + * Alt 3 for HCI payload >= 60 Bytes let air packet > + * data satisfy 60 bytes. > + */ > + if ((new_alts == 1) && (btusb_find_altsetting(data, 3))) > + new_alts = 3;
if (new_alts == 1 && btusb_find_altsetting(..)) ..
Regards
Marcel
------Please consider the environment before printing this e-mail.
| |