Patchwork Add vendor specific ID (0a5c 21f3) for BCM20702A0.

login
register
mail settings
Submitter Manoj Iyer
Date Feb. 2, 2012, 4:26 p.m.
Message ID <1328199989-4086-1-git-send-email-manoj.iyer@canonical.com>
Download mbox | patch
Permalink /patch/139152/
State New
Headers show

Comments

Manoj Iyer - Feb. 2, 2012, 4:26 p.m.
T: Bus=01 Lev=02 Prnt=02 Port=03 Cnt=03 Dev#= 5 Spd=12 MxCh= 0
D: Ver= 2.00 Cls=ff(vend.) Sub=01 Prot=01 MxPS=64 #Cfgs= 1
P: Vendor=0a5c ProdID=21f3 Rev=01.12
S: Manufacturer=Broadcom Corp
S: Product=BCM20702A0
S: SerialNumber=74DE2B344A7B
C: #Ifs= 4 Cfg#= 1 Atr=e0 MxPwr=0mA
I: If#= 0 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=01 Prot=01 Driver=(none)
I: If#= 1 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=01 Prot=01 Driver=(none)
I: If#= 2 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=(none)
I: If#= 3 Alt= 0 #EPs= 0 Cls=fe(app. ) Sub=01 Prot=01 Driver=(none)

Signed-off-by: Manoj Iyer <manoj.iyer@canonical.com>
Tested-by: Dennis Chua <dennis.chua@canonical.com>
---
 drivers/bluetooth/btusb.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
