Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
On 06/22/2015 10:23 PM, Larry Finger wrote I hope that Xi's posting of the /sys file will help. Unfortunately, I do not have ready access to one of these devices, but I'm making arrangements to gain ssh access tomorrow so that I can test any suggested patches. Beside my earlier patch, I'll send a patch which would modify the generic USB BT device entry in the device table to resolve this. You can choose one of the two patches since they both make the BT works at least on my tablet. If there are other alternate patches, I can help to test them. -- Xi Ruoyao School of Aerospace Science and Technology Xidian University, Xi'an, China -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
On 06/22/2015 05:33 AM, Marcel Holtmann wrote: Marcel, these Realtek devices are causing the most messy setup in a generic driver. And Bluetooth USB support has been really clean and simple for the last 10 years. I really wonder how many ways this gets done wrong before someone gets a clue and gets this right. At the end of the day this USB transport is 15 years old. Anyway, for obvious reasons the content of /sys/kernel/debug/usb/devices should be included here so that someone can actually look at how the USB endpoints are configured. I cannot disagree about the messiness of the Realtek setup. I hope that Xi's posting of the /sys file will help. Unfortunately, I do not have ready access to one of these devices, but I'm making arrangements to gain ssh access tomorrow so that I can test any suggested patches. Larry -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
On 06/22/2015 06:33 PM, Marcel Holtmann wrote:
Hi Larry,
In 'commit a2698a9bf9b0 ("Bluetooth: btusb: Add Realtek
8723A/8723B/8761A/8821A support"), support of some Realtek
devices was added to the Generic Bluetooth USB driver in kernel.
However, these devices are not in the module device table of
btusb.ko, so the kernel wouldn't probe them at all.
To fix this, add four entries in the device table btusb_table,
based on code from
('new' branch).
This enables bluetooth support in the Lenovo Ideapad Yoga 13
which has RTL8723AU USB device, with product ID 0bda:1724.
Signed-off-by: Xi Ruoyao
---
drivers/bluetooth/btusb.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index 3c10d4d..dd87623 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -148,6 +148,12 @@ static const struct usb_device_id btusb_table[] = {
{ USB_DEVICE(0x8087, 0x0a5a),
.driver_info = BTUSB_INTEL_BOOT | BTUSB_BROKEN_ISOC },
+ /* Realtek Bluetooth */
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0bda, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0bd5, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x13d3, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0489, 0xe0, 0x01, 0x01) },
+
I highly doubt these are needed. There is a generic entry for an USB transport
that is already covering these.
Something changed in recent kernels, and the generic USB transport entries do
not seem to be recognized in the case where a wifi and BT device share a single
USB ID. As the OP's tablet has such a case, I suggest that 0bda line be
accepted, but not the others. If I had a computer with an RTL8723AU, I would
have a chance at debugging what changed, but sadly, that is not the case.
these Realtek devices are causing the most messy setup in a generic driver. And
Bluetooth USB support has been really clean and simple for the last 10 years. I
really wonder how many ways this gets done wrong before someone gets a clue and
gets this right. At the end of the day this USB transport is 15 years old.
I agree. Not only BT, but also wifi in RTL8723AU has caused me a lot of
trouble.
Anyway, for obvious reasons the content of /sys/kernel/debug/usb/devices should
be included here so that someone can actually look at how the USB endpoints are
configured.
The contents about my RTL8723AU is here:
T: Bus=03 Lev=02 Prnt=02 Port=03 Cnt=03 Dev#= 5 Spd=480 MxCh= 0
D: Ver= 2.00 Cls=ef(misc ) Sub=02 Prot=01 MxPS=64 #Cfgs= 1
P: Vendor=0bda ProdID=1724 Rev= 2.00
S: Manufacturer=Realtek
S: Product=802.11n WLAN Adapter
S: SerialNumber=00e04c01
C:* #Ifs= 3 Cfg#= 1 Atr=e0 MxPwr=500mA
A: FirstIf#= 0 IfCount= 2 Cls=e0(wlcon) Sub=01 Prot=01
I:* If#= 0 Alt= 0 #EPs= 3 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=81(I) Atr=03(Int.) MxPS= 16 Ivl=1ms
E: Ad=02(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
E: Ad=82(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
I:* If#= 1 Alt= 0 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=03(O) Atr=01(Isoc) MxPS= 0 Ivl=1ms
E: Ad=83(I) Atr=01(Isoc) MxPS= 0 Ivl=1ms
I: If#= 1 Alt= 1 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=03(O) Atr=01(Isoc) MxPS= 9 Ivl=1ms
E: Ad=83(I) Atr=01(Isoc) MxPS= 9 Ivl=1ms
I: If#= 1 Alt= 2 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=03(O) Atr=01(Isoc) MxPS= 17 Ivl=1ms
E: Ad=83(I) Atr=01(Isoc) MxPS= 17 Ivl=1ms
I: If#= 1 Alt= 3 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=03(O) Atr=01(Isoc) MxPS= 25 Ivl=1ms
E: Ad=83(I) Atr=01(Isoc) MxPS= 25 Ivl=1ms
I: If#= 1 Alt= 4 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=03(O) Atr=01(Isoc) MxPS= 33 Ivl=1ms
E: Ad=83(I) Atr=01(Isoc) MxPS= 33 Ivl=1ms
I: If#= 1 Alt= 5 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=03(O) Atr=01(Isoc) MxPS= 49 Ivl=1ms
E: Ad=83(I) Atr=01(Isoc) MxPS= 49 Ivl=1ms
I:* If#= 2 Alt= 0 #EPs= 4 Cls=ff(vend.) Sub=ff Prot=ff Driver=rtl8723au
E: Ad=84(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
E: Ad=05(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
E: Ad=06(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
E: Ad=87(I) Atr=03(Int.) MxPS= 64 Ivl=500us
Regards
Marcel
We can see, since in RTL8723AU the wifi and BT share a single USB ID,
the device
class is 0xef (misc) instead of 0xe0. And the generic entry in btusb
driver only matches
the devices with class 0xe0. That's why RTL8723AU don't work.
I think we should change the entry from USB_DEVICE_INFO to
USB_INTERFACE_INFO.
Then this entry can match all devices which have a Bluetooth interface.
It works well
in my case at least.
--
Xi Ruoyao
School of Aerospace Science and Technology
Xidian University, Xi'an, China
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
Hi Larry,
>>> In 'commit a2698a9bf9b0 ("Bluetooth: btusb: Add Realtek
>>> 8723A/8723B/8761A/8821A support"), support of some Realtek
>>> devices was added to the Generic Bluetooth USB driver in kernel.
>>> However, these devices are not in the module device table of
>>> btusb.ko, so the kernel wouldn't probe them at all.
>>>
>>> To fix this, add four entries in the device table btusb_table,
>>> based on code from
>>> ('new' branch).
>>>
>>> This enables bluetooth support in the Lenovo Ideapad Yoga 13
>>> which has RTL8723AU USB device, with product ID 0bda:1724.
>>>
>>> Signed-off-by: Xi Ruoyao
>>> ---
>>> drivers/bluetooth/btusb.c | 6 ++
>>> 1 file changed, 6 insertions(+)
>>>
>>> diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
>>> index 3c10d4d..dd87623 100644
>>> --- a/drivers/bluetooth/btusb.c
>>> +++ b/drivers/bluetooth/btusb.c
>>> @@ -148,6 +148,12 @@ static const struct usb_device_id btusb_table[] = {
>>> { USB_DEVICE(0x8087, 0x0a5a),
>>> .driver_info = BTUSB_INTEL_BOOT | BTUSB_BROKEN_ISOC },
>>>
>>> + /* Realtek Bluetooth */
>>> + { USB_VENDOR_AND_INTERFACE_INFO(0x0bda, 0xe0, 0x01, 0x01) },
>>> + { USB_VENDOR_AND_INTERFACE_INFO(0x0bd5, 0xe0, 0x01, 0x01) },
>>> + { USB_VENDOR_AND_INTERFACE_INFO(0x13d3, 0xe0, 0x01, 0x01) },
>>> + { USB_VENDOR_AND_INTERFACE_INFO(0x0489, 0xe0, 0x01, 0x01) },
>>> +
>>
>> I highly doubt these are needed. There is a generic entry for an USB
>> transport that is already covering these.
>
> Something changed in recent kernels, and the generic USB transport entries do
> not seem to be recognized in the case where a wifi and BT device share a
> single USB ID. As the OP's tablet has such a case, I suggest that 0bda line
> be accepted, but not the others. If I had a computer with an RTL8723AU, I
> would have a chance at debugging what changed, but sadly, that is not the
> case.
these Realtek devices are causing the most messy setup in a generic driver. And
Bluetooth USB support has been really clean and simple for the last 10 years. I
really wonder how many ways this gets done wrong before someone gets a clue and
gets this right. At the end of the day this USB transport is 15 years old.
Anyway, for obvious reasons the content of /sys/kernel/debug/usb/devices should
be included here so that someone can actually look at how the USB endpoints are
configured.
Regards
Marcel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
On 06/22/2015 10:23 PM, Larry Finger wrote I hope that Xi's posting of the /sys file will help. Unfortunately, I do not have ready access to one of these devices, but I'm making arrangements to gain ssh access tomorrow so that I can test any suggested patches. Beside my earlier patch, I'll send a patch which would modify the generic USB BT device entry in the device table to resolve this. You can choose one of the two patches since they both make the BT works at least on my tablet. If there are other alternate patches, I can help to test them. -- Xi Ruoyao School of Aerospace Science and Technology Xidian University, Xi'an, China -- To unsubscribe from this list: send the line unsubscribe linux-kernel in Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
Hi Larry,
In 'commit a2698a9bf9b0 (Bluetooth: btusb: Add Realtek
8723A/8723B/8761A/8821A support), support of some Realtek
devices was added to the Generic Bluetooth USB driver in kernel.
However, these devices are not in the module device table of
btusb.ko, so the kernel wouldn't probe them at all.
To fix this, add four entries in the device table btusb_table,
based on code from https://github.com/lwfinger/rtl8723au_bt
('new' branch).
This enables bluetooth support in the Lenovo Ideapad Yoga 13
which has RTL8723AU USB device, with product ID 0bda:1724.
Signed-off-by: Xi Ruoyao xry...@outlook.com
---
drivers/bluetooth/btusb.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index 3c10d4d..dd87623 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -148,6 +148,12 @@ static const struct usb_device_id btusb_table[] = {
{ USB_DEVICE(0x8087, 0x0a5a),
.driver_info = BTUSB_INTEL_BOOT | BTUSB_BROKEN_ISOC },
+ /* Realtek Bluetooth */
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0bda, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0bd5, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x13d3, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0489, 0xe0, 0x01, 0x01) },
+
I highly doubt these are needed. There is a generic entry for an USB
transport that is already covering these.
Something changed in recent kernels, and the generic USB transport entries do
not seem to be recognized in the case where a wifi and BT device share a
single USB ID. As the OP's tablet has such a case, I suggest that 0bda line
be accepted, but not the others. If I had a computer with an RTL8723AU, I
would have a chance at debugging what changed, but sadly, that is not the
case.
these Realtek devices are causing the most messy setup in a generic driver. And
Bluetooth USB support has been really clean and simple for the last 10 years. I
really wonder how many ways this gets done wrong before someone gets a clue and
gets this right. At the end of the day this USB transport is 15 years old.
Anyway, for obvious reasons the content of /sys/kernel/debug/usb/devices should
be included here so that someone can actually look at how the USB endpoints are
configured.
Regards
Marcel
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
On 06/22/2015 06:33 PM, Marcel Holtmann wrote:
Hi Larry,
In 'commit a2698a9bf9b0 (Bluetooth: btusb: Add Realtek
8723A/8723B/8761A/8821A support), support of some Realtek
devices was added to the Generic Bluetooth USB driver in kernel.
However, these devices are not in the module device table of
btusb.ko, so the kernel wouldn't probe them at all.
To fix this, add four entries in the device table btusb_table,
based on code from https://github.com/lwfinger/rtl8723au_bt
('new' branch).
This enables bluetooth support in the Lenovo Ideapad Yoga 13
which has RTL8723AU USB device, with product ID 0bda:1724.
Signed-off-by: Xi Ruoyao xry...@outlook.com
---
drivers/bluetooth/btusb.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index 3c10d4d..dd87623 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -148,6 +148,12 @@ static const struct usb_device_id btusb_table[] = {
{ USB_DEVICE(0x8087, 0x0a5a),
.driver_info = BTUSB_INTEL_BOOT | BTUSB_BROKEN_ISOC },
+ /* Realtek Bluetooth */
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0bda, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0bd5, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x13d3, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0489, 0xe0, 0x01, 0x01) },
+
I highly doubt these are needed. There is a generic entry for an USB transport
that is already covering these.
Something changed in recent kernels, and the generic USB transport entries do
not seem to be recognized in the case where a wifi and BT device share a single
USB ID. As the OP's tablet has such a case, I suggest that 0bda line be
accepted, but not the others. If I had a computer with an RTL8723AU, I would
have a chance at debugging what changed, but sadly, that is not the case.
these Realtek devices are causing the most messy setup in a generic driver. And
Bluetooth USB support has been really clean and simple for the last 10 years. I
really wonder how many ways this gets done wrong before someone gets a clue and
gets this right. At the end of the day this USB transport is 15 years old.
I agree. Not only BT, but also wifi in RTL8723AU has caused me a lot of
trouble.
Anyway, for obvious reasons the content of /sys/kernel/debug/usb/devices should
be included here so that someone can actually look at how the USB endpoints are
configured.
The contents about my RTL8723AU is here:
T: Bus=03 Lev=02 Prnt=02 Port=03 Cnt=03 Dev#= 5 Spd=480 MxCh= 0
D: Ver= 2.00 Cls=ef(misc ) Sub=02 Prot=01 MxPS=64 #Cfgs= 1
P: Vendor=0bda ProdID=1724 Rev= 2.00
S: Manufacturer=Realtek
S: Product=802.11n WLAN Adapter
S: SerialNumber=00e04c01
C:* #Ifs= 3 Cfg#= 1 Atr=e0 MxPwr=500mA
A: FirstIf#= 0 IfCount= 2 Cls=e0(wlcon) Sub=01 Prot=01
I:* If#= 0 Alt= 0 #EPs= 3 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=81(I) Atr=03(Int.) MxPS= 16 Ivl=1ms
E: Ad=02(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
E: Ad=82(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
I:* If#= 1 Alt= 0 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=03(O) Atr=01(Isoc) MxPS= 0 Ivl=1ms
E: Ad=83(I) Atr=01(Isoc) MxPS= 0 Ivl=1ms
I: If#= 1 Alt= 1 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=03(O) Atr=01(Isoc) MxPS= 9 Ivl=1ms
E: Ad=83(I) Atr=01(Isoc) MxPS= 9 Ivl=1ms
I: If#= 1 Alt= 2 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=03(O) Atr=01(Isoc) MxPS= 17 Ivl=1ms
E: Ad=83(I) Atr=01(Isoc) MxPS= 17 Ivl=1ms
I: If#= 1 Alt= 3 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=03(O) Atr=01(Isoc) MxPS= 25 Ivl=1ms
E: Ad=83(I) Atr=01(Isoc) MxPS= 25 Ivl=1ms
I: If#= 1 Alt= 4 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=03(O) Atr=01(Isoc) MxPS= 33 Ivl=1ms
E: Ad=83(I) Atr=01(Isoc) MxPS= 33 Ivl=1ms
I: If#= 1 Alt= 5 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=(none)
E: Ad=03(O) Atr=01(Isoc) MxPS= 49 Ivl=1ms
E: Ad=83(I) Atr=01(Isoc) MxPS= 49 Ivl=1ms
I:* If#= 2 Alt= 0 #EPs= 4 Cls=ff(vend.) Sub=ff Prot=ff Driver=rtl8723au
E: Ad=84(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
E: Ad=05(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
E: Ad=06(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
E: Ad=87(I) Atr=03(Int.) MxPS= 64 Ivl=500us
Regards
Marcel
We can see, since in RTL8723AU the wifi and BT share a single USB ID,
the device
class is 0xef (misc) instead of 0xe0. And the generic entry in btusb
driver only matches
the devices with class 0xe0. That's why RTL8723AU don't work.
I think we should change the entry from USB_DEVICE_INFO to
USB_INTERFACE_INFO.
Then this entry can match all devices which have a Bluetooth interface.
It works well
in my case at least.
--
Xi Ruoyao
School of Aerospace Science and Technology
Xidian University, Xi'an, China
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
On 06/22/2015 05:33 AM, Marcel Holtmann wrote: Marcel, these Realtek devices are causing the most messy setup in a generic driver. And Bluetooth USB support has been really clean and simple for the last 10 years. I really wonder how many ways this gets done wrong before someone gets a clue and gets this right. At the end of the day this USB transport is 15 years old. Anyway, for obvious reasons the content of /sys/kernel/debug/usb/devices should be included here so that someone can actually look at how the USB endpoints are configured. I cannot disagree about the messiness of the Realtek setup. I hope that Xi's posting of the /sys file will help. Unfortunately, I do not have ready access to one of these devices, but I'm making arrangements to gain ssh access tomorrow so that I can test any suggested patches. Larry -- To unsubscribe from this list: send the line unsubscribe linux-kernel in Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
On 06/21/2015 12:11 PM, Xi Ruoyao wrote:
On 06/21/2015 09:12 AM, Larry Finger wrote:
On 06/20/2015 04:58 PM, Marcel Holtmann wrote:
Hi Xi,
In 'commit a2698a9bf9b0 ("Bluetooth: btusb: Add Realtek
8723A/8723B/8761A/8821A support"), support of some Realtek
devices was added to the Generic Bluetooth USB driver in kernel.
However, these devices are not in the module device table of
btusb.ko, so the kernel wouldn't probe them at all.
To fix this, add four entries in the device table btusb_table,
based on code from
('new' branch).
This enables bluetooth support in the Lenovo Ideapad Yoga 13
which has RTL8723AU USB device, with product ID 0bda:1724.
Signed-off-by: Xi Ruoyao
---
drivers/bluetooth/btusb.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index 3c10d4d..dd87623 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -148,6 +148,12 @@ static const struct usb_device_id
btusb_table[] = {
{ USB_DEVICE(0x8087, 0x0a5a),
.driver_info = BTUSB_INTEL_BOOT | BTUSB_BROKEN_ISOC },
+/* Realtek Bluetooth */
+{ USB_VENDOR_AND_INTERFACE_INFO(0x0bda, 0xe0, 0x01, 0x01) },
+{ USB_VENDOR_AND_INTERFACE_INFO(0x0bd5, 0xe0, 0x01, 0x01) },
+{ USB_VENDOR_AND_INTERFACE_INFO(0x13d3, 0xe0, 0x01, 0x01) },
+{ USB_VENDOR_AND_INTERFACE_INFO(0x0489, 0xe0, 0x01, 0x01) },
+
I highly doubt these are needed. There is a generic entry for an USB
transport that is already covering these.
Something changed in recent kernels, and the generic USB transport
entries do not seem to be recognized in the case where a wifi and BT
device share a single USB ID. As the OP's tablet has such a case, I
suggest that 0bda line be accepted, but not the others. If I had a
computer with an RTL8723AU, I would have a chance at debugging what
changed, but sadly, that is not the case.
Larry
I viewed the output of 'lsusb -d 0bda:1724 -v | less', and I found
that my RTL8723AU has 'bDeviceClass' 239 (0xef, means 'Miscellaneous
Device')
but 'bInterfaceClass' 224 (0xe0, means 'Wireless').
So maybe the Generic Bluetooth USB device entry should be
'{ USB_INTERFACE_INFO(0xe0, 0x01, 0x01) },'
instead of
'{ USB_DEVICE_INFO(0xe0, 0x01, 0x01) },'
Now I'm compiling this.
This approach works well on my tablet. So, which way is better, edit
Generic Bluetooth USB device entry, or add a new 0bda entry?
And below the Generic Bluetooth USB device entry there is Generic
Bluetooth AMP device entry. Should we use USB_INTERFACE_INFO on this too?
--
Xi Ruoyao
School of Aerospace Science and Technology
Xidian University, Xi'an, China
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
On 06/21/2015 09:12 AM, Larry Finger wrote:
On 06/20/2015 04:58 PM, Marcel Holtmann wrote:
Hi Xi,
In 'commit a2698a9bf9b0 ("Bluetooth: btusb: Add Realtek
8723A/8723B/8761A/8821A support"), support of some Realtek
devices was added to the Generic Bluetooth USB driver in kernel.
However, these devices are not in the module device table of
btusb.ko, so the kernel wouldn't probe them at all.
To fix this, add four entries in the device table btusb_table,
based on code from
('new' branch).
This enables bluetooth support in the Lenovo Ideapad Yoga 13
which has RTL8723AU USB device, with product ID 0bda:1724.
Signed-off-by: Xi Ruoyao
---
drivers/bluetooth/btusb.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index 3c10d4d..dd87623 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -148,6 +148,12 @@ static const struct usb_device_id btusb_table[]
= {
{ USB_DEVICE(0x8087, 0x0a5a),
.driver_info = BTUSB_INTEL_BOOT | BTUSB_BROKEN_ISOC },
+/* Realtek Bluetooth */
+{ USB_VENDOR_AND_INTERFACE_INFO(0x0bda, 0xe0, 0x01, 0x01) },
+{ USB_VENDOR_AND_INTERFACE_INFO(0x0bd5, 0xe0, 0x01, 0x01) },
+{ USB_VENDOR_AND_INTERFACE_INFO(0x13d3, 0xe0, 0x01, 0x01) },
+{ USB_VENDOR_AND_INTERFACE_INFO(0x0489, 0xe0, 0x01, 0x01) },
+
I highly doubt these are needed. There is a generic entry for an USB
transport that is already covering these.
Something changed in recent kernels, and the generic USB transport
entries do not seem to be recognized in the case where a wifi and BT
device share a single USB ID. As the OP's tablet has such a case, I
suggest that 0bda line be accepted, but not the others. If I had a
computer with an RTL8723AU, I would have a chance at debugging what
changed, but sadly, that is not the case.
Larry
I viewed the output of 'lsusb -d 0bda:1724 -v | less', and I found that
my RTL8723AU has 'bDeviceClass' 239 (0xef, means 'Miscellaneous Device')
but 'bInterfaceClass' 224 (0xe0, means 'Wireless').
So maybe the Generic Bluetooth USB device entry should be
'{ USB_INTERFACE_INFO(0xe0, 0x01, 0x01) },'
instead of
'{ USB_DEVICE_INFO(0xe0, 0x01, 0x01) },'
Now I'm compiling this.
--
Xi Ruoyao
School of Aerospace Science and Technology
Xidian University, Xi'an, China
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
On 06/20/2015 04:58 PM, Marcel Holtmann wrote:
Hi Xi,
In 'commit a2698a9bf9b0 ("Bluetooth: btusb: Add Realtek
8723A/8723B/8761A/8821A support"), support of some Realtek
devices was added to the Generic Bluetooth USB driver in kernel.
However, these devices are not in the module device table of
btusb.ko, so the kernel wouldn't probe them at all.
To fix this, add four entries in the device table btusb_table,
based on code from
('new' branch).
This enables bluetooth support in the Lenovo Ideapad Yoga 13
which has RTL8723AU USB device, with product ID 0bda:1724.
Signed-off-by: Xi Ruoyao
---
drivers/bluetooth/btusb.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index 3c10d4d..dd87623 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -148,6 +148,12 @@ static const struct usb_device_id btusb_table[] = {
{ USB_DEVICE(0x8087, 0x0a5a),
.driver_info = BTUSB_INTEL_BOOT | BTUSB_BROKEN_ISOC },
+ /* Realtek Bluetooth */
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0bda, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0bd5, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x13d3, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0489, 0xe0, 0x01, 0x01) },
+
I highly doubt these are needed. There is a generic entry for an USB transport
that is already covering these.
Something changed in recent kernels, and the generic USB transport entries do
not seem to be recognized in the case where a wifi and BT device share a single
USB ID. As the OP's tablet has such a case, I suggest that 0bda line be
accepted, but not the others. If I had a computer with an RTL8723AU, I would
have a chance at debugging what changed, but sadly, that is not the case.
Larry
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
Hi Xi,
> In 'commit a2698a9bf9b0 ("Bluetooth: btusb: Add Realtek
> 8723A/8723B/8761A/8821A support"), support of some Realtek
> devices was added to the Generic Bluetooth USB driver in kernel.
> However, these devices are not in the module device table of
> btusb.ko, so the kernel wouldn't probe them at all.
>
> To fix this, add four entries in the device table btusb_table,
> based on code from
> ('new' branch).
>
> This enables bluetooth support in the Lenovo Ideapad Yoga 13
> which has RTL8723AU USB device, with product ID 0bda:1724.
>
> Signed-off-by: Xi Ruoyao
> ---
> drivers/bluetooth/btusb.c | 6 ++
> 1 file changed, 6 insertions(+)
>
> diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
> index 3c10d4d..dd87623 100644
> --- a/drivers/bluetooth/btusb.c
> +++ b/drivers/bluetooth/btusb.c
> @@ -148,6 +148,12 @@ static const struct usb_device_id btusb_table[] = {
> { USB_DEVICE(0x8087, 0x0a5a),
> .driver_info = BTUSB_INTEL_BOOT | BTUSB_BROKEN_ISOC },
>
> + /* Realtek Bluetooth */
> + { USB_VENDOR_AND_INTERFACE_INFO(0x0bda, 0xe0, 0x01, 0x01) },
> + { USB_VENDOR_AND_INTERFACE_INFO(0x0bd5, 0xe0, 0x01, 0x01) },
> + { USB_VENDOR_AND_INTERFACE_INFO(0x13d3, 0xe0, 0x01, 0x01) },
> + { USB_VENDOR_AND_INTERFACE_INFO(0x0489, 0xe0, 0x01, 0x01) },
> +
I highly doubt these are needed. There is a generic entry for an USB transport
that is already covering these.
Regards
Marcel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
On 06/20/2015 04:58 PM, Marcel Holtmann wrote:
Hi Xi,
In 'commit a2698a9bf9b0 (Bluetooth: btusb: Add Realtek
8723A/8723B/8761A/8821A support), support of some Realtek
devices was added to the Generic Bluetooth USB driver in kernel.
However, these devices are not in the module device table of
btusb.ko, so the kernel wouldn't probe them at all.
To fix this, add four entries in the device table btusb_table,
based on code from https://github.com/lwfinger/rtl8723au_bt
('new' branch).
This enables bluetooth support in the Lenovo Ideapad Yoga 13
which has RTL8723AU USB device, with product ID 0bda:1724.
Signed-off-by: Xi Ruoyao xry...@outlook.com
---
drivers/bluetooth/btusb.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index 3c10d4d..dd87623 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -148,6 +148,12 @@ static const struct usb_device_id btusb_table[] = {
{ USB_DEVICE(0x8087, 0x0a5a),
.driver_info = BTUSB_INTEL_BOOT | BTUSB_BROKEN_ISOC },
+ /* Realtek Bluetooth */
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0bda, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0bd5, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x13d3, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0489, 0xe0, 0x01, 0x01) },
+
I highly doubt these are needed. There is a generic entry for an USB transport
that is already covering these.
Something changed in recent kernels, and the generic USB transport entries do
not seem to be recognized in the case where a wifi and BT device share a single
USB ID. As the OP's tablet has such a case, I suggest that 0bda line be
accepted, but not the others. If I had a computer with an RTL8723AU, I would
have a chance at debugging what changed, but sadly, that is not the case.
Larry
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
Hi Xi,
In 'commit a2698a9bf9b0 (Bluetooth: btusb: Add Realtek
8723A/8723B/8761A/8821A support), support of some Realtek
devices was added to the Generic Bluetooth USB driver in kernel.
However, these devices are not in the module device table of
btusb.ko, so the kernel wouldn't probe them at all.
To fix this, add four entries in the device table btusb_table,
based on code from https://github.com/lwfinger/rtl8723au_bt
('new' branch).
This enables bluetooth support in the Lenovo Ideapad Yoga 13
which has RTL8723AU USB device, with product ID 0bda:1724.
Signed-off-by: Xi Ruoyao xry...@outlook.com
---
drivers/bluetooth/btusb.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index 3c10d4d..dd87623 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -148,6 +148,12 @@ static const struct usb_device_id btusb_table[] = {
{ USB_DEVICE(0x8087, 0x0a5a),
.driver_info = BTUSB_INTEL_BOOT | BTUSB_BROKEN_ISOC },
+ /* Realtek Bluetooth */
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0bda, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0bd5, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x13d3, 0xe0, 0x01, 0x01) },
+ { USB_VENDOR_AND_INTERFACE_INFO(0x0489, 0xe0, 0x01, 0x01) },
+
I highly doubt these are needed. There is a generic entry for an USB transport
that is already covering these.
Regards
Marcel
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
On 06/21/2015 09:12 AM, Larry Finger wrote:
On 06/20/2015 04:58 PM, Marcel Holtmann wrote:
Hi Xi,
In 'commit a2698a9bf9b0 (Bluetooth: btusb: Add Realtek
8723A/8723B/8761A/8821A support), support of some Realtek
devices was added to the Generic Bluetooth USB driver in kernel.
However, these devices are not in the module device table of
btusb.ko, so the kernel wouldn't probe them at all.
To fix this, add four entries in the device table btusb_table,
based on code from https://github.com/lwfinger/rtl8723au_bt
('new' branch).
This enables bluetooth support in the Lenovo Ideapad Yoga 13
which has RTL8723AU USB device, with product ID 0bda:1724.
Signed-off-by: Xi Ruoyao xry...@outlook.com
---
drivers/bluetooth/btusb.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index 3c10d4d..dd87623 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -148,6 +148,12 @@ static const struct usb_device_id btusb_table[]
= {
{ USB_DEVICE(0x8087, 0x0a5a),
.driver_info = BTUSB_INTEL_BOOT | BTUSB_BROKEN_ISOC },
+/* Realtek Bluetooth */
+{ USB_VENDOR_AND_INTERFACE_INFO(0x0bda, 0xe0, 0x01, 0x01) },
+{ USB_VENDOR_AND_INTERFACE_INFO(0x0bd5, 0xe0, 0x01, 0x01) },
+{ USB_VENDOR_AND_INTERFACE_INFO(0x13d3, 0xe0, 0x01, 0x01) },
+{ USB_VENDOR_AND_INTERFACE_INFO(0x0489, 0xe0, 0x01, 0x01) },
+
I highly doubt these are needed. There is a generic entry for an USB
transport that is already covering these.
Something changed in recent kernels, and the generic USB transport
entries do not seem to be recognized in the case where a wifi and BT
device share a single USB ID. As the OP's tablet has such a case, I
suggest that 0bda line be accepted, but not the others. If I had a
computer with an RTL8723AU, I would have a chance at debugging what
changed, but sadly, that is not the case.
Larry
I viewed the output of 'lsusb -d 0bda:1724 -v | less', and I found that
my RTL8723AU has 'bDeviceClass' 239 (0xef, means 'Miscellaneous Device')
but 'bInterfaceClass' 224 (0xe0, means 'Wireless').
So maybe the Generic Bluetooth USB device entry should be
'{ USB_INTERFACE_INFO(0xe0, 0x01, 0x01) },'
instead of
'{ USB_DEVICE_INFO(0xe0, 0x01, 0x01) },'
Now I'm compiling this.
--
Xi Ruoyao
School of Aerospace Science and Technology
Xidian University, Xi'an, China
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] Bluetooth: btusb: Add Realtek devices into module device table
On 06/21/2015 12:11 PM, Xi Ruoyao wrote:
On 06/21/2015 09:12 AM, Larry Finger wrote:
On 06/20/2015 04:58 PM, Marcel Holtmann wrote:
Hi Xi,
In 'commit a2698a9bf9b0 (Bluetooth: btusb: Add Realtek
8723A/8723B/8761A/8821A support), support of some Realtek
devices was added to the Generic Bluetooth USB driver in kernel.
However, these devices are not in the module device table of
btusb.ko, so the kernel wouldn't probe them at all.
To fix this, add four entries in the device table btusb_table,
based on code from https://github.com/lwfinger/rtl8723au_bt
('new' branch).
This enables bluetooth support in the Lenovo Ideapad Yoga 13
which has RTL8723AU USB device, with product ID 0bda:1724.
Signed-off-by: Xi Ruoyao xry...@outlook.com
---
drivers/bluetooth/btusb.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index 3c10d4d..dd87623 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -148,6 +148,12 @@ static const struct usb_device_id
btusb_table[] = {
{ USB_DEVICE(0x8087, 0x0a5a),
.driver_info = BTUSB_INTEL_BOOT | BTUSB_BROKEN_ISOC },
+/* Realtek Bluetooth */
+{ USB_VENDOR_AND_INTERFACE_INFO(0x0bda, 0xe0, 0x01, 0x01) },
+{ USB_VENDOR_AND_INTERFACE_INFO(0x0bd5, 0xe0, 0x01, 0x01) },
+{ USB_VENDOR_AND_INTERFACE_INFO(0x13d3, 0xe0, 0x01, 0x01) },
+{ USB_VENDOR_AND_INTERFACE_INFO(0x0489, 0xe0, 0x01, 0x01) },
+
I highly doubt these are needed. There is a generic entry for an USB
transport that is already covering these.
Something changed in recent kernels, and the generic USB transport
entries do not seem to be recognized in the case where a wifi and BT
device share a single USB ID. As the OP's tablet has such a case, I
suggest that 0bda line be accepted, but not the others. If I had a
computer with an RTL8723AU, I would have a chance at debugging what
changed, but sadly, that is not the case.
Larry
I viewed the output of 'lsusb -d 0bda:1724 -v | less', and I found
that my RTL8723AU has 'bDeviceClass' 239 (0xef, means 'Miscellaneous
Device')
but 'bInterfaceClass' 224 (0xe0, means 'Wireless').
So maybe the Generic Bluetooth USB device entry should be
'{ USB_INTERFACE_INFO(0xe0, 0x01, 0x01) },'
instead of
'{ USB_DEVICE_INFO(0xe0, 0x01, 0x01) },'
Now I'm compiling this.
This approach works well on my tablet. So, which way is better, edit
Generic Bluetooth USB device entry, or add a new 0bda entry?
And below the Generic Bluetooth USB device entry there is Generic
Bluetooth AMP device entry. Should we use USB_INTERFACE_INFO on this too?
--
Xi Ruoyao
School of Aerospace Science and Technology
Xidian University, Xi'an, China
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
Please read the FAQ at http://www.tux.org/lkml/
