Bluetooth: btintel: Add missing quirks and msft ext for legacy bootloader
authorTedd Ho-Jeong An <tedd.an@intel.com>
Tue, 14 Dec 2021 07:32:14 +0000 (23:32 -0800)
committerMarcel Holtmann <marcel@holtmann.org>
Wed, 22 Dec 2021 07:53:41 +0000 (08:53 +0100)
This patch add missing HCI quirks and MSFT extension for legacy
bootloader when it is running in the operational firmware.

Signed-off-by: Tedd Ho-Jeong An <tedd.an@intel.com>
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
drivers/bluetooth/btintel.c

index 8f9109b40961f4e80016ee9c138d495a7bd9fe57..e1f96df847b8fcced9da8cc703154c54e2efeeb0 100644 (file)
@@ -2498,10 +2498,14 @@ static int btintel_setup_combined(struct hci_dev *hdev)
        case 0x12:      /* ThP */
        case 0x13:      /* HrP */
        case 0x14:      /* CcP */
-               /* Some legacy bootloader devices from JfP supports both old
-                * and TLV based HCI_Intel_Read_Version command. But we don't
-                * want to use the TLV based setup routines for those legacy
-                * bootloader device.
+               /* Some legacy bootloader devices starting from JfP,
+                * the operational firmware supports both old and TLV based
+                * HCI_Intel_Read_Version command based on the command
+                * parameter.
+                *
+                * For upgrading firmware case, the TLV based version cannot
+                * be used because the firmware filename for legacy bootloader
+                * is based on the old format.
                 *
                 * Also, it is not easy to convert TLV based version from the
                 * legacy version format.
@@ -2513,6 +2517,20 @@ static int btintel_setup_combined(struct hci_dev *hdev)
                err = btintel_read_version(hdev, &ver);
                if (err)
                        return err;
+
+               /* Apply the device specific HCI quirks
+                *
+                * All Legacy bootloader devices support WBS
+                */
+               set_bit(HCI_QUIRK_WIDEBAND_SPEECH_SUPPORTED, &hdev->quirks);
+
+               /* Valid LE States quirk for JfP/ThP familiy */
+               if (ver.hw_variant == 0x11 || ver.hw_variant == 0x12)
+                       set_bit(HCI_QUIRK_VALID_LE_STATES, &hdev->quirks);
+
+               /* Setup MSFT Extension support */
+               btintel_set_msft_opcode(hdev, ver.hw_variant);
+
                err = btintel_bootloader_setup(hdev, &ver);
                break;
        case 0x17: