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

Triband radio support #3357

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Conversation

maurerle
Copy link
Member

I'd like to reopen the changes from #1666 to make way to support #1661

This works quite well and exposes the three wifis correctly on the status-page:

image

radio0, 5GHz, 144 (client wifi)
radio1, 2.4GHz, 11 (client + mesh)
radio2, 5GHz, 44 (mesh)

and also the config mode shows the expected default config for such devices:
image

We might want to:

  • grey out the mesh network operation if a radio does only support DFS channels
  • adjust config to detect how wifis should be distributed best across the channels
  • move whole wireless configuration in uci to gluon.wireless in the future

This is currently focusing on having two 5GHz bands, where at least one does not support all channels.

@herbetom
Copy link
Contributor

I think the mac address problem is as of now unresolved? #1666 (comment)

@maurerle
Copy link
Member Author

maurerle commented Oct 14, 2024

To be honest, I did not really understand the problem with the MAC yet

I see that with 294b291 we add to the prior mac part if our counter overflows.
This seemed to work fine so I did not really understand what still has to be done about that

@T-X
Copy link
Contributor

T-X commented Oct 14, 2024

Would love to see the triband support! Devices with potential triband support and Gluon support I'm aware of: Plasmacloud PA2200 and the LibreRouter. Any other devices people would look forward to for triband support? Also, I guess this was untested with 6GHz so far? (I'd be looking forward to the Banana Pi BPI-R4, for instance, seems to have a dedicated phy for 6GHz for the WiFi7 iPA NIC module)

@maurerle
Copy link
Member Author

I think @Djfe gave a quite extensive list of affected devices in #1661 (comment)

For now, this does not include 6GHz. I have some ideas to add counters to the 6g band as well, but this would create to much noise and untested code (and I am not to familiar with 6GHz frequencies) , so I think we should do this one at a time.

From the list of open todos, I think I can handle the first two eventually, but would definitely need help for a complete overhaul (I think this way stopping the old PR mainly?). If there is someone willing to draft something where I can built on, improve and test, I would be very happy :)

@herbetom
Copy link
Contributor

@T-X Wifi 6E is probably another can of worms but relies on resolving the tripple radio problem #2906

@maurerle The mac address problem is from my understanding that we need more then the current 8 but this has to be carefully considered to not cause collisions (#2061, #1983). Just statically assigning more might not be the best approach

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants