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

Enable MIMO on RX for iwn driver (for Centrino N wireless cards) #411

Merged
merged 1 commit into from
Nov 23, 2020

Conversation

pigworlds
Copy link
Contributor

@pigworlds pigworlds commented Nov 22, 2020

Enable MIMO on the rx path for iwn driver.
This change allow up to MCS 23 rate (216M) on rx. I tested with 2x2:2 and can see MCS 15 rate (144M) on rx.
There is still issues with MIMO on the tx path need to be figure out. Tx rate is still up to MCS 7 rate (72M) and in the real world I only see up to MCS 4 rate (43M).

@zxystd
Copy link
Collaborator

zxystd commented Nov 22, 2020

does the rx rate really increase?

Copy link

@seplarack seplarack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm that this code allows MCS Index to go higher. I also see up to MCS Index: 15.

@pigworlds
Copy link
Contributor Author

pigworlds commented Nov 22, 2020

@GrimFrost Thanks for the confirm.

@zxystd The code base of iwn driver have most of the MIMO rx code ready, it is already specify the right flags in the rxon for all rxchains. However it is not advertise it in the supported MCS rates. This change is to advertise all the supported MCS. The similar code path also exists in the iwm/iwx driver.

For the real-world test, I can see up to 80mbps in the speed test, which is now on-par with iwm driver I believe.

I have another pull request #404 would like take a look.

@pigworlds pigworlds changed the title Enable MIMO on RX for iwn driver Enable MIMO on RX for iwn driver (for Centrino N wireless cards) Nov 22, 2020
@zxystd zxystd self-requested a review November 23, 2020 01:34
@zxystd zxystd merged commit 07eb9cf into OpenIntelWireless:master Nov 23, 2020
@pigworlds pigworlds deleted the iwn-mimo branch November 23, 2020 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants