2018-09-04 17:53:16

by Christian Hewitt

[permalink] [raw]
Subject: [PATCH] Bluetooth: btbcm: Add entry for BCM4335C0 UART bluetooth

This patch adds the device ID for the AMPAK AP6335 combo module used
in the 1st generation WeTek Hub Android/LibreELEC HTPC box. The WiFI
chip identifies itself as BCM4339, while Bluetooth identifies itself
as BCM4335 (rev C0):

```
[ 4.864248] Bluetooth: hci0: BCM: chip id 86
[ 4.866388] Bluetooth: hci0: BCM: features 0x2f
[ 4.889317] Bluetooth: hci0: BCM4335C0
[ 4.889332] Bluetooth: hci0: BCM4335C0 (003.001.009) build 0000
[ 9.778383] Bluetooth: hci0: BCM4335C0 (003.001.009) build 0268
```

Output from hciconfig:

```
hci0: Type: Primary Bus: UART
BD Address: 43:39:00:00:1F:AC ACL MTU: 1021:8 SCO MTU: 64:1
UP RUNNING
RX bytes:7567 acl:234 sco:0 events:386 errors:0
TX bytes:53844 acl:77 sco:0 commands:304 errors:0
Features: 0xbf 0xfe 0xcf 0xfe 0xdb 0xff 0x7b 0x87
Packet type: DM1 DM3 DM5 DH1 DH3 DH5 HV1 HV2 HV3
Link policy: RSWITCH SNIFF
Link mode: SLAVE ACCEPT
Name: 'HUB'
Class: 0x0c0000
Service Classes: Rendering, Capturing
Device Class: Miscellaneous,
HCI Version: 4.0 (0x6) Revision: 0x10c
LMP Version: 4.0 (0x6) Subversion: 0x6109
Manufacturer: Broadcom Corporation (15)
```

Signed-off-by: Christian Hewitt <[email protected]>
---
drivers/bluetooth/btbcm.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/bluetooth/btbcm.c b/drivers/bluetooth/btbcm.c
index 99cde1f..e3e4d92 100644
--- a/drivers/bluetooth/btbcm.c
+++ b/drivers/bluetooth/btbcm.c
@@ -324,6 +324,7 @@ static const struct bcm_subver_table bcm_uart_subver_table[] = {
{ 0x4103, "BCM4330B1" }, /* 002.001.003 */
{ 0x410e, "BCM43341B0" }, /* 002.001.014 */
{ 0x4406, "BCM4324B3" }, /* 002.004.006 */
+ { 0x6109, "BCM4335C0" }, /* 003.001.009 */
{ 0x610c, "BCM4354" }, /* 003.001.012 */
{ 0x2122, "BCM4343A0" }, /* 001.001.034 */
{ 0x2209, "BCM43430A1" }, /* 001.002.009 */
--
2.7.4



2018-09-27 10:50:56

by Marcel Holtmann

[permalink] [raw]
Subject: Re: [PATCH] Bluetooth: btbcm: Add entry for BCM4335C0 UART bluetooth

Hi Christian,

> This patch adds the device ID for the AMPAK AP6335 combo module used
> in the 1st generation WeTek Hub Android/LibreELEC HTPC box. The WiFI
> chip identifies itself as BCM4339, while Bluetooth identifies itself
> as BCM4335 (rev C0):
>
> ```
> [ 4.864248] Bluetooth: hci0: BCM: chip id 86
> [ 4.866388] Bluetooth: hci0: BCM: features 0x2f
> [ 4.889317] Bluetooth: hci0: BCM4335C0
> [ 4.889332] Bluetooth: hci0: BCM4335C0 (003.001.009) build 0000
> [ 9.778383] Bluetooth: hci0: BCM4335C0 (003.001.009) build 0268
> ```
>
> Output from hciconfig:
>
> ```
> hci0: Type: Primary Bus: UART
> BD Address: 43:39:00:00:1F:AC ACL MTU: 1021:8 SCO MTU: 64:1
> UP RUNNING
> RX bytes:7567 acl:234 sco:0 events:386 errors:0
> TX bytes:53844 acl:77 sco:0 commands:304 errors:0
> Features: 0xbf 0xfe 0xcf 0xfe 0xdb 0xff 0x7b 0x87
> Packet type: DM1 DM3 DM5 DH1 DH3 DH5 HV1 HV2 HV3
> Link policy: RSWITCH SNIFF
> Link mode: SLAVE ACCEPT
> Name: 'HUB'
> Class: 0x0c0000
> Service Classes: Rendering, Capturing
> Device Class: Miscellaneous,
> HCI Version: 4.0 (0x6) Revision: 0x10c
> LMP Version: 4.0 (0x6) Subversion: 0x6109
> Manufacturer: Broadcom Corporation (15)
> ```
>
> Signed-off-by: Christian Hewitt <[email protected]>
> ---
> drivers/bluetooth/btbcm.c | 1 +
> 1 file changed, 1 insertion(+)

patch has been applied to bluetooth-next tree.

Regards

Marcel