RE: [PATCH v1 2/3] Bluetooth: btintel_pcie: Add support for PCIE transport

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Luiz,

Thanks for your comments.

> -----Original Message-----
> From: Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx>
> Sent: Thursday, March 28, 2024 7:39 PM
> To: K, Kiran <kiran.k@xxxxxxxxx>
> Cc: linux-bluetooth@xxxxxxxxxxxxxxx; Srivatsa, Ravishankar
> <ravishankar.srivatsa@xxxxxxxxx>; Tumkur Narayan, Chethan
> <chethan.tumkur.narayan@xxxxxxxxx>; An, Tedd <tedd.an@xxxxxxxxx>
> Subject: Re: [PATCH v1 2/3] Bluetooth: btintel_pcie: Add support for PCIE
> transport
> 
> Hi Kiran,
> 
> On Thu, Mar 28, 2024 at 7:05 AM Kiran K <kiran.k@xxxxxxxxx> wrote:
> >
> > From: Tedd Ho-Jeong An <tedd.an@xxxxxxxxx>
> >
> > Add initial code to support Intel bluetooth devices based on PICE
> > transport. This patch allocates memory for buffers, internal structures,
> > initializes interrupts for TX & RX and initializes PCIE device.
> >
> > Signed-off-by: Tedd Ho-Jeong An <tedd.an@xxxxxxxxx>
> > Co-developed-by: Kiran K <kiran.k@xxxxxxxxx>
> > Signed-off-by: Kiran K <kiran.k@xxxxxxxxx>
> > ---
> >  drivers/bluetooth/Kconfig        |   14 +
> >  drivers/bluetooth/Makefile       |    1 +
> >  drivers/bluetooth/btintel.h      |    2 +-
> >  drivers/bluetooth/btintel_pcie.c | 1317
> ++++++++++++++++++++++++++++++
> >  drivers/bluetooth/btintel_pcie.h |  449 ++++++++++
> >  5 files changed, 1782 insertions(+), 1 deletion(-)
> >  create mode 100644 drivers/bluetooth/btintel_pcie.c
> >  create mode 100644 drivers/bluetooth/btintel_pcie.h
> >
> > diff --git a/drivers/bluetooth/Kconfig b/drivers/bluetooth/Kconfig
> > index bc211c324206..387f7b14461d 100644
> > --- a/drivers/bluetooth/Kconfig
> > +++ b/drivers/bluetooth/Kconfig
> > @@ -23,6 +23,20 @@ config BT_MTK
> >         tristate
> >         select FW_LOADER
> >
> > +config BT_INTEL_PCIE
> > +       tristate "Intel Bluetooth transport driver for PCIe"
> > +       depends on PCI
> > +       select BT_INTEL
> > +       select FW_LOADER
> > +       default y
> > +       help
> > +         Intel Bluetooth transport driver for PCIe.
> > +         This driver is required if you want to use Intel Bluetooth device
> > +         with PCIe interface.
> > +
> > +         Say Y here to compiler support for Intel Bluetooth PCIe device into
> > +         the kernel or say M to compile it as module (btintel_pcie)
> > +
> >  config BT_HCIBTUSB
> >         tristate "HCI USB driver"
> >         depends on USB
> > diff --git a/drivers/bluetooth/Makefile b/drivers/bluetooth/Makefile
> > index 7a5967e9ac48..0730d6684d1a 100644
> > --- a/drivers/bluetooth/Makefile
> > +++ b/drivers/bluetooth/Makefile
> > @@ -17,6 +17,7 @@ obj-$(CONFIG_BT_HCIBTUSB)     += btusb.o
> >  obj-$(CONFIG_BT_HCIBTSDIO)     += btsdio.o
> >
> >  obj-$(CONFIG_BT_INTEL)         += btintel.o
> > +obj-$(CONFIG_BT_INTEL_PCIE)    += btintel_pcie.o btintel.o
> >  obj-$(CONFIG_BT_ATH3K)         += ath3k.o
> >  obj-$(CONFIG_BT_MRVL)          += btmrvl.o
> >  obj-$(CONFIG_BT_MRVL_SDIO)     += btmrvl_sdio.o
> > diff --git a/drivers/bluetooth/btintel.h b/drivers/bluetooth/btintel.h
> > index 1462a57420a0..5d4685b5c1fa 100644
> > --- a/drivers/bluetooth/btintel.h
> > +++ b/drivers/bluetooth/btintel.h
> > @@ -209,7 +209,7 @@ struct btintel_data {
> >  #define btintel_wait_on_flag_timeout(hdev, nr, m, to)                  \
> >                 wait_on_bit_timeout(btintel_get_flag(hdev), (nr), m, to)
> >
> > -#if IS_ENABLED(CONFIG_BT_INTEL)
> > +#if IS_ENABLED(CONFIG_BT_INTEL) ||
> IS_ENABLED(CONFIG_BT_INTEL_PCIE)
> >
> >  int btintel_check_bdaddr(struct hci_dev *hdev);
> >  int btintel_enter_mfg(struct hci_dev *hdev);
> > diff --git a/drivers/bluetooth/btintel_pcie.c
> b/drivers/bluetooth/btintel_pcie.c
> > new file mode 100644
> > index 000000000000..e6ce2304dc57
> > --- /dev/null
> > +++ b/drivers/bluetooth/btintel_pcie.c
> > @@ -0,0 +1,1317 @@
> > +// SPDX-License-Identifier: GPL-2.0-or-later
> > +/*
> > + * Intel Bluetooth PCIE driver
> > + *
> > + * Copyright (C) 2017 Intel Corporation. All rights reserved.
> 
> The above shall probably be updated to 2024, also the All rights
> reserved don't actually fit well with GPL so please remove it.
Ack.

> 
> > + * This program is free software; you can redistribute it and/or
> > + * modify it under the terms of the GNU General Public License version
> > + * 2 as published by the Free Software Foundation.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> > + * GNU General Public License for more details.
> > + *
> > + * Copyright (C) 2022  Intel Corporation
> 
> See above.
Ack.

> 
> > + * Intel Bluetooth Driver for PCIE interface.
> > + */
> > +#include <linux/kernel.h>
> > +#include <linux/module.h>
> > +#include <linux/firmware.h>
> > +#include <linux/pci.h>
> > +#include <linux/wait.h>
> > +#include <linux/delay.h>
> > +#include <linux/interrupt.h>
> > +
> > +#include <asm/unaligned.h>
> > +
> > +#include <net/bluetooth/bluetooth.h>
> > +#include <net/bluetooth/hci_core.h>
> > +
> > +#include "btintel.h"
> > +#include "btintel_pcie.h"
> > +
> > +#define VERSION "0.1"
> > +
> > +#define BTINTEL_PCI_DEVICE(dev, subdev)        \
> > +       .vendor = PCI_VENDOR_ID_INTEL,  \
> > +       .device = (dev),                \
> > +       .subvendor = PCI_ANY_ID,        \
> > +       .subdevice = (subdev),          \
> > +       .driver_data = 0
> > +
> > +/* Intel Bluetooth PCIe device id table */
> > +static const struct pci_device_id btintel_pcie_table[] = {
> > +       { BTINTEL_PCI_DEVICE(0xA876, PCI_ANY_ID) },
> > +       { 0 }
> > +};
> > +MODULE_DEVICE_TABLE(pci, btintel_pcie_table);
> > +
> > +/* Intel PCIe uses 4 bytes of HCI type instead of 1 byte BT SIG HCI type */
> > +#define BTINTEL_PCIE_HCI_TYPE_LEN      4
> > +#define BTINTEL_PCIE_HCI_CMD_PKT       0x00000001
> > +#define BTINTEL_PCIE_HCI_ACL_PKT       0x00000002
> > +#define BTINTEL_PCIE_HCI_SCO_PKT       0x00000003
> > +#define BTINTEL_PCIE_HCI_EVT_PKT       0x00000004
> > +
> > +#define BTITNEL_PCIE_ENABLE_HCI_DUMP   0
> > +
> > +#if BTITNEL_PCIE_ENABLE_HCI_DUMP
> > +static inline void btintel_pcie_hci_dump(const char *p, const void *b, int s)
> > +{
> > +       const unsigned char *ptr = (const unsigned char *)b;
> > +       char str[64];
> > +       int c, i;
> > +
> > +       for (i = c = 0; c < s; c++) {
> > +               i += snprintf(str + i, sizeof(str) - i, "%02x ", ptr[c]);
> > +               if ((c > 0 && (c + 1) % 8 == 0) || (c == s - 1)) {
> > +                       BT_DBG("%s: %s", p, str);
> > +                       i = 0;
> > +               }
> > +       }
> > +}
> > +#else
> > +static inline void btintel_pcie_hci_dump(const char *p, const void *b, int s)
> > +{
> > +}
> > +#endif
> 
> We have the monitor to process HCI traffic, not really sure what is
> the purpose with the functions above.
Ack. I will remove this function.

> 
> > +static void ipc_print_ia_ring(struct ia *ia, u16 queue_num)
> > +{
> > +       BT_DBG("[%s] ---------------- ia ----------------",
> > +              queue_num == TXQ_NUM ? "TXQ" : "RXQ");
> > +       BT_DBG("[%s] tr-h:%02u  tr-t:%02u  cr-h:%02u  cr-t:%02u",
> > +              queue_num == TXQ_NUM ? "TXQ" : "RXQ",
> > +              ia->tr_hia[queue_num], ia->tr_tia[queue_num],
> > +              ia->cr_hia[queue_num], ia->cr_tia[queue_num]);
> > +}
> > +
> > +static void ipc_print_urbd0(struct urbd0 *urbd0, u16 index)
> > +{
> > +       BT_DBG("[TXQ] -------------- urbd0[%u] --------------", index);
> > +       BT_DBG("[TXQ] tfd_index:%u num_txq:%u cmpl_cnt:%u
> immediate_cmpl:0x%x",
> > +              urbd0->tfd_index, urbd0->num_txq, urbd0->cmpl_count,
> > +              urbd0->immediate_cmpl);
> > +}
> > +
> > +static void ipc_print_frbd(struct frbd *frbd, u16 index)
> > +{
> > +       BT_DBG("[RXQ] -------------- frbd[%u] --------------", index);
> > +       BT_DBG("[RXQ] tag:%u addr:0x%llx", frbd->tag, frbd->addr);
> > +}
> > +
> > +static void ipc_print_urbd1(struct urbd1 *urbd1, u16 index)
> > +{
> > +       BT_DBG("[RXQ] -------------- urbd1[%u] --------------", index);
> > +       BT_DBG("[RXQ] frbd_tag:%u status: 0x%x fixed:0x%x",
> > +              urbd1->frbd_tag, urbd1->status, urbd1->fixed);
> > +}
> 
> We need to replace usage of BT_DBG with bt_dev_dbg, etc.
Ack. I keep only relevant logs / logging functions.

> 
> > +/* Poll internal in microseconds */
> > +#define POLL_INTERVAL                  10
> > +
> > +static int btintel_pcie_poll_bit(struct btintel_pcie_data *data, u32 offset,
> > +                                u32 bits, u32 mask, int timeout)
> > +{
> > +       int t = 0;
> > +       u32 reg;
> > +
> > +       BT_DBG("Enter poll_bit");
> > +       do {
> > +               reg = btintel_pcie_rd_reg32(data, offset);
> > +               BT_DBG("CURRENT FUNC_CTRL_REG: 0x%x", reg);
> > +
> > +               if ((reg & mask) == (bits & mask)) {
> > +                       BT_DBG("Poll bit matched");
> > +                       return t;
> > +               }
> > +               udelay(POLL_INTERVAL);
> > +               t += POLL_INTERVAL;
> > +               BT_DBG("Poll wait: %d", t);
> > +       } while (t < timeout);
> 
> Way too much debugging for such a trivial function, I'd actually
> remove all calls to BT_DBG here.
Ack.

> 
> > +       return -ETIMEDOUT;
> > +}
> > +
> > +static struct btintel_pcie_data *btintel_pcie_get_data(struct msix_entry
> *entry)
> > +{
> > +       u8 queue = entry->entry;
> > +       struct msix_entry *entries = entry - queue;
> > +
> > +       return container_of(entries, struct btintel_pcie_data, msix_entries[0]);
> > +}
> > +
> > +/* Set the doorbell for RXQ to notify the device that @index(actually index-
> 1)
> > + * is available to receive the data
> > + */
> > +static void btintel_pcie_set_rx_db(struct btintel_pcie_data *data, u16
> index)
> > +{
> > +       u32 val;
> > +
> > +       val = index;
> > +       val |= (513 << 16);
> > +
> > +       BT_DBG("[RXQ] Set doorbell for index: %u", index);
> > +       btintel_pcie_wr_reg32(data, CSR_HBUS_TARG_WRPTR, val);
> > +}
> > +
> > +/* Update the FRBD(free buffer descriptor) with the @frbd_index and the
> > + * DMA address of the free buffer.
> > + */
> > +static void btintel_pcie_prepare_rx(struct rxq *rxq, u16 frbd_index)
> > +{
> > +       struct data_buf *buf;
> > +       struct frbd *frbd;
> > +
> > +       /* Get the buffer of the frbd for DMA */
> > +       buf = &rxq->bufs[frbd_index];
> > +
> > +       frbd = &rxq->frbds[frbd_index];
> > +       memset(frbd, 0, sizeof(*frbd));
> > +
> > +       /* Update FRBD */
> > +       frbd->tag = frbd_index;
> > +       frbd->addr = buf->data_p_addr;
> > +       ipc_print_frbd(frbd, frbd_index);
> > +}
> > +
> > +static int btintel_pcie_submit_rx(struct btintel_pcie_data *data)
> > +{
> > +       u16 frbd_index;
> > +       struct rxq *rxq = &data->rxq;
> > +
> > +       /* Read the frbd index from the TR_HIA(Head Index Array) for RXQ */
> > +       frbd_index = data->ia.tr_hia[RXQ_NUM];
> > +       BT_DBG("[RXQ] current frbd_index: %u", frbd_index);
> > +
> > +       /* Make sure the index value is within the range. It shouldn't be
> > +        * bigger than the total count of the queue.
> > +        */
> > +       if (frbd_index > rxq->count) {
> > +               BT_ERR("[RXQ] RXQ out of range: (0x%x)", frbd_index);
> > +               return -ERANGE;
> > +       }
> > +
> > +       /* Prepare for RX submit. It updates the FRBD with the address of DMA
> > +        * buffer
> > +        */
> > +       btintel_pcie_prepare_rx(rxq, frbd_index);
> > +
> > +       /* Update TR_HIA with new FRBD index */
> > +       frbd_index = (frbd_index + 1) % rxq->count;
> > +       data->ia.tr_hia[RXQ_NUM] = frbd_index;
> > +       ipc_print_ia_ring(&data->ia, RXQ_NUM);
> > +
> > +       /* Set the doorbell to notify the device */
> > +       btintel_pcie_set_rx_db(data, frbd_index);
> > +
> > +       BT_DBG("[RXQ] rx sumbit completed");
> > +
> > +       return 0;
> > +}
> > +
> > +static int btintel_pcie_start_rx(struct btintel_pcie_data *data)
> > +{
> > +       int i, ret;
> > +
> > +       for (i = 0; i < RX_MAX_QUEUE; i++) {
> > +               ret = btintel_pcie_submit_rx(data);
> > +               if (ret) {
> > +                       BT_ERR("[RXQ] failed to submit frbd(%d)", ret);
> > +                       return ret;
> > +               }
> > +       }
> > +
> > +       return 0;
> > +}
> > +
> > +static void btintel_pcie_reset_ia(struct btintel_pcie_data *data)
> > +{
> > +       memset(data->ia.tr_hia, 0, sizeof(u16) * NUM_QUEUES);
> > +       memset(data->ia.tr_tia, 0, sizeof(u16) * NUM_QUEUES);
> > +       memset(data->ia.cr_hia, 0, sizeof(u16) * NUM_QUEUES);
> > +       memset(data->ia.cr_tia, 0, sizeof(u16) * NUM_QUEUES);
> > +       BT_DBG("Index Arrays are reset");
> > +}
> > +
> > +static void btintel_pcie_reset_bt(struct btintel_pcie_data *data)
> > +{
> > +       BT_INFO("Reset BT Function ");
> > +       btintel_pcie_wr_reg32(data, CSR_FUNC_CTRL_REG,
> CSR_FUNC_CTRL_SW_RESET);
> > +}
> > +
> > +/* This function enables BT function by setting CSR_FUNC_CTRL_MAC_INIT
> bit in
> > + * CSR_FUNC_CTRL_REG register and wait for MSI-X with
> MSIX_HW_INT_CAUSES_GP0.
> > + * Then the host reads firmware version from CSR_F2D_MBX and the boot
> stage
> > + * from CSR_BOOT_STAGE_REG.
> > + */
> > +static int btintel_pcie_enable_bt(struct btintel_pcie_data *data)
> > +{
> > +       int err;
> > +       u32 reg;
> > +
> > +       data->gp0_received = false;
> > +
> > +       /* Update the DMA address of CI struct to CSR */
> > +       btintel_pcie_wr_reg32(data, CSR_CI_ADDR_LSB_REG,
> > +                             data->ci_p_addr & 0xffffffff);
> > +       btintel_pcie_wr_reg32(data, CSR_CI_ADDR_MSB_REG,
> > +                             data->ci_p_addr >> 32);
> > +
> > +       /* Reset the cached value of boot stage. it is updated by the msix
> > +        * gp0 interrupt handler.
> > +        */
> > +       data->boot_stage_cache = 0x0;
> > +
> > +       /* Set MAC_INIT bit to start primary bootloader */
> > +       reg = btintel_pcie_rd_reg32(data, CSR_FUNC_CTRL_REG);
> > +       BT_INFO("Before: FUNC_CTRL_REG: 0x%x", reg);
> > +
> > +       btintel_pcie_set_reg_bits(data, CSR_FUNC_CTRL_REG,
> > +                                 CSR_FUNC_CTRL_MAC_INIT);
> > +       BT_INFO("MAC_INIT is set");
> 
> It is useless to printing this sort of information, let alone using BT_INFO...
Ack.

> 
> > +       /* Wait until MAC_ACCESS is granted */
> > +       err = btintel_pcie_poll_bit(data, CSR_FUNC_CTRL_REG,
> > +                                   CSR_FUNC_CTRL_MAC_ACCESS_STS,
> > +                                   CSR_FUNC_CTRL_MAC_ACCESS_STS,
> > +                                   DEFAULT_MAC_ACCESS_TIMEOUT);
> > +       if (err < 0) {
> > +               BT_ERR("Failed to start bootloader even after %u ns",
> > +                      DEFAULT_MAC_ACCESS_TIMEOUT);
> > +               return -ENODEV;
> > +       }
> > +
> > +       /* MAC is ready. Enable BT FUNC */
> > +       btintel_pcie_set_reg_bits(data, CSR_FUNC_CTRL_REG,
> > +                                 CSR_FUNC_CTRL_FUNC_ENA |
> > +                                 CSR_FUNC_CTRL_FUNC_INIT);
> > +
> > +       reg = btintel_pcie_rd_reg32(data, CSR_FUNC_CTRL_REG);
> > +       BT_INFO("After: FUNC_CTRL_REG: 0x%x", reg);
> 
> Ditto.
Ack.

> 
> > +       /* wait for interrupt from the device after booting up to primary
> > +        * bootloader.
> > +        */
> > +       err = wait_event_timeout(data->gp0_wait_q, data->gp0_received,
> > +                                msecs_to_jiffies(DEFAULT_INTR_TIMEOUT));
> > +       if (!err) {
> > +               BT_ERR("Failed to receive mac_init interrupt");
> > +               return -ETIME;
> > +       }
> > +
> > +       /* Check cached boot stage is CSR_BOOT_STAGE_ROM(BIT(0)) */
> > +       if (~data->boot_stage_cache & CSR_BOOT_STAGE_ROM) {
> > +               BT_ERR("Device is not running in rom");
> > +               return -ENODEV;
> > +       }
> > +
> > +       return 0;
> > +}
> > +
> > +/* This function handles the MSI-X interrupt for gp0 cause(bit 0 in
> > + * CSR_MSIX_HW_INT_CAUSES) which is sent for boot stage and image
> response.
> > + */
> > +static void btintel_pcie_msix_gp0_handler(struct btintel_pcie_data *data)
> > +{
> > +       u32 reg;
> > +
> > +       /* This interrupt is for three different causes and it is not easy to
> > +        * know what causes the interrupt. So, it compares each register value
> > +        * with cached value and update it before it wake up the queue.
> > +        */
> > +       reg = btintel_pcie_rd_reg32(data, CSR_BOOT_STAGE_REG);
> > +       if (reg != data->boot_stage_cache) {
> > +               data->boot_stage_cache = reg;
> > +
> > +               BT_DBG("Boot Stage updated: 0x%x", reg);
> > +       }
> > +
> > +       reg = btintel_pcie_rd_reg32(data, CSR_IMG_RESPONSE_REG);
> > +       if (reg != data->img_resp_cache) {
> > +               data->img_resp_cache = reg;
> > +
> > +               BT_DBG("Image Response updated: 0x%x", reg);
> > +       }
> > +
> > +       BT_DBG("---------- cached GP0 registers ----------");
> > +       BT_DBG("Cached Boot Stage Reg: 0x%x", data->boot_stage_cache);
> > +       BT_DBG("Cached Image Resp Reg: 0x%x", data->img_resp_cache);
> > +
> > +       data->gp0_received = true;
> > +
> > +       /* If the boot stage is OP or IML, reset IA and start RX again */
> > +       if (data->boot_stage_cache & CSR_BOOT_STAGE_OPFW ||
> > +           data->boot_stage_cache & CSR_BOOT_STAGE_IML) {
> > +               btintel_pcie_reset_ia(data);
> > +               btintel_pcie_start_rx(data);
> > +       }
> > +
> > +       wake_up(&data->gp0_wait_q);
> > +}
> > +
> > +/* This function handles the MSX-X interrupt for rx queue 0 which is for TX
> > + */
> > +static void btintel_pcie_msix_tx_handle(struct btintel_pcie_data *data)
> > +{
> > +       u16 cr_tia, cr_hia;
> > +       struct txq *txq;
> > +       struct urbd0 *urbd0;
> > +
> > +       cr_tia = data->ia.cr_tia[TXQ_NUM];
> > +       cr_hia = data->ia.cr_hia[TXQ_NUM];
> > +
> > +       BT_DBG("[TXQ] cr_hia=%u  cr_tia=%u", cr_hia, cr_tia);
> > +
> > +       /* Check CR_TIA and CR_HIA for change */
> > +       if (cr_tia == cr_hia) {
> > +               BT_ERR("[TXQ] no new CD found");
> > +               return;
> > +       }
> > +
> > +       txq = &data->txq;
> > +
> > +       while (cr_tia != cr_hia) {
> > +               BT_DBG("[TXQ] wake up tx_wait_q");
> > +
> > +               data->tx_wait_done = true;
> > +               wake_up(&data->tx_wait_q);
> > +
> > +               /* Get URBD0 pointed by cr_tia */
> > +               urbd0 = &txq->urbd0s[cr_tia];
> > +               ipc_print_urbd0(urbd0, cr_tia);
> > +
> > +               /* Make sure the completed TFD index is within the range */
> > +               if (urbd0->tfd_index > txq->count) {
> > +                       BT_ERR("[TXQ] out of range: (0x%x)", urbd0->tfd_index);
> > +                       return;
> > +               }
> > +
> > +               /* Increase cr_tia */
> > +               cr_tia = (cr_tia + 1) % txq->count;
> > +               data->ia.cr_tia[TXQ_NUM] = cr_tia;
> > +               ipc_print_ia_ring(&data->ia, TXQ_NUM);
> > +       }
> > +}
> > +
> > +static int btintel_pcie_recv_event_intel(struct hci_dev *hdev,
> > +                                        struct sk_buff *skb)
> > +{
> > +       if (btintel_test_flag(hdev, INTEL_BOOTLOADER)) {
> > +               struct hci_event_hdr *hdr = (void *)skb->data;
> > +
> > +               if (skb->len > HCI_EVENT_HDR_SIZE && hdr->evt == 0xff &&
> > +                   hdr->plen > 0) {
> > +                       const void *ptr = skb->data + HCI_EVENT_HDR_SIZE + 1;
> > +                       unsigned int len = skb->len - HCI_EVENT_HDR_SIZE - 1;
> > +
> > +                       switch (skb->data[2]) {
> > +                       case 0x02:
> > +                               /* When switching to the operational firmware
> > +                                * the device sends a vendor specific event
> > +                                * indicating that the bootup completed.
> > +                                */
> > +                               btintel_bootup(hdev, ptr, len);
> > +                               break;
> > +                       case 0x06:
> > +                               /* When the firmware loading completes the
> > +                                * device sends out a vendor specific event
> > +                                * indicating the result of the firmware
> > +                                * loading.
> > +                                */
> > +                               btintel_secure_send_result(hdev, ptr, len);
> > +                               break;
> > +                       }
> > +               }
> > +       }
> > +
> > +       return hci_recv_frame(hdev, skb);
> > +}
> 
> Replace the above with btintel_recv_event?
Ack.

> 
> > +/* Process the received rx data
> > + * It check the frame header to identify the data type and create skb
> > + * and calling HCI API
> > + */
> > +static int btintel_pcie_hci_recv_frame(struct btintel_pcie_data *data,
> > +                                      void *buf, int count)
> > +{
> > +       struct hci_dev *hdev = data->hdev;
> > +       int ret;
> > +       u32 pkt_type;
> > +       u16 plen;
> > +       struct sk_buff *skb;
> > +
> > +       spin_lock(&data->hci_rx_lock);
> > +
> > +       /* The first 4 bytes indicates the Intel PCIe specific packet type.
> > +        * Read the packet type here before remove it.
> > +        */
> > +       pkt_type = get_unaligned_le32(buf);
> > +       bt_dev_dbg(hdev, "pkt_type=%u count=%d", pkt_type, count);
> > +
> > +       buf += BTINTEL_PCIE_HCI_TYPE_LEN;
> > +       count -= BTINTEL_PCIE_HCI_TYPE_LEN;
> > +
> > +       hdev->stat.byte_rx += count;
> > +
> > +       skb = bt_skb_alloc(count, GFP_ATOMIC);
> > +       if (!skb) {
> > +               bt_dev_err(hdev, "Failed to allocate skb for event");
> > +               ret = -ENOMEM;
> > +               goto exit_error;
> > +       }
> > +
> > +       switch (pkt_type) {
> > +       case BTINTEL_PCIE_HCI_ACL_PKT:
> > +               hci_skb_pkt_type(skb) = HCI_ACLDATA_PKT;
> > +               memcpy(skb_put(skb, HCI_ACL_HDR_SIZE), buf,
> HCI_ACL_HDR_SIZE);
> > +               plen = hci_acl_hdr(skb)->dlen;
> > +               buf += HCI_ACL_HDR_SIZE;
> > +               break;
> > +       case BTINTEL_PCIE_HCI_SCO_PKT:
> > +               hci_skb_pkt_type(skb) = HCI_SCODATA_PKT;
> > +               memcpy(skb_put(skb, HCI_SCO_HDR_SIZE), buf,
> HCI_SCO_HDR_SIZE);
> > +               plen = hci_sco_hdr(skb)->dlen;
> > +               buf += HCI_SCO_HDR_SIZE;
> > +               break;
> > +       case BTINTEL_PCIE_HCI_EVT_PKT:
> > +               hci_skb_pkt_type(skb) = HCI_EVENT_PKT;
> > +               memcpy(skb_put(skb, HCI_EVENT_HDR_SIZE), buf,
> > +                      HCI_EVENT_HDR_SIZE);
> > +               plen = hci_event_hdr(skb)->plen;
> > +               buf += HCI_EVENT_HDR_SIZE;
> > +               break;
> > +       default:
> > +               ret = -EILSEQ;
> > +               kfree_skb(skb);
> > +               goto exit_error;
> > +       }
> > +       memcpy(skb_put(skb, plen), buf, plen);
> 
> This is very unsafe, to say the least, since we are not checking if
> buf actually contains all the data since we never seem to use the
> count parameter.
Ack.
> 
> > +
> > +       if (pkt_type == BTINTEL_PCIE_HCI_EVT_PKT)
> > +               ret = btintel_pcie_recv_event_intel(hdev, skb);
> > +       else
> > +               ret = hci_recv_frame(hdev, skb);
> > +
> > +exit_error:
> > +       if (ret)
> > +               hdev->stat.err_rx++;
> > +
> > +       spin_unlock(&data->hci_rx_lock);
> > +
> > +       return ret;
> > +}
> 
> Hmm, this doesn't sound quite right compared to the likes of
> h4_recv_buf, are we guaranteed to only receive complete packets for
> example?
Yes. Host receives always complete packets. I will refactor this function to take care of spurious packets.
> 
> > +/* RX work queue */
> > +static void btintel_pcie_rx_work(struct work_struct *work)
> > +{
> > +       struct btintel_pcie_data *data = container_of(work,
> > +                                       struct btintel_pcie_data, rx_work);
> > +       struct sk_buff *skb;
> > +       int err;
> > +
> > +       /* Process the sk_buf in queue and send to the hci layer */
> > +       while ((skb = skb_dequeue(&data->rx_skb_q))) {
> > +               err = btintel_pcie_hci_recv_frame(data, skb->data, skb->len);
> 
> If the data comes as skb then pass the skb and not just the data, len,
> that way we can actually make use of things like skb_pull_data to
> advance in the buffer, etc.
Ack.

> 
> > +               if (err) {
> > +                       BT_ERR("Failed to send received frame: %d", err);
> > +                       kfree_skb(skb);
> > +               }
> > +       }
> > +}
> > +
> > +/* create the sk_buff with data and save it to queue and start rx work
> > + */
> > +static int btintel_pcie_submit_rx_work(struct btintel_pcie_data *data, u8
> status,
> > +                                      void *buf)
> > +{
> > +       int ret, len;
> > +       struct rfh_hdr *rfh_hdr;
> > +       struct sk_buff *skb;
> > +
> > +       rfh_hdr = (struct rfh_hdr *)buf;
> > +       btintel_pcie_hci_dump("RFH HDR", buf, sizeof(*rfh_hdr));
> > +
> > +       len = rfh_hdr->packet_len;
> > +
> > +       /* Remove RFH header */
> > +       buf += sizeof(*rfh_hdr);
> > +       btintel_pcie_hci_dump("RX", buf, len);
> > +
> > +       /* Create the sk_buf with packet in the buf and save it to sk_buf queue
> > +        */
> > +       skb = alloc_skb(len, GFP_ATOMIC);
> > +       if (!skb) {
> > +               ret = -ENOMEM;
> > +               goto resubmit;
> > +       }
> > +
> > +       /* Copy the data to skb */
> > +       memcpy(skb_put(skb, len), buf, len);
> > +
> > +       /* Save the skb to rx queue */
> > +       skb_queue_tail(&data->rx_skb_q, skb);
> > +
> > +       /* Calling rx_work queue to process the skb */
> > +       queue_work(data->workqueue, &data->rx_work);
> > +
> > +resubmit:
> > +       BT_DBG("submit next read request");
> > +
> > +       /* submit read */
> > +       ret = btintel_pcie_submit_rx(data);
> > +
> > +       return ret;
> > +}
> > +
> > +/* This function handles the MSI-X interrupt for rx queue 1 which is for RX
> > + */
> > +static void btintel_pcie_msix_rx_handle(struct btintel_pcie_data *data)
> > +{
> > +       u16 cr_hia, cr_tia;
> > +       struct rxq *rxq;
> > +       struct urbd1 *urbd1;
> > +       struct frbd *frbd;
> > +       struct data_buf *buf;
> > +       int ret;
> > +
> > +       cr_hia = data->ia.cr_hia[RXQ_NUM];
> > +       cr_tia = data->ia.cr_tia[RXQ_NUM];
> > +
> > +       BT_DBG("[RXQ] cr_hia=%u  cr_tia=%u", cr_hia, cr_tia);
> > +
> > +       /* Check CR_TIA and CR_HIA for change */
> > +       if (cr_tia == cr_hia) {
> > +               BT_ERR("[RXQ] no new CD found");
> > +               return;
> > +       }
> > +
> > +       rxq = &data->rxq;
> > +
> > +       /* The firmware sends multiple CD in a single MSIX and it needs to
> > +        * process all received CDs in this interrupt.
> > +        */
> > +       while (cr_tia != cr_hia) {
> > +               /* Get URBD1 pointed by cr_tia */
> > +               urbd1 = &rxq->urbd1s[cr_tia];
> > +               ipc_print_urbd1(urbd1, cr_tia);
> > +
> > +               /* Get FRBD poined by urbd1->frbd_tag */
> > +               frbd = &rxq->frbds[urbd1->frbd_tag];
> > +
> > +               /* Get buf from FRBD tag */
> > +               buf = &rxq->bufs[urbd1->frbd_tag];
> > +               if (!buf) {
> > +                       BT_ERR("[RXQ] failed to get the DMA buffer for %d",
> > +                              urbd1->frbd_tag);
> > +                       return;
> > +               }
> > +
> > +               /* prepare RX work */
> > +               ret = btintel_pcie_submit_rx_work(data, urbd1->status,
> > +                                                 buf->data);
> > +               if (ret) {
> > +                       BT_ERR("[RXQ] failed to submit rx request");
> > +                       return;
> > +               }
> > +
> > +               /* Update cr_tia */
> > +               cr_tia = (cr_tia + 1) % rxq->count;
> > +               data->ia.cr_tia[RXQ_NUM] = cr_tia;
> > +               ipc_print_ia_ring(&data->ia, RXQ_NUM);
> > +       }
> > +       BT_DBG("[RXQ] completed rx interrupt");
> > +}
> > +
> > +static irqreturn_t btintel_pcie_msix_isr(int irq, void *data)
> > +{
> > +       return IRQ_WAKE_THREAD;
> > +}
> > +
> > +static irqreturn_t btintel_pcie_irq_msix_handler(int irq, void *dev_id)
> > +{
> > +       struct msix_entry *entry = dev_id;
> > +       struct btintel_pcie_data *data = btintel_pcie_get_data(entry);
> > +       u32 intr_fh, intr_hw;
> > +
> > +       BT_DBG("handling msix(irq=%d dev_id=0x%p)", irq, dev_id);
> > +
> > +       spin_lock(&data->irq_lock);
> > +       intr_fh = btintel_pcie_rd_reg32(data, CSR_MSIX_FH_INT_CAUSES);
> > +       intr_hw = btintel_pcie_rd_reg32(data, CSR_MSIX_HW_INT_CAUSES);
> > +
> > +       /* Clear causes registers to avoid being handling the same cause */
> > +       btintel_pcie_wr_reg32(data, CSR_MSIX_FH_INT_CAUSES, intr_fh);
> > +       btintel_pcie_wr_reg32(data, CSR_MSIX_HW_INT_CAUSES, intr_hw);
> > +       spin_unlock(&data->irq_lock);
> > +
> > +       BT_DBG("intr_fh=0x%x intr_hw=0x%x", intr_fh, intr_hw);
> > +
> > +       if (unlikely(!(intr_fh | intr_hw))) {
> > +               BT_DBG("Ignore interrupt, inta == 0");
> > +               return IRQ_NONE;
> > +       }
> > +
> > +       /* This interrupt is triggered by the firmware after updating
> > +        * boot_stage register and image_response register
> > +        */
> > +       if (intr_hw & MSIX_HW_INT_CAUSES_GP0) {
> > +               BT_DBG("intr for MSIX_HW_INT_CAUSES_GP0");
> > +               btintel_pcie_msix_gp0_handler(data);
> > +       }
> > +
> > +       /* For TX */
> > +       if (intr_fh & MSIX_FH_INT_CAUSES_0) {
> > +               BT_DBG("intr for MSIX_FH_INT_CAUSES_0");
> > +               btintel_pcie_msix_tx_handle(data);
> > +       }
> > +
> > +       /* For RX */
> > +       if (intr_fh & MSIX_FH_INT_CAUSES_1) {
> > +               BT_DBG("intr for MSIX_FH_INT_CAUSES_1");
> > +               btintel_pcie_msix_rx_handle(data);
> > +       }
> > +
> > +       /* TODO: Add handler for other causes */
> > +       /*
> > +        * Before sending the interrupt the HW disables it to prevent
> > +        * a nested interrupt. This is done by writing 1 to the corresponding
> > +        * bit in the mask register. After handling the interrupt, it should be
> > +        * re-enabled by clearing this bit. This register is defined as
> > +        * write 1 clear (W1C) register, meaning that it's being clear
> > +        * by writing 1 to the bit.
> > +        */
> > +       btintel_pcie_wr_reg32(data, CSR_MSIX_AUTOMASK_ST, BIT(entry-
> >entry));
> > +
> > +       return IRQ_HANDLED;
> > +}
> > +
> > +/* This function requests the irq for msix and registers the handlers per irq.
> > + * Currently, it requests only 1 irq for all interrupt causes.
> > + */
> > +static int btintel_pcie_setup_irq(struct btintel_pcie_data *data)
> > +{
> > +       int err;
> > +       int num_irqs, i;
> > +
> > +       BT_DBG("Initialize msix_entries...");
> > +       for (i = 0; i < MSIX_VEC_MAX; i++) {
> > +               data->msix_entries[i].entry = i;
> > +               BT_DBG("msix_entries[%d] vector=0x%x entry=0x%x",
> > +                      i, data->msix_entries[i].vector,
> > +                      data->msix_entries[i].entry);
> > +       }
> > +
> > +       num_irqs = pci_enable_msix_range(data->pdev, data->msix_entries,
> > +                                        MSIX_VEC_MIN,
> > +                                        MSIX_VEC_MAX);
> > +       if (num_irqs < 0) {
> > +               BT_ERR("Failed to enable msix range (%d)", num_irqs);
> > +               return num_irqs;
> > +       }
> > +
> > +       data->alloc_vecs = num_irqs;
> > +       data->msix_enabled = 1;
> > +       data->def_irq = 0;
> > +
> > +       BT_DBG("Returned num_irqs=%d", num_irqs);
> > +       for (i = 0; i < num_irqs; i++) {
> > +               BT_DBG("msix_entries[%d] vector=0x%x entry=0x%x", i,
> > +                      data->msix_entries[i].vector,
> > +                      data->msix_entries[i].entry);
> > +       }
> > +
> > +       BT_DBG("setup irq handler");
> > +       for (i = 0; i < data->alloc_vecs; i++) {
> > +               struct msix_entry *msix_entry;
> > +
> > +               msix_entry = &data->msix_entries[i];
> > +
> > +               err = devm_request_threaded_irq(&data->pdev->dev,
> > +                                               msix_entry->vector,
> > +                                               btintel_pcie_msix_isr,
> > +                                               btintel_pcie_irq_msix_handler,
> > +                                               IRQF_SHARED,
> > +                                               KBUILD_MODNAME,
> > +                                               msix_entry);
> > +               if (err) {
> > +                       BT_ERR("Failed to allocate irq handler (%d)", err);
> > +                       return err;
> > +               }
> > +       }
> > +
> > +       return 0;
> > +}
> > +
> > +struct btintel_pcie_causes_list {
> > +       u32 cause;
> > +       u32 mask_reg;
> > +       u8 cause_num;
> > +};
> > +
> > +struct btintel_pcie_causes_list causes_list[] = {
> > +       { MSIX_FH_INT_CAUSES_0,         CSR_MSIX_FH_INT_MASK,   0x00 },
> > +       { MSIX_FH_INT_CAUSES_1,         CSR_MSIX_FH_INT_MASK,   0x01 },
> > +       { MSIX_HW_INT_CAUSES_GP0,       CSR_MSIX_HW_INT_MASK,   0x20 },
> > +};
> > +
> > +/* This function configures the interrupt masks for both HW_INT_CAUSES
> and
> > + * FH_INT_CAUSES which are meaningful to us.
> > + *
> > + * After resetting BT function via PCIE FLR or FUNC_CTRL reset, the driver
> > + * need to call this function again to configure it again since the masks
> > + * are reset to 0xFFFFFFFF after reset.
> > + */
> > +static void btintel_pcie_config_msix(struct btintel_pcie_data *data)
> > +{
> > +       int i;
> > +       int val = data->def_irq | MSIX_NON_AUTO_CLEAR_CAUSE;
> > +
> > +       /* Set Non Auto Clear Cause */
> > +       for (i = 0; i < ARRAY_SIZE(causes_list); i++) {
> > +               btintel_pcie_wr_reg8(data,
> > +                                    CSR_MSIX_IVAR(causes_list[i].cause_num),
> > +                                    val);
> > +               btintel_pcie_clr_reg_bits(data,
> > +                                         causes_list[i].mask_reg,
> > +                                         causes_list[i].cause);
> > +       }
> > +
> > +       /* Save the initial interrupt mask */
> > +       data->fh_init_mask = ~btintel_pcie_rd_reg32(data,
> CSR_MSIX_FH_INT_MASK);
> > +       data->hw_init_mask = ~btintel_pcie_rd_reg32(data,
> CSR_MSIX_HW_INT_MASK);
> > +       BT_DBG("init_mask: fh=0x%x hw=0x%x", data->fh_init_mask,
> > +              data->hw_init_mask);
> > +}
> > +
> > +static int btintel_pcie_config_pcie(struct pci_dev *pdev,
> > +                                   struct btintel_pcie_data *data)
> > +{
> > +       int err;
> > +
> > +       err = pcim_enable_device(pdev);
> > +       if (err) {
> > +               BT_ERR("Failed to enable pci device (%d)", err);
> > +               return err;
> > +       }
> > +       pci_set_master(pdev);
> > +
> > +       /* Setup DMA mask */
> > +       BT_DBG("Set DMA_MASK(64)");
> > +       err = dma_set_mask_and_coherent(&pdev->dev,
> DMA_BIT_MASK(64));
> > +       if (err) {
> > +               BT_DBG("Set DMA_MASK(32)");
> > +               err = dma_set_mask_and_coherent(&pdev->dev,
> DMA_BIT_MASK(32));
> > +               /* Both attempt failed */
> > +               if (err) {
> > +                       BT_ERR("No suitable DMA available");
> > +                       return err;
> > +               }
> > +       }
> > +
> > +       /* Get BAR to access CSR */
> > +       err = pcim_iomap_regions(pdev, BIT(0), KBUILD_MODNAME);
> > +       if (err) {
> > +               BT_ERR("Failed to get iomap regions (%d)", err);
> > +               return err;
> > +       }
> > +
> > +       data->base_addr = pcim_iomap_table(pdev)[0];
> > +       if (!data->base_addr) {
> > +               BT_ERR("Failed to get base address");
> > +               return -ENODEV;
> > +       }
> > +
> > +       err = btintel_pcie_setup_irq(data);
> > +       if (err) {
> > +               BT_ERR("Failed to setup irq for msix");
> > +               return err;
> > +       }
> > +
> > +       /* Configure MSI-X with causes list */
> > +       btintel_pcie_config_msix(data);
> > +
> > +       return 0;
> > +}
> > +
> > +static void btintel_pcie_init_ci(struct btintel_pcie_data *data,
> > +                                struct ctx_info *ci)
> > +{
> > +       ci->version = 0x1;
> > +       ci->size = sizeof(*ci);
> > +       ci->config = 0x0000;
> > +       ci->addr_cr_hia = data->ia.cr_hia_p_addr;
> > +       ci->addr_tr_tia = data->ia.tr_tia_p_addr;
> > +       ci->addr_cr_tia = data->ia.cr_tia_p_addr;
> > +       ci->addr_tr_hia = data->ia.tr_hia_p_addr;
> > +       ci->num_cr_ia = NUM_QUEUES;
> > +       ci->num_tr_ia = NUM_QUEUES;
> > +       ci->addr_urbdq0 = data->txq.urbd0s_p_addr;
> > +       ci->addr_tfdq = data->txq.tfds_p_addr;
> > +       ci->num_tfdq = data->txq.count;
> > +       ci->num_urbdq0 = data->txq.count;
> > +       ci->tfdq_db_vec = TXQ_NUM;
> > +       ci->urbdq0_db_vec = TXQ_NUM;
> > +       ci->rbd_size = RBD_SIZE_4K;
> > +       ci->addr_frbdq = data->rxq.frbds_p_addr;
> > +       ci->num_frbdq = data->rxq.count;
> > +       ci->frbdq_db_vec = RXQ_NUM;
> > +       ci->addr_urbdq1 = data->rxq.urbd1s_p_addr;
> > +       ci->num_urbdq1 = data->rxq.count;
> > +       ci->urbdq_db_vec = RXQ_NUM;
> > +}
> > +
> > +static void btintel_pcie_free_txq_bufs(struct btintel_pcie_data *data,
> > +                                      struct txq *txq)
> > +{
> > +       /* Free data buffers first */
> > +       dma_free_coherent(&data->pdev->dev, txq->count * BUFFER_SIZE,
> > +                         txq->buf_v_addr, txq->buf_p_addr);
> > +       kfree(txq->bufs);
> > +       BT_DBG("txq buffers are freed");
> > +}
> > +
> > +static int btintel_pcie_setup_txq_bufs(struct btintel_pcie_data *data,
> > +                                      struct txq *txq)
> > +{
> > +       int err = 0, i;
> > +       struct data_buf *buf;
> > +
> > +       if (txq->count == 0) {
> > +               BT_ERR("invalid parameter: txq->count");
> > +               err = -EINVAL;
> > +               goto exit_error;
> > +       }
> > +
> > +       /* Allocate the same number of buffers as the descriptor */
> > +       txq->bufs = kmalloc_array(txq->count, sizeof(*buf), GFP_KERNEL);
> > +       if (!txq->bufs) {
> > +               err = -ENOMEM;
> > +               goto exit_error;
> > +       }
> > +
> > +       /* Allocate full chunk of data buffer for DMA first and do indexing and
> > +        * initialization next, so it can be freed easily
> > +        */
> > +       txq->buf_v_addr = dma_alloc_coherent(&data->pdev->dev,
> > +                                            txq->count * BUFFER_SIZE,
> > +                                            &txq->buf_p_addr,
> > +                                            GFP_KERNEL | __GFP_NOWARN);
> > +       if (!txq->buf_v_addr) {
> > +               BT_ERR("Failed to allocate DMA buf");
> > +               err = -ENOMEM;
> > +               kfree(txq->bufs);
> > +               goto exit_error;
> > +       }
> > +       memset(txq->buf_v_addr, 0, txq->count * BUFFER_SIZE);
> > +
> > +       BT_DBG("alloc bufs: p=0x%llx v=0x%p", txq->buf_p_addr, txq-
> >buf_v_addr);
> > +
> > +       /* Setup the allocated DMA buffer to bufs. Each data_buf should
> > +        * have virtual address and physical address
> > +        */
> > +       for (i = 0; i < txq->count; i++) {
> > +               buf = &txq->bufs[i];
> > +               buf->data_p_addr = txq->buf_p_addr + (i * BUFFER_SIZE);
> > +               buf->data = txq->buf_v_addr + (i * BUFFER_SIZE);
> > +       }
> > +
> > +exit_error:
> > +       return err;
> > +}
> > +
> > +static void btintel_pcie_free_rxq_bufs(struct btintel_pcie_data *data,
> > +                                      struct rxq *rxq)
> > +{
> > +       /* Free data buffers first */
> > +       dma_free_coherent(&data->pdev->dev, rxq->count * BUFFER_SIZE,
> > +                         rxq->buf_v_addr, rxq->buf_p_addr);
> > +       kfree(rxq->bufs);
> > +       BT_DBG("rxq buffers are freed");
> > +}
> > +
> > +static int btintel_pcie_setup_rxq_bufs(struct btintel_pcie_data *data,
> > +                                      struct rxq *rxq)
> > +{
> > +       int err = 0, i;
> > +       struct data_buf *buf;
> > +
> > +       if (rxq->count == 0) {
> > +               BT_ERR("invalid parameter: rxq->count");
> > +               err = -EINVAL;
> > +               goto exit_error;
> > +       }
> > +
> > +       /* Allocate the same number of buffers as the descriptor */
> > +       rxq->bufs = kmalloc_array(rxq->count, sizeof(*buf), GFP_KERNEL);
> > +       if (!rxq->bufs) {
> > +               err = -ENOMEM;
> > +               goto exit_error;
> > +       }
> > +
> > +       /* Allocate full chunk of data buffer for DMA first and do indexing and
> > +        * initialization next, so it can be freed easily
> > +        */
> > +       rxq->buf_v_addr = dma_alloc_coherent(&data->pdev->dev,
> > +                                            rxq->count * BUFFER_SIZE,
> > +                                            &rxq->buf_p_addr,
> > +                                            GFP_KERNEL | __GFP_NOWARN);
> > +       if (!rxq->buf_v_addr) {
> > +               BT_ERR("Failed to allocate DMA buf");
> > +               err = -ENOMEM;
> > +               kfree(rxq->bufs);
> > +               goto exit_error;
> > +       }
> > +       memset(rxq->buf_v_addr, 0, rxq->count * BUFFER_SIZE);
> > +
> > +       BT_DBG("alloc bufs: p=0x%llx v=0x%p", rxq->buf_p_addr, rxq-
> >buf_v_addr);
> > +
> > +       /* Setup the allocated DMA buffer to bufs. Each data_buf should
> > +        * have virtual address and physical address
> > +        */
> > +       for (i = 0; i < rxq->count; i++) {
> > +               buf = &rxq->bufs[i];
> > +               buf->data_p_addr = rxq->buf_p_addr + (i * BUFFER_SIZE);
> > +               buf->data = rxq->buf_v_addr + (i * BUFFER_SIZE);
> > +       }
> > +
> > +exit_error:
> > +
> > +       return err;
> > +}
> > +
> > +static void btintel_pcie_setup_ia(struct btintel_pcie_data *data,
> > +                                 dma_addr_t p_addr, void *v_addr,
> > +                                 struct ia *ia)
> > +{
> > +       /* TR Head Index Array */
> > +       ia->tr_hia_p_addr = p_addr;
> > +       ia->tr_hia = v_addr;
> > +
> > +       /* TR Tail Index Array */
> > +       ia->tr_tia_p_addr = p_addr + sizeof(u16) * NUM_QUEUES;
> > +       ia->tr_tia = v_addr + sizeof(u16) * NUM_QUEUES;
> > +
> > +       /* CR Head index Array */
> > +       ia->cr_hia_p_addr = p_addr + (sizeof(u16) * NUM_QUEUES * 2);
> > +       ia->cr_hia = v_addr + (sizeof(u16) * NUM_QUEUES * 2);
> > +
> > +       /* CR Tail Index Array */
> > +       ia->cr_tia_p_addr = p_addr + (sizeof(u16) * NUM_QUEUES * 3);
> > +       ia->cr_tia = v_addr + (sizeof(u16) * NUM_QUEUES * 3);
> > +}
> > +
> > +static void btintel_pcie_free(struct btintel_pcie_data *data)
> > +{
> > +       btintel_pcie_free_rxq_bufs(data, &data->rxq);
> > +       btintel_pcie_free_txq_bufs(data, &data->txq);
> > +
> > +       dma_pool_free(data->dma_pool, data->dma_v_addr, data-
> >dma_p_addr);
> > +       dma_pool_destroy(data->dma_pool);
> > +       BT_DBG("DMA memory is freed");
> 
> Printing this kind of information is useless, we should really make
> sure there are no leaks but there are better tools for that then
> printing when we free data.
Ack.

> 
> > +}
> > +
> > +/* Allocate tx and rx queues, any related data structures and buffers.
> > + */
> > +static int btintel_pcie_alloc(struct btintel_pcie_data *data)
> > +{
> > +       int err = 0;
> > +       size_t total;
> > +       dma_addr_t p_addr;
> > +       void *v_addr;
> > +
> > +       /* Allocate the chunk of DMA memory for descriptors, index array, and
> > +        * context information, instead of allocating individually.
> > +        * The DMA memory for data buffer is allocated while setting up the
> > +        * each queue.
> > +        *
> > +        * Total size is sum of the following
> > +        *  + size of TFD * Number of descriptors in queue
> > +        *  + size of URBD0 * Number of descriptors in queue
> > +        *  + size of FRBD * Number of descriptors in queue
> > +        *  + size of URBD1 * Number of descriptors in queue
> > +        *  + size of index * Number of queues(2) * type of index array(4)
> > +        *  + size of context information
> > +        */
> > +       total = (sizeof(struct tfd) + sizeof(struct urbd0) + sizeof(struct frbd)
> > +               + sizeof(struct urbd1)) * DESCS_COUNT;
> > +
> > +       /* Add the sum of size of index array and size of ci struct */
> > +       total += (sizeof(u16) * NUM_QUEUES * 4) + sizeof(struct ctx_info);
> > +
> > +       /* Allocate DMA Pool */
> > +       data->dma_pool = dma_pool_create(KBUILD_MODNAME, &data-
> >pdev->dev,
> > +                                        total, DMA_POOL_ALIGNMENT, 0);
> > +       if (!data->dma_pool) {
> > +               BT_ERR("Failed to allocate dma pool for queues");
> > +               err = -ENOMEM;
> > +               goto exit_error;
> > +       }
> > +
> > +       v_addr = dma_pool_zalloc(data->dma_pool, GFP_KERNEL |
> __GFP_NOWARN,
> > +                                &p_addr);
> > +       if (!v_addr) {
> > +               BT_ERR("Failed to alloc dma memory for queues");
> > +               dma_pool_destroy(data->dma_pool);
> > +               err = -ENOMEM;
> > +               goto exit_error;
> > +       }
> > +
> > +       data->dma_p_addr = p_addr;
> > +       data->dma_v_addr = v_addr;
> > +
> > +       BT_DBG("dma pool: p_addr=0x%llx v_addr=0x%p", p_addr, v_addr);
> > +
> > +       /* Setup descriptor count */
> > +       data->txq.count = DESCS_COUNT;
> > +       data->rxq.count = DESCS_COUNT;
> > +
> > +       /* Setup tfds */
> > +       data->txq.tfds_p_addr = p_addr;
> > +       data->txq.tfds = v_addr;
> > +
> > +       p_addr += (sizeof(struct tfd) * DESCS_COUNT);
> > +       v_addr += (sizeof(struct tfd) * DESCS_COUNT);
> > +
> > +       /* Setup urbd0 */
> > +       data->txq.urbd0s_p_addr = p_addr;
> > +       data->txq.urbd0s = v_addr;
> > +
> > +       p_addr += (sizeof(struct urbd0) * DESCS_COUNT);
> > +       v_addr += (sizeof(struct urbd0) * DESCS_COUNT);
> > +
> > +       /* Setup frbd */
> > +       data->rxq.frbds_p_addr = p_addr;
> > +       data->rxq.frbds = v_addr;
> > +
> > +       p_addr += (sizeof(struct frbd) * DESCS_COUNT);
> > +       v_addr += (sizeof(struct frbd) * DESCS_COUNT);
> > +
> > +       /* Setup urbd1 */
> > +       data->rxq.urbd1s_p_addr = p_addr;
> > +       data->rxq.urbd1s = v_addr;
> > +
> > +       p_addr += (sizeof(struct urbd1) * DESCS_COUNT);
> > +       v_addr += (sizeof(struct urbd1) * DESCS_COUNT);
> > +
> > +       /* Setup data buffers for txq */
> > +       err = btintel_pcie_setup_txq_bufs(data, &data->txq);
> > +       if (err) {
> > +               BT_ERR("Failed to setup txq buffers: %d", err);
> > +               goto exit_error_pool;
> > +       }
> > +
> > +       /* Setup data buffers for rxq */
> > +       err = btintel_pcie_setup_rxq_bufs(data, &data->rxq);
> > +       if (err) {
> > +               BT_ERR("Failed to allocate rxq buffers: %d", err);
> > +               goto exit_error_txq;
> > +       }
> > +
> > +       /* Setup Index Array */
> > +       btintel_pcie_setup_ia(data, p_addr, v_addr, &data->ia);
> > +
> > +       /* Setup Context Information */
> > +       p_addr += sizeof(u16) * NUM_QUEUES * 4;
> > +       v_addr += sizeof(u16) * NUM_QUEUES * 4;
> > +
> > +       data->ci = v_addr;
> > +       data->ci_p_addr = p_addr;
> > +
> > +       /* Initialize the CI */
> > +       btintel_pcie_init_ci(data, data->ci);
> > +
> > +       return 0;
> > +
> > +exit_error_txq:
> > +       btintel_pcie_free_txq_bufs(data, &data->txq);
> > +exit_error_pool:
> > +       dma_pool_free(data->dma_pool, data->dma_v_addr, data-
> >dma_p_addr);
> > +       dma_pool_destroy(data->dma_pool);
> > +exit_error:
> > +       return err;
> > +}
> > +
> > +static void btintel_pcie_release_hdev(struct btintel_pcie_data *data)
> > +{
> > +       struct hci_dev *hdev;
> > +
> > +       hdev = data->hdev;
> > +       if (hdev) {
> > +               hci_unregister_dev(hdev);
> > +               hci_free_dev(hdev);
> > +       }
> > +       data->hdev = NULL;
> > +}
> > +
> > +static int btintel_pcie_setup_hdev(struct btintel_pcie_data *data)
> > +{
> > +       /* TODO: initialize hdev and assign the callbacks to hdev */
> > +       return -ENODEV;
> > +}
> > +
> > +static int btintel_pcie_probe(struct pci_dev *pdev,
> > +                             const struct pci_device_id *ent)
> > +{
> > +       int err;
> > +       struct btintel_pcie_data *data;
> > +
> > +       if (!pdev)
> > +               return -ENODEV;
> > +
> > +       data = devm_kzalloc(&pdev->dev, sizeof(*data), GFP_KERNEL);
> > +       if (!data)
> > +               return -ENOMEM;
> > +
> > +       /* initialize the btintel_pcie data struct */
> > +       data->pdev = pdev;
> > +
> > +       spin_lock_init(&data->irq_lock);
> > +       spin_lock_init(&data->hci_rx_lock);
> > +
> > +       init_waitqueue_head(&data->gp0_wait_q);
> > +       data->gp0_received = false;
> > +
> > +       init_waitqueue_head(&data->tx_wait_q);
> > +       data->tx_wait_done = false;
> > +
> > +       data->workqueue = alloc_ordered_workqueue(KBUILD_MODNAME,
> WQ_HIGHPRI);
> > +       if (!data->workqueue) {
> > +               BT_ERR("Failed to create workqueue");
> > +               return -ENOMEM;
> > +       }
> > +       skb_queue_head_init(&data->rx_skb_q);
> > +       INIT_WORK(&data->rx_work, btintel_pcie_rx_work);
> > +
> > +       data->boot_stage_cache = 0x00;
> > +       data->img_resp_cache = 0x00;
> > +
> > +       /* PCIe specific all to configure it for this device includes
> > +        * enabling pice device, setting master, reading BAR[0], configuring
> > +        * MSIx, setting DMA mask, and save the driver data.
> > +        */
> > +       err = btintel_pcie_config_pcie(pdev, data);
> > +       if (err) {
> > +               BT_ERR("Failed to config pcie (%d)", err);
> > +               goto exit_error;
> > +       }
> > +
> > +       /* Set driver data for this PCI device */
> > +       pci_set_drvdata(pdev, data);
> > +
> > +       /* allocate the IPC struct */
> > +       err = btintel_pcie_alloc(data);
> > +       if (err) {
> > +               BT_ERR("Failed to allocate queues(%d)", err);
> > +               goto exit_error;
> > +       }
> > +
> > +       /* Enable BT function */
> > +       err = btintel_pcie_enable_bt(data);
> > +       if (err) {
> > +               BT_ERR("Failed to start bluetooth device(%d)", err);
> > +               goto exit_error;
> > +       }
> > +
> > +       /* CNV information (CNVi and CNVr) is in CSR */
> > +       data->cnvi = btintel_pcie_rd_reg32(data, CSR_HW_REV_REG);
> > +       BT_DBG("cnvi:   0x%08x", data->cnvi);
> > +
> > +       data->cnvr = btintel_pcie_rd_reg32(data, CSR_RF_ID_REG);
> > +       BT_DBG("cnvr:   0x%08x", data->cnvr);
> > +
> > +       err = btintel_pcie_start_rx(data);
> > +       if (err) {
> > +               BT_ERR("Failed to start rx (%d)", err);
> > +               goto exit_error;
> > +       }
> > +
> > +       err = btintel_pcie_setup_hdev(data);
> > +       if (err) {
> > +               BT_ERR("Failed to setup HCI module");
> > +               goto exit_error;
> > +       }
> > +
> > +       return 0;
> > +
> > +exit_error:
> > +       /* reset device before leave */
> > +       btintel_pcie_reset_bt(data);
> > +
> > +       /* clear bus mastering */
> > +       pci_clear_master(pdev);
> > +
> > +       /* Unset driver data for PCI device */
> > +       pci_set_drvdata(pdev, NULL);
> > +
> > +       return err;
> > +}
> > +
> > +static void btintel_pcie_remove(struct pci_dev *pdev)
> > +{
> > +       struct btintel_pcie_data *data;
> > +
> > +       if (!pdev) {
> > +               BT_ERR("Invalid parameter: pdev");
> > +               return;
> > +       }
> > +
> > +       data = pci_get_drvdata(pdev);
> > +       if (!data) {
> > +               BT_ERR("data is empty");
> > +               return;
> > +       }
> > +
> > +       btintel_pcie_release_hdev(data);
> > +
> > +       flush_work(&data->rx_work);
> > +
> > +       destroy_workqueue(data->workqueue);
> > +
> > +       btintel_pcie_free(data);
> > +
> > +       /* reset device before leave */
> > +       btintel_pcie_reset_bt(data);
> > +
> > +       /* clear bus mastering */
> > +       pci_clear_master(pdev);
> > +
> > +       /* Unset driver data for PCI device */
> > +       pci_set_drvdata(pdev, NULL);
> > +}
> > +
> > +#ifdef CONFIG_PM
> > +static int btintel_pcie_suspend(struct device *dev)
> > +{
> > +       /* TODO: Add support suspend */
> > +       return 0;
> > +}
> > +
> > +static int btintel_pcie_resume(struct device *dev)
> > +{
> > +       /* TODO: Add support resume */
> > +       return 0;
> > +}
> > +
> > +static SIMPLE_DEV_PM_OPS(btintel_pcie_pm_ops, btintel_pcie_suspend,
> > +                                                       btintel_pcie_resume);
> > +#endif /* CONFIG_PM */
> > +
> > +static struct pci_driver btintel_pcie_driver = {
> > +       .name = KBUILD_MODNAME,
> > +       .id_table = btintel_pcie_table,
> > +       .probe = btintel_pcie_probe,
> > +       .remove = btintel_pcie_remove,
> > +#ifdef CONFIG_PM
> > +       .driver.pm = &btintel_pcie_pm_ops,
> > +#endif /* CONFIG_PM */
> > +};
> > +module_pci_driver(btintel_pcie_driver);
> > +
> > +MODULE_AUTHOR("Tedd Ho-Jeong An <tedd.an@xxxxxxxxx>");
> > +MODULE_DESCRIPTION("Intel Bluetooth PCIe transport driver ver "
> VERSION);
> > +MODULE_VERSION(VERSION);
> > +MODULE_LICENSE("GPL");
> 
> Botton line: We have to tone down the debbuging since it pretting a
> lot of useless information and when it does it prints with old BT_DBG
> macros which doesn't print the controller info so on system with
> multiple controller that would be impossible to say which controller
> is generating the logs.
Ack.  I will remove the unwanted debug logs and switch to bt_dev_dbg/info/err wherever applicable.
> 
> > diff --git a/drivers/bluetooth/btintel_pcie.h
> b/drivers/bluetooth/btintel_pcie.h
> > new file mode 100644
> > index 000000000000..1554964686bd
> > --- /dev/null
> > +++ b/drivers/bluetooth/btintel_pcie.h
> > @@ -0,0 +1,449 @@
> > +/* SPDX-License-Identifier: GPL-2.0-or-later */
> > +/*
> > + * Intel Bluetooth PCIE driver
> > + *
> > + * Copyright (C) 2017 Intel Corporation. All rights reserved.
> 
> Ditto.
> 
Ack.

> > + *
> > + * This program is free software; you can redistribute it and/or
> > + * modify it under the terms of the GNU General Public License version
> > + * 2 as published by the Free Software Foundation.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> > + * GNU General Public License for more details.
> > + *
> > + * Copyright (C) 2022  Intel Corporation
> > + *
> > + * Intel Bluetooth Driver for PCIE interface.
> > + */
> > +
> > +/* Control and Status Register(CSR) */
> > +#define CSR_BASE                       (0x000)
> > +#define CSR_FUNC_CTRL_REG              (CSR_BASE + 0x024)
> > +#define CSR_HW_REV_REG                 (CSR_BASE + 0x028)
> > +#define CSR_RF_ID_REG                  (CSR_BASE + 0x09C)
> > +#define CSR_BOOT_STAGE_REG             (CSR_BASE + 0x108)
> > +#define CSR_CI_ADDR_LSB_REG            (CSR_BASE + 0x118)
> > +#define CSR_CI_ADDR_MSB_REG            (CSR_BASE + 0x11C)
> > +#define CSR_IMG_RESPONSE_REG           (CSR_BASE + 0x12C)
> > +#define CSR_HBUS_TARG_WRPTR            (CSR_BASE + 0x460)
> > +
> > +/* CSR Function Control Register */
> > +#define CSR_FUNC_CTRL_FUNC_ENA         (BIT(0))
> > +#define CSR_FUNC_CTRL_MAC_INIT         (BIT(6))
> > +#define CSR_FUNC_CTRL_FUNC_INIT                (BIT(7))
> > +#define CSR_FUNC_CTRL_MAC_ACCESS_STS   (BIT(20))
> > +#define CSR_FUNC_CTRL_SW_RESET         (BIT(31))
> > +
> > +/* Value for CSR_BOOT_STAGE register */
> > +#define CSR_BOOT_STAGE_ROM             (BIT(0))
> > +#define CSR_BOOT_STAGE_IML             (BIT(1))
> > +#define CSR_BOOT_STAGE_OPFW            (BIT(2))
> > +#define CSR_BOOT_STAGE_ROM_LOCKDOWN    (BIT(10))
> > +#define CSR_BOOT_STAGE_IML_LOCKDOWN    (BIT(11))
> > +#define CSR_BOOT_STAGE_MAC_ACCESS_ON   (BIT(16))
> > +#define CSR_BOOT_STAGE_ALIVE           (BIT(23))
> > +
> > +/* Registers for MSIX */
> > +#define CSR_MSIX_BASE                  (0x2000)
> > +#define CSR_MSIX_FH_INT_CAUSES         (CSR_MSIX_BASE + 0x0800)
> > +#define CSR_MSIX_FH_INT_MASK           (CSR_MSIX_BASE + 0x0804)
> > +#define CSR_MSIX_HW_INT_CAUSES         (CSR_MSIX_BASE + 0x0808)
> > +#define CSR_MSIX_HW_INT_MASK           (CSR_MSIX_BASE + 0x080C)
> > +#define CSR_MSIX_AUTOMASK_ST           (CSR_MSIX_BASE + 0x0810)
> > +#define CSR_MSIX_AUTOMASK_EN           (CSR_MSIX_BASE + 0x0814)
> > +#define CSR_MSIX_IVAR_BASE             (CSR_MSIX_BASE + 0x0880)
> > +#define CSR_MSIX_IVAR(cause)           (CSR_MSIX_IVAR_BASE + (cause))
> > +
> > +/* Causes for the FH register interrupts */
> > +enum msix_fh_int_causes {
> > +       MSIX_FH_INT_CAUSES_0            = BIT(0),       /* cause 0 */
> > +       MSIX_FH_INT_CAUSES_1            = BIT(1),       /* cause 1 */
> > +};
> > +
> > +/* Causes for the HW register interrupts */
> > +enum msix_hw_int_causes {
> > +       MSIX_HW_INT_CAUSES_GP0          = BIT(0),       /* cause 32 */
> > +};
> > +
> > +#define MSIX_NON_AUTO_CLEAR_CAUSE      BIT(7)
> > +
> > +/* Minimum and Maximum number of MSIx Vector
> > + * Intel Bluetooth PCIe support only 1 vector
> > + */
> > +#define MSIX_VEC_MAX                   1
> > +#define MSIX_VEC_MIN                   1
> > +
> > +/* Default Poll time for MAC access during init*/
> > +#define DEFAULT_MAC_ACCESS_TIMEOUT     200000
> > +
> > +/* Default interrupt timeout in msec */
> > +#define DEFAULT_INTR_TIMEOUT           3000
> > +
> > +/* The number of descriptors in TX/RX queues */
> > +#define DESCS_COUNT            16
> > +
> > +/* Number of Queue for TX and RX
> > + * It indicates the index of the IA(Index Array)
> > + */
> > +enum {
> > +       TXQ_NUM = 0,
> > +       RXQ_NUM = 1,
> > +       NUM_QUEUES = 2,
> > +};
> > +
> > +/* The size of DMA buffer for TX and RX */
> > +#define BUFFER_SIZE                    4096
> > +
> > +/* DMA allocation alignment */
> > +#define DMA_POOL_ALIGNMENT             256
> > +
> > +/* TX wait time (jiffies) */
> > +#define TX_WAIT_TIMEOUT                        500
> > +
> > +/* Number of pending RX requests for downlink */
> > +#define RX_MAX_QUEUE                   6
> > +
> > +/* Enum for RBD buffer size mappting */
> > +enum {
> > +       RBD_SIZE_1K = 0x1,
> > +       RBD_SIZE_2K = 0x2,
> > +       RBD_SIZE_4K = 0x4,
> > +       RBD_SIZE_8K = 0x8,
> > +       RBD_SIZE_12K = 0x9,
> > +       RBD_SIZE_16K = 0xA,
> > +       RBD_SIZE_20K = 0xB,
> > +       RBD_SIZE_24K = 0xC,
> > +       RBD_SIZE_28K = 0xD,
> > +       RBD_SIZE_32K = 0xE,
> > +};
> > +
> > +/*
> > + * Struct for Context Information (v2)
> > + *
> > + * All members are write-only for host and read-only for device.
> > + *
> > + * @version: Version of context information
> > + * @size: Size of context information
> > + * @config: Config with which host wants peripheral to execute
> > + *     Subset of capability register published by device
> > + * @addr_tr_hia: Address of TR Head Index Array
> > + * @addr_tr_tia: Address of TR Tail Index Array
> > + * @addr_cr_hia: Address of CR Head Index Array
> > + * @addr_cr_tia: Address of CR Tail Index Array
> > + * @num_tr_ia: Number of entries in TR Index Arrays
> > + * @num_cr_ia: Number of entries in CR Index Arrays
> > + * @rbd_siz: RBD Size { 0x4=4K }
> > + * @addr_tfdq: Address of TFD Queue(tx)
> > + * @addr_urbdq0: Address of URBD Queue(tx)
> > + * @num_tfdq: Number of TFD in TFD Queue(tx)
> > + * @num_urbdq0: Number of URBD in URBD Queue(tx)
> > + * @tfdq_db_vec: Queue number of TFD
> > + * @urbdq0_db_vec: Queue number of URBD
> > + * @addr_frbdq: Address of FRBD Queue(rx)
> > + * @addr_urbdq1: Address of URBD Queue(rx)
> > + * @num_frbdq: Number of FRBD in FRBD Queue(rx)
> > + * @frbdq_db_vec: Queue number of FRBD
> > + * @num_urbdq1: Number of URBD in URBD Queue(rx)
> > + * @urbdq_db_vec: Queue number of URBDQ1
> > + * @tr_msi_vec: Transfer Ring MSI Vector
> > + * @cr_msi_vec: Completion Ring MSI Vector
> > + * @dbgc_addr: DBGC first fragmemt address
> > + * @dbgc_size: DBGC buffer size
> > + * @early_enable: Enarly debug enable
> > + * @dbg_output_mode: Debug output mode
> > + *     Bit[4] DBGC O/P { 0=SRAM, 1=DRAM(not relevant for NPK) }
> > + *     Bit[5] DBGC I/P { 0=BDBG, 1=DBGI }
> > + *     Bits[6:7] DBGI O/P(relevant if bit[5] = 1)
> > + *      0=BT DBGC, 1=WiFi DBGC, 2=NPK }
> > + * @dbg_preset: Debug preset
> > + * @ext_addr: Address of context information extension
> > + * @ext_size: Size of context information part
> > + *
> > + * Total 38 DWords
> > + *
> > + */
> > +struct ctx_info {
> > +       u16     version;
> > +       u16     size;
> > +       u32     config;
> > +       u32     reserved_dw02;
> > +       u32     reserved_dw03;
> > +       u64     addr_tr_hia;
> > +       u64     addr_tr_tia;
> > +       u64     addr_cr_hia;
> > +       u64     addr_cr_tia;
> > +       u16     num_tr_ia;
> > +       u16     num_cr_ia;
> > +       u32     rbd_size:4,
> > +               reserved_dw13:28;
> > +       u64     addr_tfdq;
> > +       u64     addr_urbdq0;
> > +       u16     num_tfdq;
> > +       u16     num_urbdq0;
> > +       u16     tfdq_db_vec;
> > +       u16     urbdq0_db_vec;
> > +       u64     addr_frbdq;
> > +       u64     addr_urbdq1;
> > +       u16     num_frbdq;
> > +       u16     frbdq_db_vec;
> > +       u16     num_urbdq1;
> > +       u16     urbdq_db_vec;
> > +       u16     tr_msi_vec;
> > +       u16     cr_msi_vec;
> > +       u32     reserved_dw27;
> > +       u64     dbgc_addr;
> > +       u32     dbgc_size;
> > +       u32     early_enable:1,
> > +               reserved_dw31:3,
> > +               dbg_output_mode:4,
> > +               dbg_preset:8,
> > +               reserved2_dw31:16;
> > +       u64     ext_addr;
> > +       u32     ext_size;
> > +       u32     test_param;
> > +       u32     reserved_dw36;
> > +       u32     reserved_dw37;
> > +} __packed;
> > +
> > +/* Transfer Descriptor for TX
> > + * @type: Not in use. Set to 0x0
> > + * @size: Size of data in the buffer
> > + * @addr: DMA Address of buffer
> > + */
> > +struct tfd {
> > +       u8      type;
> > +       u16     size;
> > +       u8      reserved;
> > +       u64     addr;
> > +       u32     reserved1;
> > +} __packed;
> > +
> > +/* URB Descriptor for TX
> > + * @tfd_index: Index of TFD in TFDQ + 1
> > + * @num_txq: Queue index of TFD Queue
> > + * @cmpl_count: Completion count. Always 0x01
> > + * @immediate_cmpl: Immediate completion flag: Always 0x01
> > + */
> > +struct urbd0 {
> > +       u32     tfd_index:16,
> > +               num_txq:8,
> > +               cmpl_count:4,
> > +               reserved:3,
> > +               immediate_cmpl:1;
> > +} __packed;
> > +
> > +/* FRB Descriptor for RX
> > + * @tag: RX buffer tag (index of RX buffer queue)
> > + * @addr: Address of buffer
> > + */
> > +struct frbd {
> > +       u32     tag:16,
> > +               reserved:16;
> > +       u32     reserved2;
> > +       u64     addr;
> > +} __packed;
> > +
> > +/* URB Descriptor for RX
> > + * @frbd_tag: Tag from FRBD
> > + * @status: Status
> > + */
> > +struct urbd1 {
> > +       u32     frbd_tag:16,
> > +               status:1,
> > +               reserved:14,
> > +               fixed:1;
> > +} __packed;
> > +
> > +/* RFH header in RX packet
> > + * @packet_len: Length of the data in the buffer
> > + * @rxq: RX Queue number
> > + * @cmd_id: Command ID. Not in Use
> > + */
> > +struct rfh_hdr {
> > +       u64     packet_len:16,
> > +               rxq:6,
> > +               reserved:10,
> > +               cmd_id:16,
> > +               reserved1:16;
> > +} __packed;
> > +
> > +/* Internal data buffer
> > + * @data: pointer to the data buffer
> > + * @p_addr: physical address of data buffer
> > + */
> > +struct data_buf {
> > +       u8              *data;
> > +       dma_addr_t      data_p_addr;
> > +};
> > +
> > +/* Index Array */
> > +struct ia {
> > +       dma_addr_t      tr_hia_p_addr;
> > +       u16             *tr_hia;
> > +       dma_addr_t      tr_tia_p_addr;
> > +       u16             *tr_tia;
> > +       dma_addr_t      cr_hia_p_addr;
> > +       u16             *cr_hia;
> > +       dma_addr_t      cr_tia_p_addr;
> > +       u16             *cr_tia;
> > +};
> > +
> > +/* Structure for TX Queue
> > + * @count: Number of descriptors
> > + * @tfds: Array of TFD
> > + * @urbd0s: Array of URBD0
> > + * @buf: Array of data_buf structure
> > + */
> > +struct txq {
> > +       u16             count;
> > +
> > +       dma_addr_t      tfds_p_addr;
> > +       struct tfd      *tfds;
> > +
> > +       dma_addr_t      urbd0s_p_addr;
> > +       struct urbd0    *urbd0s;
> > +
> > +       dma_addr_t      buf_p_addr;
> > +       void            *buf_v_addr;
> > +       struct data_buf *bufs;
> > +};
> > +
> > +/* Structure for RX Queue
> > + * @count: Number of descriptors
> > + * @frbds: Array of FRBD
> > + * @urbd1s: Array of URBD1
> > + * @buf: Array of data_buf structure
> > + */
> > +struct rxq {
> > +       u16             count;
> > +
> > +       dma_addr_t      frbds_p_addr;
> > +       struct frbd     *frbds;
> > +
> > +       dma_addr_t      urbd1s_p_addr;
> > +       struct urbd1    *urbd1s;
> > +
> > +       dma_addr_t      buf_p_addr;
> > +       void            *buf_v_addr;
> > +       struct data_buf *bufs;
> > +};
> > +
> > +/* struct btintel_pcie_data
> > + * @pdev: pci device
> > + * @hdev: hdev device
> > + * @flags: driver state
> > + * @irq_lock: spinlock for MSIX
> > + * @hci_rx_lock: spinlock for HCI RX flow
> > + * @base_addr: pci base address (from BAR)
> > + * @msix_entries: array of MSIX entries
> > + * @msix_enabled: true if MSIX is enabled;
> > + * @alloc_vecs: number of interrupt vectors allocated
> > + * @def_irq: default irq for all causes
> > + * @fh_init_mask: initial unmasked rxq causes
> > + * @hw_init_mask: initial unmaksed hw causes
> > + * @boot_stage_cache: cached value of boot stage register
> > + * @img_resp_cache: cached value of image response register
> > + * @cnvi: CNVi register value
> > + * @cnvr: CNVr register value
> > + * @gp0_received: condition for gp0 interrupt
> > + * @gp0_wait_q: wait_q for gp0 interrupt
> > + * @tx_wait_done: condition for tx interrupt
> > + * @tx_wait_q: wait_q for tx interrupt
> > + * @workqueue: workqueue for RX work
> > + * @rx_skb_q: SKB queue for RX packet
> > + * @rx_work: RX work struct to process the RX packet in @rx_skb_q
> > + * @dma_pool: DMA pool for descriptors, index array and ci
> > + * @dma_p_addr: DMA address for pool
> > + * @dma_v_addr: address of pool
> > + * @ci_p_addr: DMA address for CI struct
> > + * @ci: CI struct
> > + * @ia: Index Array struct
> > + * @txq: TX Queue struct
> > + * @rxq: RX Queue struct
> > + */
> > +struct btintel_pcie_data {
> > +       struct pci_dev  *pdev;
> > +       struct hci_dev  *hdev;
> > +
> > +       unsigned long   flags;
> > +       /* lock used in MSIX interrupt */
> > +       spinlock_t      irq_lock;
> > +       /* lock to serialize rx events */
> > +       spinlock_t      hci_rx_lock;
> > +
> > +       void __iomem    *base_addr;
> > +
> > +       struct msix_entry       msix_entries[MSIX_VEC_MAX];
> > +       bool    msix_enabled;
> > +       u32     alloc_vecs;
> > +       u32     def_irq;
> > +
> > +       u32     fh_init_mask;
> > +       u32     hw_init_mask;
> > +
> > +       u32     boot_stage_cache;
> > +       u32     img_resp_cache;
> > +
> > +       u32     cnvi;
> > +       u32     cnvr;
> > +
> > +       bool    gp0_received;
> > +       wait_queue_head_t       gp0_wait_q;
> > +
> > +       bool    tx_wait_done;
> > +       wait_queue_head_t       tx_wait_q;
> > +
> > +       struct workqueue_struct *workqueue;
> > +       struct sk_buff_head     rx_skb_q;
> > +       struct work_struct      rx_work;
> > +
> > +       struct dma_pool *dma_pool;
> > +       dma_addr_t      dma_p_addr;
> > +       void            *dma_v_addr;
> > +
> > +       dma_addr_t      ci_p_addr;
> > +       struct ctx_info *ci;
> > +       struct ia       ia;
> > +       struct txq      txq;
> > +       struct rxq      rxq;
> > +};
> > +
> > +static inline u32 btintel_pcie_rd_reg32(struct btintel_pcie_data *data,
> > +                                       u32 offset)
> > +{
> > +       return ioread32(data->base_addr + offset);
> > +}
> > +
> > +static inline void btintel_pcie_wr_reg8(struct btintel_pcie_data *data,
> > +                                       u32 offset, u8 val)
> > +{
> > +       iowrite8(val, data->base_addr + offset);
> > +}
> > +
> > +static inline void btintel_pcie_wr_reg32(struct btintel_pcie_data *data,
> > +                                        u32 offset, u32 val)
> > +{
> > +       iowrite32(val, data->base_addr + offset);
> > +}
> > +
> > +static inline void btintel_pcie_set_reg_bits(struct btintel_pcie_data *data,
> > +                                            u32 offset, u32 bits)
> > +{
> > +       u32 r;
> > +
> > +       r = ioread32(data->base_addr + offset);
> > +       r |= bits;
> > +       iowrite32(r, data->base_addr + offset);
> > +}
> > +
> > +static inline void btintel_pcie_clr_reg_bits(struct btintel_pcie_data *data,
> > +                                            u32 offset, u32 bits)
> > +{
> > +       u32 r;
> > +
> > +       r = ioread32(data->base_addr + offset);
> > +       r &= ~bits;
> > +       iowrite32(r, data->base_addr + offset);
> > +}
> > --
> > 2.34.1
> 
> All symbols must be properly prefixed with BTINTEL_PCIE
Ack.

> 
> >
> 
> 
> --
> Luiz Augusto von Dentz

Thanks,
Kiran





[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux