public inbox for linux-wireless@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] wifi: rtl8xxxu: Add additional USB IDs for RTL8192EU devices
@ 2023-12-16 16:52 Zenm Chen
  2023-12-16 18:35 ` Larry Finger
  2023-12-16 20:13 ` [PATCH v2] wifi: rtl8xxxu: Add additional USB IDs for RTL8192EU Zenm Chen
  0 siblings, 2 replies; 3+ messages in thread
From: Zenm Chen @ 2023-12-16 16:52 UTC (permalink / raw)
  To: Jes.Sorensen
  Cc: kvalo, linux-wireless, linux-kernel, pkshih, rtl8821cerfe2,
	zenmchen

Add additional USB IDs found in the vendor driver from
https://github.com/Mange/rtl8192eu-linux-driver to support more
RTL8192EU devices.

Signed-off-by: Zenm Chen <zenmchen@gmail.com>
---
v2:
 - Not to put these USB IDs in CONFIG_RTL8XXXU_UNTESTED
---
 .../net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c    | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c
index 43ee7592bc6e..3f30795f9375 100644
--- a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c
+++ b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_core.c
@@ -7732,12 +7732,24 @@ static const struct usb_device_id dev_table[] = {
 /* TP-Link TL-WN822N v4 */
 {USB_DEVICE_AND_INTERFACE_INFO(0x2357, 0x0108, 0xff, 0xff, 0xff),
 	.driver_info = (unsigned long)&rtl8192eu_fops},
+/* D-Link DWA-131 rev C1 */
+{USB_DEVICE_AND_INTERFACE_INFO(0x2001, 0x3312, 0xff, 0xff, 0xff),
+	.driver_info = (unsigned long)&rtl8192eu_fops},
 /* D-Link DWA-131 rev E1, tested by David Patiño */
 {USB_DEVICE_AND_INTERFACE_INFO(0x2001, 0x3319, 0xff, 0xff, 0xff),
 	.driver_info = (unsigned long)&rtl8192eu_fops},
 /* Tested by Myckel Habets */
 {USB_DEVICE_AND_INTERFACE_INFO(0x2357, 0x0109, 0xff, 0xff, 0xff),
 	.driver_info = (unsigned long)&rtl8192eu_fops},
+/* TP-Link TL-WN8200ND V2 */
+{USB_DEVICE_AND_INTERFACE_INFO(0x2357, 0x0126, 0xff, 0xff, 0xff),
+	.driver_info = (unsigned long)&rtl8192eu_fops},
+/* Mercusys MW300UM */
+{USB_DEVICE_AND_INTERFACE_INFO(0x2c4e, 0x0100, 0xff, 0xff, 0xff),
+	.driver_info = (unsigned long)&rtl8192eu_fops},
+/* Mercusys MW300UH */
+{USB_DEVICE_AND_INTERFACE_INFO(0x2c4e, 0x0104, 0xff, 0xff, 0xff),
+	.driver_info = (unsigned long)&rtl8192eu_fops},
 {USB_DEVICE_AND_INTERFACE_INFO(USB_VENDOR_ID_REALTEK, 0xb720, 0xff, 0xff, 0xff),
 	.driver_info = (unsigned long)&rtl8723bu_fops},
 {USB_DEVICE_AND_INTERFACE_INFO(0x7392, 0xa611, 0xff, 0xff, 0xff),
-- 
2.43.0


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

* Re: [PATCH v2] wifi: rtl8xxxu: Add additional USB IDs for RTL8192EU devices
  2023-12-16 16:52 [PATCH v2] wifi: rtl8xxxu: Add additional USB IDs for RTL8192EU devices Zenm Chen
@ 2023-12-16 18:35 ` Larry Finger
  2023-12-16 20:13 ` [PATCH v2] wifi: rtl8xxxu: Add additional USB IDs for RTL8192EU Zenm Chen
  1 sibling, 0 replies; 3+ messages in thread
From: Larry Finger @ 2023-12-16 18:35 UTC (permalink / raw)
  To: Zenm Chen, Jes.Sorensen
  Cc: kvalo, linux-wireless, linux-kernel, pkshih, rtl8821cerfe2

On 12/16/23 10:52, Zenm Chen wrote:
> Add additional USB IDs found in the vendor driver from
> https://github.com/Mange/rtl8192eu-linux-driver to support more
> RTL8192EU devices.
> 
> Signed-off-by: Zenm Chen <zenmchen@gmail.com>
> ---
> v2:
>   - Not to put these USB IDs in CONFIG_RTL8XXXU_UNTESTED

Have you tested all modes including STA, AP, P2P, etc? If not, then your devices 
belong in the CONFIG_RTL8XXXU_UNTESTED section.

I am also not fond of splitting the rtl8192eu devices into several different 
spots in the table. That only makes it harder to see what devices are supported.

NACKed by Larry Finger <Larry.Finger@lwfinger.net>

Larry


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

* Re: [PATCH v2] wifi: rtl8xxxu: Add additional USB IDs for RTL8192EU
  2023-12-16 16:52 [PATCH v2] wifi: rtl8xxxu: Add additional USB IDs for RTL8192EU devices Zenm Chen
  2023-12-16 18:35 ` Larry Finger
@ 2023-12-16 20:13 ` Zenm Chen
  1 sibling, 0 replies; 3+ messages in thread
From: Zenm Chen @ 2023-12-16 20:13 UTC (permalink / raw)
  To: Larry.Finger
  Cc: Jes.Sorensen, kvalo, linux-kernel, linux-wireless, pkshih,
	rtl8821cerfe2, zenmchen

>> On 12/16/23 10:52, Zenm Chen wrote:
>> Add additional USB IDs found in the vendor driver from
>> https://github.com/Mange/rtl8192eu-linux-driver to support more
>> RTL8192EU devices.
>>
>> Signed-off-by: Zenm Chen <zenmchen@gmail.com>
>> ---
>> v2:
>>   - Not to put these USB IDs in CONFIG_RTL8XXXU_UNTESTED
>
> Have you tested all modes including STA, AP, P2P, etc? If not, then your devices
> belong in the CONFIG_RTL8XXXU_UNTESTED section.

You're right, I haven't tested these four devices, so they should be in the 
CONFIG_RTL8XXXU_UNTESTED section.

Please drop this patch, thanks.

>
> I am also not fond of splitting the rtl8192eu devices into several different
> spots in the table. That only makes it harder to see what devices are supported.
>
> NACKed by Larry Finger <Larry.Finger@lwfinger.net>
>
> Larry

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

end of thread, other threads:[~2023-12-16 20:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-12-16 16:52 [PATCH v2] wifi: rtl8xxxu: Add additional USB IDs for RTL8192EU devices Zenm Chen
2023-12-16 18:35 ` Larry Finger
2023-12-16 20:13 ` [PATCH v2] wifi: rtl8xxxu: Add additional USB IDs for RTL8192EU Zenm Chen

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