| Message ID | 20230320061813.69895-1-chethan.tumkur.narayan@intel.com |
|---|---|
| State | Accepted |
| Commit | 5b23f8375a3685b93022feb625a2c4b43e5c0ad3 |
| Headers | show |
| Series | Bluetooth: btintel: Fix: Add LE States quirk form Solar onwards | expand |
Thanks for the feedback, will send the updated patch. > -----Original Message----- > From: Paul Menzel <pmenzel@molgen.mpg.de> > Sent: Monday, March 20, 2023 6:40 PM > To: Tumkur Narayan, Chethan <chethan.tumkur.narayan@intel.com> > Cc: Srivatsa, Ravishankar <ravishankar.srivatsa@intel.com>; K, Kiran > <kiran.k@intel.com>; linux-bluetooth@vger.kernel.org > Subject: Re: [PATCH] Bluetooth: btintel: Fix: Add LE States quirk form Solar > onwards > > Dear Chethan, > > > Thank you for your patch. There is a small typo in the summary/title: > s/form/from/ > > Am 20.03.23 um 07:18 schrieb Chethan T N: > > This patch shall enable the LE States quirks by default on all > > s/shall enable/enables/ > > > Intel controller from Solar products on wards. > > onwards > > Where is it documented, to enable that quirk for all? > > > Signed-off-by: Chethan T N <chethan.tumkur.narayan@intel.com> > > --- > > drivers/bluetooth/btintel.c | 5 ++--- > > 1 file changed, 2 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/bluetooth/btintel.c b/drivers/bluetooth/btintel.c > > index af774688f1c0..7a6dc05553f1 100644 > > --- a/drivers/bluetooth/btintel.c > > +++ b/drivers/bluetooth/btintel.c > > @@ -2684,9 +2684,8 @@ static int btintel_setup_combined(struct hci_dev > *hdev) > > */ > > set_bit(HCI_QUIRK_WIDEBAND_SPEECH_SUPPORTED, > &hdev->quirks); > > > > - /* Valid LE States quirk for GfP */ > > - if (INTEL_HW_VARIANT(ver_tlv.cnvi_bt) == 0x18) > > - set_bit(HCI_QUIRK_VALID_LE_STATES, &hdev- > >quirks); > > + /* Apply LE States quirk from solar onwards */ > > + set_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks); > > From the diff it is unclear, if Solar is the first “device”. So I’d be nice if you > elaborated in the commit message. > > > > > /* Setup MSFT Extension support */ > > btintel_set_msft_opcode(hdev, > > > Kind regards, > > Paul
Hello: This patch was applied to bluetooth/bluetooth-next.git (master) by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>: On Mon, 20 Mar 2023 11:48:13 +0530 you wrote: > This patch shall enable the LE States quirks by default on all > Intel controller from Solar products on wards. > > Signed-off-by: Chethan T N <chethan.tumkur.narayan@intel.com> > --- > drivers/bluetooth/btintel.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) Here is the summary with links: - Bluetooth: btintel: Fix: Add LE States quirk form Solar onwards https://git.kernel.org/bluetooth/bluetooth-next/c/5b23f8375a36 You are awesome, thank you!
diff --git a/drivers/bluetooth/btintel.c b/drivers/bluetooth/btintel.c index af774688f1c0..7a6dc05553f1 100644 --- a/drivers/bluetooth/btintel.c +++ b/drivers/bluetooth/btintel.c @@ -2684,9 +2684,8 @@ static int btintel_setup_combined(struct hci_dev *hdev) */ set_bit(HCI_QUIRK_WIDEBAND_SPEECH_SUPPORTED, &hdev->quirks); - /* Valid LE States quirk for GfP */ - if (INTEL_HW_VARIANT(ver_tlv.cnvi_bt) == 0x18) - set_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks); + /* Apply LE States quirk from solar onwards */ + set_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks); /* Setup MSFT Extension support */ btintel_set_msft_opcode(hdev,
This patch shall enable the LE States quirks by default on all Intel controller from Solar products on wards. Signed-off-by: Chethan T N <chethan.tumkur.narayan@intel.com> --- drivers/bluetooth/btintel.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)