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

Fix compile errors on 3.x kernels #25

Open
wants to merge 38 commits into
base: master
Choose a base branch
from

Conversation

Darkar25
Copy link

@Darkar25 Darkar25 commented Nov 19, 2021

i ported this driver to 3.4.113(and maybe even older) kernel, tested monitor mode, packet injection, some parameters (tx power, frequency)...seems to work properly...also i changed files according to #11 pull request...
this merge will resolve those issues:
#23 #21

@kimocoder
Copy link
Owner

I will look at this one this week 👍

@Darkar25
Copy link
Author

I will look at this one this week 👍

Ok, thanks)

@kimocoder
Copy link
Owner

btw, which platform you develop for, Android or PC?

@Darkar25
Copy link
Author

Darkar25 commented Dec 1, 2021

btw, which platform you develop for, Android or PC?

well...i develop for android, but it should work on pc too...btw your android compile instruction doesnt quite work...at least when i tried it it just skipped the realtek folder...maybe because the 3.x kernels doesnt have the realtek folder by default, maybe because of config names does not match in all the driver files(i started porting driver before lazerlord merge so i had to change some config names manually)...idk, but i got it working somehow and since then i do not touch that at all otherwise it will break))

@Darkar25
Copy link
Author

Darkar25 commented Dec 4, 2021

yup, idk what i am doing))i just tried to apply your changes to my repo but github did something weird and now all your commits showing here like i changed them and there is now duplicated commits with different changes...
turned out github is harder that i thought...
i don`t know if this pull request can be cleaned up already xd...so much junk in here...

@Darkar25
Copy link
Author

Fixed #26

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