mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 1/2] add device entry for Broadcom Valentine combo card
@ 2011-10-20  4:58 Cho, Yu-Chen
  2011-10-20  4:58 ` [PATCH 2/2] fix device id for Broadcom SoftSailing Cho, Yu-Chen
  2011-10-31 19:18 ` [PATCH 1/2] add device entry for Broadcom Valentine combo card Gustavo Padovan
  0 siblings, 2 replies; 6+ messages in thread
From: Cho, Yu-Chen @ 2011-10-20  4:58 UTC (permalink / raw)
  To: marcel, padovan, linux-bluetooth, linux-kernel
  Cc: acho, jlee, oneukum, Cho, Yu-Chen

This device declares itself to be vendor specific
It therefore needs to be added to the device table
to make btusb bind.

Signed-off-by: Cho, Yu-Chen <acho@suse.com>
---
 drivers/bluetooth/btusb.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index abfc4ee..5893ba4 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -97,6 +97,10 @@ static struct usb_device_id btusb_table[] = {
 	/* Ericsson with non-standard id */
 	{ USB_DEVICE(0x0bdb, 0x1002) },
 
+	/* Broadcom 20702 Bluetooth 4.0 Adapter */
+	/* Broadcom Valentine 802.11bgn 1x1 Combo HMC */
+	{ USB_DEVICE(0xa5c, 0x21e3) },
+
 	/* Canyon CN-BTU1 with HID interfaces */
 	{ USB_DEVICE(0x0c10, 0x0000) },
 
-- 
1.7.3.4


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 2/2] fix device id for Broadcom SoftSailing
  2011-10-20  4:58 [PATCH 1/2] add device entry for Broadcom Valentine combo card Cho, Yu-Chen
@ 2011-10-20  4:58 ` Cho, Yu-Chen
  2011-10-31 19:18 ` [PATCH 1/2] add device entry for Broadcom Valentine combo card Gustavo Padovan
  1 sibling, 0 replies; 6+ messages in thread
From: Cho, Yu-Chen @ 2011-10-20  4:58 UTC (permalink / raw)
  To: marcel, padovan, linux-bluetooth, linux-kernel
  Cc: acho, jlee, oneukum, Cho, Yu-Chen

Fixed the device VID:PID typo 05ac:21e1 to 0a5c:21e1

