mbox series

[0/2] Onboard USB device support for RTL8188 2.4GHz USB WiFi module

Message ID 20250403-rtl-onboard-v1-0-10ca9a6a4ee0@posteo.net (mailing list archive)
Headers show
Series Onboard USB device support for RTL8188 2.4GHz USB WiFi module | expand

Message

J. Neuschäfer April 3, 2025, 2:07 p.m. UTC
This patchset adds rtl8188 (usbbda,179) to the onboard_usb_dev driver.
It is found in a set-top box called "Fernsehfee 3.0".

As a side note, this device is currently marked untested in the RTL8XXXU
driver. In my experience it works (tested with a WPA2 home network).

Signed-off-by: J. Neuschäfer <j.ne@posteo.net>
---
J. Neuschäfer (2):
      dt-bindings: net: wireless: Add Realtek RTL8188 USB WiFi
      usb: misc: onboard_dev: Add Realtek RTL8188 WiFi (0bda:0179)

 .../bindings/net/wireless/realtek,rtl8188.yaml     | 51 ++++++++++++++++++++++
 drivers/usb/misc/onboard_usb_dev.c                 |  1 +
 drivers/usb/misc/onboard_usb_dev.h                 |  8 ++++
 3 files changed, 60 insertions(+)
---
base-commit: 38fec10eb60d687e30c8c6b5420d86e8149f7557
change-id: 20250403-rtl-onboard-f38354f0b14b

Best regards,

Comments

Bitterblue Smith April 6, 2025, 10:37 p.m. UTC | #1
On 03/04/2025 17:07, J. Neuschäfer via B4 Relay wrote:
> This patchset adds rtl8188 (usbbda,179) to the onboard_usb_dev driver.

RTL8188 could mean at least five different chips (C, E, F, G).
Should you use a more specific name like RTL8188EU?

> It is found in a set-top box called "Fernsehfee 3.0".
> 
> As a side note, this device is currently marked untested in the RTL8XXXU
> driver. In my experience it works (tested with a WPA2 home network).
> 
> Signed-off-by: J. Neuschäfer <j.ne@posteo.net>
> ---
> J. Neuschäfer (2):
>       dt-bindings: net: wireless: Add Realtek RTL8188 USB WiFi
>       usb: misc: onboard_dev: Add Realtek RTL8188 WiFi (0bda:0179)
> 
>  .../bindings/net/wireless/realtek,rtl8188.yaml     | 51 ++++++++++++++++++++++
>  drivers/usb/misc/onboard_usb_dev.c                 |  1 +
>  drivers/usb/misc/onboard_usb_dev.h                 |  8 ++++
>  3 files changed, 60 insertions(+)
> ---
> base-commit: 38fec10eb60d687e30c8c6b5420d86e8149f7557
> change-id: 20250403-rtl-onboard-f38354f0b14b
> 
> Best regards,
J. Neuschäfer April 7, 2025, 10:56 a.m. UTC | #2
On Mon, Apr 07, 2025 at 01:37:17AM +0300, Bitterblue Smith wrote:
> On 03/04/2025 17:07, J. Neuschäfer via B4 Relay wrote:
> > This patchset adds rtl8188 (usbbda,179) to the onboard_usb_dev driver.
> 
> RTL8188 could mean at least five different chips (C, E, F, G).
> Should you use a more specific name like RTL8188EU?

Yes, that's a good point.

I'm not completely sure which name to pick, though:

 - The chip is marked RTL8188ETV (see photo[1])
 - The RTL8XXXU driver reports it as RTL8188EU


Best regards,
J. Neuschäfer


[1]: https://c3voc.de/gallery/misc/fernsehfee/92QVxQl.jpg.html
Bitterblue Smith April 7, 2025, 1:14 p.m. UTC | #3
On 07/04/2025 13:56, J. Neuschäfer wrote:
> On Mon, Apr 07, 2025 at 01:37:17AM +0300, Bitterblue Smith wrote:
>> On 03/04/2025 17:07, J. Neuschäfer via B4 Relay wrote:
>>> This patchset adds rtl8188 (usbbda,179) to the onboard_usb_dev driver.
>>
>> RTL8188 could mean at least five different chips (C, E, F, G).
>> Should you use a more specific name like RTL8188EU?
> 
> Yes, that's a good point.
> 
> I'm not completely sure which name to pick, though:
> 
>  - The chip is marked RTL8188ETV (see photo[1])
>  - The RTL8XXXU driver reports it as RTL8188EU
> 

Both is good.

> 
> Best regards,
> J. Neuschäfer
> 
> 
> [1]: https://c3voc.de/gallery/misc/fernsehfee/92QVxQl.jpg.html