antoineco / broadcom-wl

Broadcom Linux hybrid wireless driver (64-bit)
https://www.broadcom.com/support/download-search?pg=Wireless+Embedded+Solutions+and+RF+Components&pf=Legacy+Wireless&pa=Driver&dk=BCM4312&l=true
161 stars 47 forks source link

Fix build on 5.1-rc1 #7

Closed LawnGnome closed 5 years ago

LawnGnome commented 5 years ago

5.1 removes the venerable get_ds() function, which breaks the build for this module. This trivial PR allows compilation. KERNEL_DS has been defined since at least 3.0, so I don't think there are any compatibility concerns there.

Unfortunately, I get panics on 5.1-rc1 with this fix, but I suspect the panics are unrelated to this change. One problem at a time.

antoineco commented 5 years ago

Thanks, nice catch! :raised_hands: I haven't taken the time to test ~5.0 or~ 5.1 with that hardware yet and won't be able to do so in the next 2 weeks. Don't hesitate to update the PR if you manage to pinpoint the reason of the crashes.