Re: [PATCH v2] Bluetooth: Another vendor specific ID for BCM20702A0[0a5c:21f1]

From: Michal Marek
Date: Wed Aug 08 2012 - 04:39:05 EST


On 6.8.2012 20:40, Gustavo Padovan wrote:
> Author: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx>
> Date: Mon Aug 6 15:36:49 2012 -0300
>
> Bluetooth: Use USB_VENDOR_AND_INTERFACE_INFO() for Broadcom devices
>
> Many Broadcom devices has a vendor specific devices class, with this rule
> we match all existent and future controllers with this behavior.
>
> Signed-off-by: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx>
>
> diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
> index cef3bac..504231e 100644
> --- a/drivers/bluetooth/btusb.c
> +++ b/drivers/bluetooth/btusb.c
> @@ -104,6 +104,9 @@ static struct usb_device_id btusb_table[] = {
> /* Foxconn - Hon Hai */
> { USB_DEVICE(0x0489, 0xe033) },
>
> + /*Broadcom devices with vendor specific id */
> + { USB_VENDOR_AND_INTERFACE_INFO(0x0a5c, 0xff, 0x01, 0x01) },

Hi Gustavo,

Yes, this works. Well, I don't posses the card personally, but a helpful
person at HP confirmed that the btusb driver with this patch works with
the new card. So you can add

Tested-by: John Hommel <john.hommel@xxxxxx>

BTW, the following ID can also be removed after adding the above match:

{ USB_DEVICE(0x0a5c, 0x21e1) },
{ USB_DEVICE(0x0a5c, 0x21e3) },
{ USB_DEVICE(0x0a5c, 0x21e6) },
{ USB_DEVICE(0x0a5c, 0x21e8) },
{ USB_DEVICE(0x0a5c, 0x21f3) },

Thanks,
Michal
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/