* [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