2011-01-11 19:49:29

by Arik Nemtsov

Subject: [PATCH] wl12xx: Add channel 14 to list of supported 2ghz channels

Signed-off-by: Arik Nemtsov <[email protected]>
---
drivers/net/wireless/wl12xx/main.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/net/wireless/wl12xx/main.c b/drivers/net/wireless/wl12xx/main.c
index 062247e..a7afdca 100644
--- a/drivers/net/wireless/wl12xx/main.c
+++ b/drivers/net/wireless/wl12xx/main.c
@@ -2305,6 +2305,7 @@ static struct ieee80211_channel wl1271_channels[] = {
{ .hw_value = 11, .center_freq = 2462, .max_power = 25 },
{ .hw_value = 12, .center_freq = 2467, .max_power = 25 },
{ .hw_value = 13, .center_freq = 2472, .max_power = 25 },
+ { .hw_value = 14, .center_freq = 2484, .max_power = 25 },
};

/* mapping to indexes for wl1271_rates */
--
1.7.1

2011-02-03 10:29:42

by Juuso Oikarinen

Subject: Re: [PATCH] wl12xx: Add channel 14 to list of supported 2ghz channels

On Tue, 2011-01-11 at 21:49 +0200, ext Arik Nemtsov wrote:
> Signed-off-by: Arik Nemtsov <[email protected]>
> ---
> drivers/net/wireless/wl12xx/main.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/wireless/wl12xx/main.c b/drivers/net/wireless/wl12xx/main.c
> index 062247e..a7afdca 100644
> --- a/drivers/net/wireless/wl12xx/main.c
> +++ b/drivers/net/wireless/wl12xx/main.c
> @@ -2305,6 +2305,7 @@ static struct ieee80211_channel wl1271_channels[] = {
> { .hw_value = 11, .center_freq = 2462, .max_power = 25 },
> { .hw_value = 12, .center_freq = 2467, .max_power = 25 },
> { .hw_value = 13, .center_freq = 2472, .max_power = 25 },
> + { .hw_value = 14, .center_freq = 2484, .max_power = 25 },
> };
>
> /* mapping to indexes for wl1271_rates */

Do the radio-parameter structs (i.e. the INI-file) already have a field
for channel 14? If not, does not adding this channel introduce strange
problems?

We used to have some 4.9GHz channels on the 11a band, which are
theoretically supported but were lacking the respective radio-parameter
fields, which caused some issues and they had to be removed until the
radio-parameter interface is updated.

-Juuso