Signed-off-by: Cho, Yu-Chen <acho@suse.com>
---
 drivers/bluetooth/btusb.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index 5893ba4..1877e91 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -60,9 +60,6 @@ static struct usb_device_id btusb_table[] = {
 	/* Generic Bluetooth USB device */
 	{ USB_DEVICE_INFO(0xe0, 0x01, 0x01) },
 
-	/* Broadcom SoftSailing reporting vendor specific */
-	{ USB_DEVICE(0x05ac, 0x21e1) },
-
 	/* Apple MacBookPro 7,1 */
 	{ USB_DEVICE(0x05ac, 0x8213) },
 
@@ -98,6 +95,8 @@ static struct usb_device_id btusb_table[] = {
 	{ USB_DEVICE(0x0bdb, 0x1002) },
 
 	/* Broadcom 20702 Bluetooth 4.0 Adapter */
+	/* Broadcom SoftSailing reporting vendor specific */
+	{ USB_DEVICE(0xa5c, 0x21e1) },
 	/* Broadcom Valentine 802.11bgn 1x1 Combo HMC */
 	{ USB_DEVICE(0xa5c, 0x21e3) },
 
-- 
1.7.3.4


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] add device entry for Broadcom Valentine combo card
  2011-10-20  4:58 [PATCH 1/2] add device entry for Broadcom Valentine combo card Cho, Yu-Chen
  2011-10-20  4:58 ` [PATCH 2/2] fix device id for Broadcom SoftSailing Cho, Yu-Chen
@ 2011-10-31 19:18 ` Gustavo Padovan
  2011-11-16  8:24   ` Al Cho
  1 sibling, 1 reply; 6+ messages in thread
From: Gustavo Padovan @ 2011-10-31 19:18 UTC (permalink / raw)
  To: Cho, Yu-Chen; +Cc: marcel, linux-bluetooth, linux-kernel, acho, jlee, oneukum

Hi Yu-Chen,

* Cho, Yu-Chen <acho@suse.com> [2011-10-20 12:58:29 +0800]:

> This device declares itself to be vendor specific
> It therefore needs to be added to the device table
> to make btusb bind.

It would be good to have the output of usb-devices for this specific device in
the commit message.

	Gustavo

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] add device entry for Broadcom Valentine combo card
  2011-10-31 19:18 ` [PATCH 1/2] add device entry for Broadcom Valentine combo card Gustavo Padovan
@ 2011-11-16  8:24   ` Al Cho
  2011-11-16  9:19     ` Marcel Holtmann
  0 siblings, 1 reply; 6+ messages in thread
From: Al Cho @ 2011-11-16  8:24 UTC (permalink / raw)
  To: Cho, Yu-Chen, marcel, linux-bluetooth, linux-kernel, acho, jlee, oneukum

Hi Gustavo,

Sorry for late to reply,

On Tue, Nov 1, 2011 at 3:18 AM, Gustavo Padovan <padovan@profusion.mobi> wrote:
> Hi Yu-Chen,
>
> * Cho, Yu-Chen <acho@suse.com> [2011-10-20 12:58:29 +0800]:
>
>> This device declares itself to be vendor specific
>> It therefore needs to be added to the device table
>> to make btusb bind.
>
> It would be good to have the output of usb-devices for this specific device in
> the commit message.

Would you please kindly explain what is "the output of usb-device" to me?
The patch is for Broadcom Valentine 802.11bgn 1x1 wireless and
bluetooth combo card.

Thanks,
           AL

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] add device entry for Broadcom Valentine combo card
  2011-11-16  8:24   ` Al Cho
@ 2011-11-16  9:19     ` Marcel Holtmann
  2011-12-02  4:48       ` Al Cho
  0 siblings, 1 reply; 6+ messages in thread
From: Marcel Holtmann @ 2011-11-16  9:19 UTC (permalink / raw)
  To: Al Cho; +Cc: linux-bluetooth, linux-kernel, acho, jlee, oneukum

Hi Al,

> > * Cho, Yu-Chen <acho@suse.com> [2011-10-20 12:58:29 +0800]:
> >
> >> This device declares itself to be vendor specific
> >> It therefore needs to be added to the device table
> >> to make btusb bind.
> >
> > It would be good to have the output of usb-devices for this specific device in
> > the commit message.
> 
> Would you please kindly explain what is "the output of usb-device" to me?
> The patch is for Broadcom Valentine 802.11bgn 1x1 wireless and
> bluetooth combo card.

open an xterm and type "usb-devices", then press return. And we want to
see the part about this device.

Regards

Marcel



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] add device entry for Broadcom Valentine combo card
  2011-11-16  9:19     ` Marcel Holtmann
@ 2011-12-02  4:48       ` Al Cho
  0 siblings, 0 replies; 6+ messages in thread
From: Al Cho @ 2011-12-02  4:48 UTC (permalink / raw)
  To: Marcel Holtmann; +Cc: linux-bluetooth, linux-kernel, jlee, oneukum

On Wed, Nov 16, 2011 at 5:19 PM, Marcel Holtmann <marcel@holtmann.org> wrote:
> Hi Al,
>
>> > * Cho, Yu-Chen <acho@suse.com> [2011-10-20 12:58:29 +0800]:
>> >
>> >> This device declares itself to be vendor specific
>> >> It therefore needs to be added to the device table
>> >> to make btusb bind.
>> >
>> > It would be good to have the output of usb-devices for this specific device in
>> > the commit message.
>>
>> Would you please kindly explain what is "the output of usb-device" to me?
>> The patch is for Broadcom Valentine 802.11bgn 1x1 wireless and
>> bluetooth combo card.
>
> open an xterm and type "usb-devices", then press return. And we want to
> see the part about this device.
>
> Regards
>
> Marcel
>
>
>

Hi All,

Really sorry for late to reply again.
List the information below:

T:  Bus=01 Lev=02 Prnt=02 Port=03 Cnt=01 Dev#=  3 Spd=12   MxCh= 0
D:  Ver= 2.00 Cls=ff(vend.) Sub=01 Prot=01 MxPS=64 #Cfgs=  1
P:  Vendor=0a5c ProdID=21e3 Rev= 1.12
S:  Manufacturer=Broadcom Corp
S:  Product=BCM20702A0
S:  SerialNumber=889FFAABE086
C:* #Ifs= 4 Cfg#= 1 Atr=e0 MxPwr=  0mA
I:* If#= 0 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=01 Prot=01 Driver=btusb
E:  Ad=81(I) Atr=03(Int.) MxPS=  16 Ivl=1ms
E:  Ad=82(I) Atr=02(Bulk) MxPS=  64 Ivl=0ms
E:  Ad=02(O) Atr=02(Bulk) MxPS=  64 Ivl=0ms
I:* If#= 1 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=01 Prot=01 Driver=btusb
E:  Ad=83(I) Atr=01(Isoc) MxPS=   0 Ivl=1ms
E:  Ad=03(O) Atr=01(Isoc) MxPS=   0 Ivl=1ms
I:  If#= 1 Alt= 1 #EPs= 2 Cls=ff(vend.) Sub=01 Prot=01 Driver=btusb
E:  Ad=83(I) Atr=01(Isoc) MxPS=   9 Ivl=1ms
E:  Ad=03(O) Atr=01(Isoc) MxPS=   9 Ivl=1ms
I:  If#= 1 Alt= 2 #EPs= 2 Cls=ff(vend.) Sub=01 Prot=01 Driver=btusb
E:  Ad=83(I) Atr=01(Isoc) MxPS=  17 Ivl=1ms
E:  Ad=03(O) Atr=01(Isoc) MxPS=  17 Ivl=1ms
I:  If#= 1 Alt= 3 #EPs= 2 Cls=ff(vend.) Sub=01 Prot=01 Driver=btusb
E:  Ad=83(I) Atr=01(Isoc) MxPS=  25 Ivl=1ms
E:  Ad=03(O) Atr=01(Isoc) MxPS=  25 Ivl=1ms
I:  If#= 1 Alt= 4 #EPs= 2 Cls=ff(vend.) Sub=01 Prot=01 Driver=btusb
E:  Ad=83(I) Atr=01(Isoc) MxPS=  33 Ivl=1ms
E:  Ad=03(O) Atr=01(Isoc) MxPS=  33 Ivl=1ms
I:  If#= 1 Alt= 5 #EPs= 2 Cls=ff(vend.) Sub=01 Prot=01 Driver=btusb
E:  Ad=83(I) Atr=01(Isoc) MxPS=  49 Ivl=1ms
E:  Ad=03(O) Atr=01(Isoc) MxPS=  49 Ivl=1ms
I:* If#= 2 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=(none)
E:  Ad=84(I) Atr=02(Bulk) MxPS=  32 Ivl=0ms
E:  Ad=04(O) Atr=02(Bulk) MxPS=  32 Ivl=0ms
I:* If#= 3 Alt= 0 #EPs= 0 Cls=fe(app. ) Sub=01 Prot=01 Driver=(none)

Sincerely,
             AL

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2011-12-02  4:48 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-10-20  4:58 [PATCH 1/2] add device entry for Broadcom Valentine combo card Cho, Yu-Chen
2011-10-20  4:58 ` [PATCH 2/2] fix device id for Broadcom SoftSailing Cho, Yu-Chen
2011-10-31 19:18 ` [PATCH 1/2] add device entry for Broadcom Valentine combo card Gustavo Padovan
2011-11-16  8:24   ` Al Cho
2011-11-16  9:19     ` Marcel Holtmann
2011-12-02  4:48       ` Al Cho

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome