Skip to content
This repository has been archived by the owner on Sep 7, 2020. It is now read-only.

nl80211: use the 5Ghz band for dual bands radio #1597

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rmelotte
Copy link
Collaborator

The Turris Omnia comes with a 2.4Ghz and a dual band radio by default.

Up to now we've only been taking into account the first band of each
radio, which means we'll report two 2.4Ghz radios.

Supporting dual bands radio properly requires some refactoring, as the
users of the RadioInfo struct would need to take the bands into
account as well.

To allow the progress on the certification tests for the nl80211 bwl
to continue, when a radio is a dual-band radio assume it's a 5Ghz
radio only.

A Jira task has been added to properly support dual bands radios:
https://jira.prplfoundation.org/browse/PPM-366

Signed-off-by: Raphaël Mélotte [email protected]

The Turris Omnia comes with a 2.4Ghz and a dual band radio by default.

Up to now we've only been taking into account the first band of each
radio, which means we'll report two 2.4Ghz radios.

Supporting dual bands radio properly requires some refactoring, as the
users of the RadioInfo struct would need to take the bands into
account as well.

To allow the progress on the certification tests for the nl80211 bwl
to continue, when a radio is a dual-band radio assume it's a 5Ghz
radio only.

A Jira task has been added to properly support dual bands radios:
https://jira.prplfoundation.org/browse/PPM-366

Signed-off-by: Raphaël Mélotte <[email protected]>
@rmelotte rmelotte added the certification Tasks/bugs involving the certification testbed label Aug 13, 2020
@rmelotte rmelotte self-assigned this Aug 13, 2020
Copy link
Collaborator

@mariomaz mariomaz left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
certification Tasks/bugs involving the certification testbed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants