Search Linux Wireless

Re: [PATCH] ipw2x00: remove ipw2100_rates_11b[]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Stanislav,

On Fri, Feb 10, 2012 at 12:23, Stanislav Yakovlev
<stas.yakovlev@xxxxxxxxx> wrote:
> It's just a duplicate of ipw2100_bg_rates[].

Looks sensible to me.

Reviewed-by: Julian Calaby <julian.calaby@xxxxxxxxx>

> Signed-off-by: Stanislav Yakovlev <stas.yakovlev@xxxxxxxxx>
> ---
>  drivers/net/wireless/ipw2x00/ipw2100.c |   11 ++---------
>  1 files changed, 2 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/net/wireless/ipw2x00/ipw2100.c b/drivers/net/wireless/ipw2x00/ipw2100.c
> index a0e5c21..63567fb 100644
> --- a/drivers/net/wireless/ipw2x00/ipw2100.c
> +++ b/drivers/net/wireless/ipw2x00/ipw2100.c
> @@ -309,13 +309,6 @@ static const long ipw2100_frequencies[] = {
>
>  #define FREQ_COUNT     ARRAY_SIZE(ipw2100_frequencies)
>
> -static const long ipw2100_rates_11b[] = {
> -       1000000,
> -       2000000,
> -       5500000,
> -       11000000
> -};
> -
>  static struct ieee80211_rate ipw2100_bg_rates[] = {
>        { .bitrate = 10 },
>        { .bitrate = 20, .flags = IEEE80211_RATE_SHORT_PREAMBLE },
> @@ -323,7 +316,7 @@ static struct ieee80211_rate ipw2100_bg_rates[] = {
>        { .bitrate = 110, .flags = IEEE80211_RATE_SHORT_PREAMBLE },
>  };
>
> -#define RATE_COUNT ARRAY_SIZE(ipw2100_rates_11b)
> +#define RATE_COUNT ARRAY_SIZE(ipw2100_bg_rates)
>
>  /* Pre-decl until we get the code solid and then we can clean it up */
>  static void ipw2100_tx_send_commands(struct ipw2100_priv *priv);
> @@ -6896,7 +6889,7 @@ static int ipw2100_wx_get_range(struct net_device *dev,
>        range->num_bitrates = RATE_COUNT;
>
>        for (i = 0; i < RATE_COUNT && i < IW_MAX_BITRATES; i++) {
> -               range->bitrate[i] = ipw2100_rates_11b[i];
> +               range->bitrate[i] = ipw2100_bg_rates[i].bitrate * 100 * 1000;
>        }
>
>        range->min_rts = MIN_RTS_THRESHOLD;
> --
> 1.7.2.5
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
Julian Calaby

Email: julian.calaby@xxxxxxxxx
Profile: http://www.google.com/profiles/julian.calaby/
.Plan: http://sites.google.com/site/juliancalaby/
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux