Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Edimax EW-7822UTC to the short list. #485

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rk9qn3j
Copy link

@rk9qn3j rk9qn3j commented Aug 12, 2024

No description provided.

@morrownr
Copy link
Owner

Hi @rk9qn3j

I appreciate the patch but the quality of support for the rtl8822b chip in rtw88 is not good enough at this point for this adapter to go on a list that is basically "the best of the best". Several patches are going into linux-wireless right now that will greatly improve this driver for kernel 6.12 so please remind me about this patch after you test with kernel 6.12.

If you are interested in testing our test repo:

https://github.com/lwfinger/rtw88

@rk9qn3j
Copy link
Author

rk9qn3j commented Aug 13, 2024

Not good enough? Interesting! 🤓 I mean from a user experience (mine), this particular device just works right out of the box. What is it that doesn't work as it is supposed to do?

@morrownr
Copy link
Owner

Not good enough? Interesting! 🤓 I mean from a user experience (mine), this particular device just works right out of the box. What is it that doesn't work as it is supposed to do?

My reply is not about your adapter. My reply is because the rtw88 in-kernel driver is not of good enough quality at this point for a strong recommendation to Linux users to buy any rtl8812bu based adapter. Don't be upset. This situation may change soon so once you are running kernel 6.12, let me know if the performance is good and we can look at merging this patch.

@rk9qn3j
Copy link
Author

rk9qn3j commented Aug 14, 2024

Got it! I'm not upset - just curious. Great project! I'll get back to you after 6.12 has landed. 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants