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

Driver will not compile if you enable bluetooth coexistence. #11

Closed
yavincl opened this issue Jun 18, 2020 · 4 comments
Closed

Driver will not compile if you enable bluetooth coexistence. #11

yavincl opened this issue Jun 18, 2020 · 4 comments

Comments

@yavincl
Copy link
Contributor

yavincl commented Jun 18, 2020

Set CONFIG_BT_COEXIST in Makefile to y (it is set at n by default) to build in btcoexist. However the compilation hangs up at a single line. (rtl8192ee/hal/hal_btcoex.c:2591:2:)

 2591 |  rtw_init_timer(&GLBtcBtMpOperTimer, padapter, _btmpoper_timer_hdl, pBtCoexist);
      |  ^~~~~~~~~~~~~~
      |  rtw_init_netdev

Commenting out the line lets it compile, but I doubt that is the intended behaviour.

@lwfinger
Copy link
Owner

There are nearly 10 other instances of what needs to be done to fix rtw_init_timer() for kernels newer than 4.15 in the driver. I will do this, but not immediately as I have more pressing problems.

@lwfinger
Copy link
Owner

That one is fixed and it will now compile; however, there is a ton of code that has never been executed before. Good luck.

@yavincl
Copy link
Contributor Author

yavincl commented Jun 18, 2020

I shall test it and if all is good include it by default in the driver.

@yavincl
Copy link
Contributor Author

yavincl commented Jun 24, 2020

Testing reveals this just makes the driver crap itself near a bluetooth device, and just starts dropping packets forever. I'm submitting a PR to remove this from the makefile before anyone tries to use it.

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

No branches or pull requests

2 participants