lkml.org 
[lkml]   [2006]   [Apr]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[PATCH] Fix SCO on some bluetooth adapters
    Some bluetooth adapters return an incorrect number of sco packets in
    READ_BUFFER_SIZE. Add a quirk hook to fix it from the driver side,
    and use the hook for a first adapter.

    Signed-off-by: Olivier Galibert <galibert@pobox.com>

    ---

    I tried to take Marcel Holtmann's remarks into account, but he seems
    unavailable to tell me whether he likes that version, so I'll send it
    to the whole shebang instead of just him this time.

    The main point was "only the driver knows how to fix the problem",
    hence a function pointer quirk hook. This is definitively not limited
    to broadcom USB devices, googling for "SCO MTU 64:0" has interesting
    results.

    Pavel, '4' doesn't seem to be a value that happens in any correctly
    answering adapter, while 8 happens often. Try "SCO MTU 64:4" and "SCO
    MTU 64:8" for comparison.


    drivers/bluetooth/hci_usb.c | 12 ++++++++++++
    drivers/bluetooth/hci_usb.h | 1 +
    include/net/bluetooth/hci_core.h | 1 +
    net/bluetooth/hci_event.c | 16 ++++++++++++++--
    4 files changed, 28 insertions(+), 2 deletions(-)

    841d7690e75189803907fbc4616b984087e7f89c
    diff --git a/drivers/bluetooth/hci_usb.c b/drivers/bluetooth/hci_usb.c
    index 92382e8..0cea36b 100644
    --- a/drivers/bluetooth/hci_usb.c
    +++ b/drivers/bluetooth/hci_usb.c
    @@ -130,6 +130,9 @@ static struct usb_device_id blacklist_id
    /* CSR BlueCore Bluetooth Sniffer */
    { USB_DEVICE(0x0a12, 0x0002), .driver_info = HCI_SNIFFER },

    + /* Belkin F8T012 */
    + { USB_DEVICE(0x050d, 0x0012), .driver_info = HCI_READ_BUFFER_SIZE },
    +
    { } /* Terminating entry */
    };

    @@ -817,6 +820,12 @@ static void hci_usb_notify(struct hci_de
    BT_DBG("%s evt %d", hdev->name, evt);
    }

    +static void hci_usb_quirk_read_buffer_size(struct hci_dev *hdev)
    +{
    + if (!hdev->sco_pkts)
    + hdev->sco_pkts = 8;
    +}
    +
    static int hci_usb_probe(struct usb_interface *intf, const struct usb_device_id *id)
    {
    struct usb_device *udev = interface_to_usbdev(intf);
    @@ -984,6 +993,9 @@ #endif

    if (reset || id->driver_info & HCI_RESET)
    set_bit(HCI_QUIRK_RESET_ON_INIT, &hdev->quirks);
    +
    + if (id->driver_info & HCI_READ_BUFFER_SIZE)
    + hdev->quirk_read_buffer_size = hci_usb_quirk_read_buffer_size;

    if (id->driver_info & HCI_SNIFFER) {
    if (le16_to_cpu(udev->descriptor.bcdDevice) > 0x997)
    diff --git a/drivers/bluetooth/hci_usb.h b/drivers/bluetooth/hci_usb.h
    index 37100a6..ea11df7 100644
    --- a/drivers/bluetooth/hci_usb.h
    +++ b/drivers/bluetooth/hci_usb.h
    @@ -35,6 +35,7 @@ #define HCI_CSR 0x08
    #define HCI_SNIFFER 0x10
    #define HCI_BCM92035 0x20
    #define HCI_BROKEN_ISOC 0x40
    +#define HCI_READ_BUFFER_SIZE 0x80

    #define HCI_MAX_IFACE_NUM 3

    diff --git a/include/net/bluetooth/hci_core.h b/include/net/bluetooth/hci_core.h
    index bb9f81d..045b61f 100644
    --- a/include/net/bluetooth/hci_core.h
    +++ b/include/net/bluetooth/hci_core.h
    @@ -134,6 +134,7 @@ struct hci_dev {
    void (*destruct)(struct hci_dev *hdev);
    void (*notify)(struct hci_dev *hdev, unsigned int evt);
    int (*ioctl)(struct hci_dev *hdev, unsigned int cmd, unsigned long arg);
    + void (*quirk_read_buffer_size)(struct hci_dev *hdev);
    };

    struct hci_conn {
    diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c
    index eb64555..87358b8 100644
    --- a/net/bluetooth/hci_event.c
    +++ b/net/bluetooth/hci_event.c
    @@ -321,8 +321,20 @@ static void hci_cc_info_param(struct hci

    hdev->acl_mtu = __le16_to_cpu(bs->acl_mtu);
    hdev->sco_mtu = bs->sco_mtu ? bs->sco_mtu : 64;
    - hdev->acl_pkts = hdev->acl_cnt = __le16_to_cpu(bs->acl_max_pkt);
    - hdev->sco_pkts = hdev->sco_cnt = __le16_to_cpu(bs->sco_max_pkt);
    + hdev->acl_pkts = __le16_to_cpu(bs->acl_max_pkt);
    + hdev->sco_pkts = __le16_to_cpu(bs->sco_max_pkt);
    +
    + if (hdev->quirk_read_buffer_size)
    + hdev->quirk_read_buffer_size(hdev);
    +
    + if (!hdev->acl_mtu || !hdev->sco_mtu || !hdev->acl_pkts || !hdev->sco_pkts) {
    + printk (KERN_WARNING "Your Bluetooth adapter has an incorrect OCF_READ_BUFFER_SIZE reply (%d:%d, %d:%d)\n",
    + hdev->acl_mtu, hdev->acl_pkts, hdev->sco_mtu, hdev->sco_pkts);
    + printk (KERN_WARNING "It won't work correctly. Please contact Marcel Holtmann <marcel@holtmann.org> with information about your device to try workarounds.\n");
    + }
    +
    + hdev->acl_cnt = hdev->acl_pkts;
    + hdev->sco_cnt = hdev->sco_pkts;

    BT_DBG("%s mtu: acl %d, sco %d max_pkt: acl %d, sco %d", hdev->name,
    hdev->acl_mtu, hdev->sco_mtu, hdev->acl_pkts, hdev->sco_pkts);
    --
    1.3.0.rc1.g40e9
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2006-04-20 16:07    [W:4.366 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site