Tim Gardner - Feb. 2, 2012, 5:06 p.m.
On 02/02/2012 09:26 AM, Manoj Iyer wrote:
> T: Bus=01 Lev=02 Prnt=02 Port=03 Cnt=03 Dev#= 5 Spd=12 MxCh= 0
> D: Ver= 2.00 Cls=ff(vend.) Sub=01 Prot=01 MxPS=64 #Cfgs= 1
> P: Vendor=0a5c ProdID=21f3 Rev=01.12
> S: Manufacturer=Broadcom Corp
> S: Product=BCM20702A0
> S: SerialNumber=74DE2B344A7B
> C: #Ifs= 4 Cfg#= 1 Atr=e0 MxPwr=0mA
> I: If#= 0 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=01 Prot=01 Driver=(none)
> I: If#= 1 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=01 Prot=01 Driver=(none)
> I: If#= 2 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=(none)
> I: If#= 3 Alt= 0 #EPs= 0 Cls=fe(app. ) Sub=01 Prot=01 Driver=(none)
>
> Signed-off-by: Manoj Iyer<manoj.iyer@canonical.com>
> Tested-by: Dennis Chua<dennis.chua@canonical.com>
> ---
>   drivers/bluetooth/btusb.c |    1 +
>   1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
> index f00f596..10e33be 100644
> --- a/drivers/bluetooth/btusb.c
> +++ b/drivers/bluetooth/btusb.c
> @@ -102,6 +102,7 @@ static struct usb_device_id btusb_table[] = {
>
>   	/* Broadcom BCM20702A0 */
>   	{ USB_DEVICE(0x0a5c, 0x21e3) },
> +	{ USB_DEVICE(0x0a5c, 0x21f3) },
>   	{ USB_DEVICE(0x413c, 0x8197) },
>
>   	{ }	/* Terminating entry */

Have you sent this patch upstream?

scripts/get_maintainer.pl -f drivers/bluetooth/btusb.c
Marcel Holtmann <marcel@holtmann.org> (maintainer:BLUETOOTH DRIVERS)
"Gustavo F. Padovan" <padovan@profusion.mobi> (maintainer:BLUETOOTH DRIVERS)
linux-bluetooth@vger.kernel.org (open list:BLUETOOTH DRIVERS)
linux-kernel@vger.kernel.org (open list)
Manoj Iyer - Feb. 2, 2012, 5:10 p.m.
On Thu, 2 Feb 2012, Tim Gardner wrote:

> On 02/02/2012 09:26 AM, Manoj Iyer wrote:
>> T: Bus=01 Lev=02 Prnt=02 Port=03 Cnt=03 Dev#= 5 Spd=12 MxCh= 0
>> D: Ver= 2.00 Cls=ff(vend.) Sub=01 Prot=01 MxPS=64 #Cfgs= 1
>> P: Vendor=0a5c ProdID=21f3 Rev=01.12
>> S: Manufacturer=Broadcom Corp
>> S: Product=BCM20702A0
>> S: SerialNumber=74DE2B344A7B
>> C: #Ifs= 4 Cfg#= 1 Atr=e0 MxPwr=0mA
>> I: If#= 0 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=01 Prot=01 Driver=(none)
>> I: If#= 1 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=01 Prot=01 Driver=(none)
>> I: If#= 2 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=(none)
>> I: If#= 3 Alt= 0 #EPs= 0 Cls=fe(app. ) Sub=01 Prot=01 Driver=(none)
>> 
>> Signed-off-by: Manoj Iyer<manoj.iyer@canonical.com>
>> Tested-by: Dennis Chua<dennis.chua@canonical.com>
>> ---
>>   drivers/bluetooth/btusb.c |    1 +
>>   1 files changed, 1 insertions(+), 0 deletions(-)
>> 
>> diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
>> index f00f596..10e33be 100644
>> --- a/drivers/bluetooth/btusb.c
>> +++ b/drivers/bluetooth/btusb.c
>> @@ -102,6 +102,7 @@ static struct usb_device_id btusb_table[] = {
>>
>>   	/* Broadcom BCM20702A0 */
>>   	{ USB_DEVICE(0x0a5c, 0x21e3) },
>> +	{ USB_DEVICE(0x0a5c, 0x21f3) },
>>   	{ USB_DEVICE(0x413c, 0x8197) },
>>
>>   	{ }	/* Terminating entry */
>
> Have you sent this patch upstream?

Yes I have, sorry forgot to attach a cover letter for this patch, 
https://lkml.org/lkml/2012/2/2/220, and it was acked by maintainer.

>
> scripts/get_maintainer.pl -f drivers/bluetooth/btusb.c
> Marcel Holtmann <marcel@holtmann.org> (maintainer:BLUETOOTH DRIVERS)
> "Gustavo F. Padovan" <padovan@profusion.mobi> (maintainer:BLUETOOTH DRIVERS)
> linux-bluetooth@vger.kernel.org (open list:BLUETOOTH DRIVERS)
> linux-kernel@vger.kernel.org (open list)
>
> -- 
> Tim Gardner tim.gardner@canonical.com
>
>

--
====================
Manoj Iyer
Ubuntu/Canonical
Hardware Enablement
====================
Tim Gardner - Feb. 2, 2012, 5:38 p.m.
On 02/02/2012 10:10 AM, Manoj Iyer wrote:
> On Thu, 2 Feb 2012, Tim Gardner wrote:
>
>> On 02/02/2012 09:26 AM, Manoj Iyer wrote:
>>> T: Bus=01 Lev=02 Prnt=02 Port=03 Cnt=03 Dev#= 5 Spd=12 MxCh= 0
>>> D: Ver= 2.00 Cls=ff(vend.) Sub=01 Prot=01 MxPS=64 #Cfgs= 1
>>> P: Vendor=0a5c ProdID=21f3 Rev=01.12
>>> S: Manufacturer=Broadcom Corp
>>> S: Product=BCM20702A0
>>> S: SerialNumber=74DE2B344A7B
>>> C: #Ifs= 4 Cfg#= 1 Atr=e0 MxPwr=0mA
>>> I: If#= 0 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=01 Prot=01 Driver=(none)
>>> I: If#= 1 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=01 Prot=01 Driver=(none)
>>> I: If#= 2 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=(none)
>>> I: If#= 3 Alt= 0 #EPs= 0 Cls=fe(app. ) Sub=01 Prot=01 Driver=(none)
>>>
>>> Signed-off-by: Manoj Iyer<manoj.iyer@canonical.com>
>>> Tested-by: Dennis Chua<dennis.chua@canonical.com>
>>> ---
>>> drivers/bluetooth/btusb.c | 1 +
>>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
>>> index f00f596..10e33be 100644
>>> --- a/drivers/bluetooth/btusb.c
>>> +++ b/drivers/bluetooth/btusb.c
>>> @@ -102,6 +102,7 @@ static struct usb_device_id btusb_table[] = {
>>>
>>> /* Broadcom BCM20702A0 */
>>> { USB_DEVICE(0x0a5c, 0x21e3) },
>>> + { USB_DEVICE(0x0a5c, 0x21f3) },
>>> { USB_DEVICE(0x413c, 0x8197) },
>>>
>>> { } /* Terminating entry */
>>
>> Have you sent this patch upstream?
>
> Yes I have, sorry forgot to attach a cover letter for this patch,
> https://lkml.org/lkml/2012/2/2/220, and it was acked by maintainer.
>
>>
>> scripts/get_maintainer.pl -f drivers/bluetooth/btusb.c
>> Marcel Holtmann <marcel@holtmann.org> (maintainer:BLUETOOTH DRIVERS)
>> "Gustavo F. Padovan" <padovan@profusion.mobi> (maintainer:BLUETOOTH
>> DRIVERS)
>> linux-bluetooth@vger.kernel.org (open list:BLUETOOTH DRIVERS)
>> linux-kernel@vger.kernel.org (open list)
>>
>> --
>> Tim Gardner tim.gardner@canonical.com
>>
>>
>
> --
> ====================
> Manoj Iyer
> Ubuntu/Canonical
> Hardware Enablement
> ====================

Once this lands upstream, please follow up with a request to 
stable@vger.kernel.org that this patch be included in stable updates for 
the appropriate kernel versions. I assume 3.0+ ?

In the future if you add 'Cc: stable@vger.kernel.org' it will happen 
automagically.

rtg

Patch

diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index f00f596..10e33be 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -102,6 +102,7 @@  static struct usb_device_id btusb_table[] = {
 
 	/* Broadcom BCM20702A0 */
 	{ USB_DEVICE(0x0a5c, 0x21e3) },
+	{ USB_DEVICE(0x0a5c, 0x21f3) },
 	{ USB_DEVICE(0x413c, 0x8197) },
 
 	{ }	/* Terminating entry */