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

Blacklisting unwanted USB wifi drivers #383

Closed
wants to merge 1 commit into from
Closed

Blacklisting unwanted USB wifi drivers #383

wants to merge 1 commit into from

Conversation

macmpi
Copy link
Contributor

@macmpi macmpi commented Jul 8, 2019

rtl8192cu and rtl8xxxu come with 4.19.x kernel

rtl8192cu and rtl8xxxu come with 4.19.x kernel
@volumio
Copy link
Owner

volumio commented Jul 9, 2019

Thanks, can we add a version check before those?
I mean: if kernel version is >= 4.19 then blacklist them

What do you think?

@macmpi
Copy link
Contributor Author

macmpi commented Jul 9, 2019

Hi.
Your call: don't know how much you intend to support/roll-back to older kernels: anyway these blacklists would not cause issue with earlier kernels either...
(BTW about older kernels: could we not remove (some of) the older ones and linked stuff?)

@volumio
Copy link
Owner

volumio commented Jul 9, 2019

We need older kernel switches for some OEM builds ;)
To this regard that's why I asked for the switch as well ;)

@macmpi
Copy link
Contributor Author

macmpi commented Jul 9, 2019

Comparing version numbers is a bit tricky... (and those blacklists cause no harm to previous Kernels)

Would this switch make sense to you? (second one, without sort -V)?

@macmpi
Copy link
Contributor Author

macmpi commented Oct 2, 2019

:) forgot I did submit that a while back.
We inherit those blacklists from raspberrypi-sys-mods anyway, so closing.

Note though, rtl8192cu seems better for AP mode under 4.19...and may come back depending on RPi-Distro/raspberrypi-sys-mods#37

@macmpi macmpi closed this Oct 2, 2019
@macmpi macmpi deleted the patch-3 branch October 2, 2019 10:21
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