Skip to content
This repository has been archived by the owner on Nov 19, 2018. It is now read-only.

Support for D-Link DWA-182 Rev. D #3

Closed
MaxG87 opened this issue Jun 20, 2018 · 4 comments
Closed

Support for D-Link DWA-182 Rev. D #3

MaxG87 opened this issue Jun 20, 2018 · 4 comments

Comments

@MaxG87
Copy link

MaxG87 commented Jun 20, 2018

Hi,

I fixed the official D-Link driver for DWA-182 Rev. D for Linux 4.16, much like you did here. I had to fix the driver myself since your fixed driver seems to not support the DWA-182. However, your repository was a very big help. Thank you very much already.

I would like to merge both drivers as I do not see why there should be two repositories essentially doing the same thing. However, it seems to be quite an effort to align both drivers. Furthermore, while doing test merges, I noticed several conflicts I could not resolve due to a lack of domain knowledge.

Would you welcome such an effort? If yes, could you help me here and there?

Sincerely,

Max

@cilynx
Copy link
Owner

cilynx commented Jun 21, 2018

Hi Max -- thanks for reaching out. Did you try adding you D-Link card's identifier to usb_intf.c? Looking at your repo, this is the line you need to bring over. I'm curious to know if the driver here works with your card with that update.

It looks like D-Link provided you with a slightly newer driver base than the latest one I've seen: v5.2.4.4_26334.20180126_COEX20171012-5044 vs v5.2.4.4_25643.20171212_COEX20171012-5044. I'll start looking into the differences between the base versions and see what we need to do to get a single repo consolidated that works on both sides.

@cilynx
Copy link
Owner

cilynx commented Jun 21, 2018

I pulled v5.2.4.4_26334.20180126_COEX20171012-5044 from D-Link and backported the patches. I confirmed that the Wsky adapters work with this driver. Can you confirm if your DWA-182 Rev. D works?

https://github.com/cilynx/rtl88x2BU_WiFi_linux_v5.2.4.4_26334.20180126_COEX20171012-5044

@MaxG87
Copy link
Author

MaxG87 commented Jun 21, 2018

I think its better to continue the discussion over at the new repository.

@cilynx
Copy link
Owner

cilynx commented Jun 22, 2018

@cilynx cilynx closed this as completed Jun 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants