2021-06-30 08:06:21

by Kiran K

[permalink] [raw]
Subject: [PATCH v10 03/10] Bluetooth: btintel: Read supported offload usecases

Read offload usecases supported by controller.

Signed-off-by: Kiran K <[email protected]>
Reviewed-by: Chethan T N <[email protected]>
Reviewed-by: Srivatsa Ravishankar <[email protected]>
---
* changes in v10:
- restructure patch to have definition and call of callaback in the
same patch
* changes in v9:
- define a separate patch for core changes

drivers/bluetooth/btintel.c | 32 ++++++++++++++++++++++++++++++++
drivers/bluetooth/btintel.h | 16 ++++++++++++++++
drivers/bluetooth/btusb.c | 5 +++++
3 files changed, 53 insertions(+)

diff --git a/drivers/bluetooth/btintel.c b/drivers/bluetooth/btintel.c
index e44b6993cf91..7f9b5f82d01f 100644
--- a/drivers/bluetooth/btintel.c
+++ b/drivers/bluetooth/btintel.c
@@ -1272,6 +1272,38 @@ int btintel_set_debug_features(struct hci_dev *hdev,
}
EXPORT_SYMBOL_GPL(btintel_set_debug_features);

+#if IS_ENABLED(CONFIG_BT_OFFLOAD_CODECS)
+int btintel_configure_offload_usecases(struct hci_dev *hdev)
+{
+ struct sk_buff *skb;
+ int err = 0;
+ struct intel_offload_usecases *usecases;
+
+ skb = __hci_cmd_sync(hdev, 0xfc86, 0, NULL, HCI_INIT_TIMEOUT);
+ if (IS_ERR(skb)) {
+ bt_dev_err(hdev, "Reading offload usecases failed (%ld)",
+ PTR_ERR(skb));
+ return PTR_ERR(skb);
+ }
+
+ if (skb->len < sizeof(*usecases)) {
+ err = -EIO;
+ goto error;
+ }
+
+ usecases = (void *)skb->data;
+
+ if (usecases->status) {
+ err = -bt_to_errno(skb->data[0]);
+ goto error;
+ }
+error:
+ kfree_skb(skb);
+ return err;
+}
+EXPORT_SYMBOL_GPL(btintel_configure_offload_usecases);
+#endif
+
MODULE_AUTHOR("Marcel Holtmann <[email protected]>");
MODULE_DESCRIPTION("Bluetooth support for Intel devices ver " VERSION);
MODULE_VERSION(VERSION);
diff --git a/drivers/bluetooth/btintel.h b/drivers/bluetooth/btintel.h
index d184064a5e7c..b667fc000c03 100644
--- a/drivers/bluetooth/btintel.h
+++ b/drivers/bluetooth/btintel.h
@@ -132,6 +132,13 @@ struct intel_debug_features {
__u8 page1[16];
} __packed;

+#if IS_ENABLED(CONFIG_BT_OFFLOAD_CODECS)
+struct intel_offload_usecases {
+ __u8 status;
+ __u8 preset[8];
+} __packed;
+#endif
+
#define INTEL_HW_PLATFORM(cnvx_bt) ((u8)(((cnvx_bt) & 0x0000ff00) >> 8))
#define INTEL_HW_VARIANT(cnvx_bt) ((u8)(((cnvx_bt) & 0x003f0000) >> 16))
#define INTEL_CNVX_TOP_TYPE(cnvx_top) ((cnvx_top) & 0x00000fff)
@@ -175,6 +182,9 @@ int btintel_read_debug_features(struct hci_dev *hdev,
struct intel_debug_features *features);
int btintel_set_debug_features(struct hci_dev *hdev,
const struct intel_debug_features *features);
+#if IS_ENABLED(CONFIG_BT_OFFLOAD_CODECS)
+int btintel_configure_offload_usecases(struct hci_dev *hdev);
+#endif
#else

static inline int btintel_check_bdaddr(struct hci_dev *hdev)
@@ -307,4 +317,10 @@ static inline int btintel_set_debug_features(struct hci_dev *hdev,
return -EOPNOTSUPP;
}

+#if IS_ENABLED(CONFIG_BT_OFFLOAD_CODECS)
+static int btintel_configure_offload_usecases(struct hci_dev *hdev)
+{
+ return -EOPNOTSUPP;
+}
+#endif
#endif
diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index a9855a2dd561..0184e8668a84 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -3008,6 +3008,11 @@ static int btusb_setup_intel_newgen(struct hci_dev *hdev)
/* Set DDC mask for available debug features */
btintel_set_debug_features(hdev, &features);

+#if IS_ENABLED(CONFIG_BT_OFFLOAD_CODECS)
+ /* Read supported usecases and set callbacks to fetch datapath id */
+ btintel_configure_offload_usecases(hdev);
+#endif
+
/* Read the Intel version information after loading the FW */
err = btintel_read_version_tlv(hdev, &version);
if (err)
--
2.17.1