Added patch to fix compiling with 4.7 kernel. Tested with 4.6 too.

[VGerris]

Basically do :s%/IEEE80211_BAND_5GHZ/NL80211_BAND_5GHZ/g
and :s%/IEEE80211_BAND_5GHZ/NL80211_BAND_5GHZ/g in vim.
Inspired by:
https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c?id=57fbcce37be7c1d2622b56587c10ade00e96afa3
Tested with 4.7 and 4.6 kernel, not further back.

[s7dhansh]

Just tested, it works for 4.4 too.

[longsleep]

Thanks. Before this can be merged it needs to become a patch for dkms and not a direct change of the source code. See the other patches in the Debian folder and activate it for kernel 4.4 and later.

[VGerris]

Do you happen to have some instructions on the actual approach and the patch command used?
I am rather new to this. I can undo the original change and make some patch, but some help would be appreciated :).

[VGerris]

Hi Longsleep, can you contact Alberto Milone to see if you can join efforts? He maintains the driver for Canonical as far as I know, he has a git repo here:
https://github.com/tseliot/bcmwl
There was some overlap in patchfiles, but all should be fixed in that one.
Maybe you can rebase?

Greetings,
Vincent

[longsleep]

Sure - but is my package still needed when Ubuntu has it by default? I can certainly merge but what's the difference then? The whole point of my package was to provide what's not available.

[VGerris]

Hi,

I think it is included, yes.
Maybe it's the extra repositories.
You can check with Alberto Milone to be sure.
The package is listed here:
https://launchpad.net/~broadcom-sta-hackers
but since I asked to commit he mentioned this git repo for that reason:
https://github.com/tseliot/bcmwl

I think the best thing is to have a chat with him and see what the best
option is?

It would be good to merge the efforts :).

Cheers,
Vincent

On Sun, Aug 21, 2016 at 9:44 PM, Simon Eisenmann notifications@github.com
wrote:

Sure - but is my package still needed when Ubuntu has it by default? I can
certainly merge but what's the difference then? The whole point of my
package was to provide what's not available.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#18 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAwdiTGJldhO9DlKFnLPbHJpX4py-C59ks5qiKqcgaJpZM4JH2CH
.

[cooks8]

Just tested for 4.7.2. Worked like a charm. Excellent work

[longsleep]

Please try https://github.com/tseliot/bcmwl instead of my packaging.