Search Linux Wireless

Re: [PATCH 7/7] ath5k: Add documentation for struct ath5k_rate

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

 



2007/11/1, Luis R. Rodriguez <mcgrof@xxxxxxxxx>:
> This adds documentation for struct ath5k_rate. This also removes
> some unused variables, lp_ack_duration and sp_ack_duration which
> are simply unnecessary. We obviously have information about the rest
> of the rate values, we can add more as we go, this just starts this up.
> I'll next target cleaning up the RATE macros, think that may be there
> the other G mode issues are in.
>
> Changes to ath5k.h
> Changes-licensed-under: ISC
>
> Signed-off-by: Luis R. Rodriguez <mcgrof@xxxxxxxxx>
> ---
>  drivers/net/wireless/ath5k/ath5k.h |   58 +++++++++++++++++++++++++++++++----
>  1 files changed, 51 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/net/wireless/ath5k/ath5k.h b/drivers/net/wireless/ath5k/ath5k.h
> index c8ab09a..7147fb4 100644
> --- a/drivers/net/wireless/ath5k/ath5k.h
> +++ b/drivers/net/wireless/ath5k/ath5k.h
> @@ -549,17 +549,61 @@ struct ath5k_athchan_2ghz {
>   *      used by the rate control algorytm on MadWiFi.
>   */
>
> -#define AR5K_MAX_RATES 32 /*max number of rates on the rate table*/
> +/* Max number of rates on the rate table and what it seems
> + * Atheros hardware supports */
> +#define AR5K_MAX_RATES 32
>
> +/**
> + * struct ath5k_rate - rate structure
> + *
> + * This structure is used to get the RX rate or set the TX rate on the
> + * hardware descriptors. It is also used for internal modulation control
> + * and settings.
> + *
> + * @valid: is this a valid rate for the current mode
> + * @modulation: respective mac80211 modulation
> + * @rate_kbps: rate in kbit/s
> + * @rate_code: hardware rate value, used in &struct ath5k_desc, on RX on
> + *     &struct ath5k_rx_status.rs_rate and on TX on
> + *     &struct ath5k_tx_status.ts_rate. Seems the ar5xxx harware supports
> + *     up to 32 rates, indexed by 1-32. This means we really only need
> + *     6 bits for the rate_code.
> + * @dot11_rate: respective IEEE-802.11 rate value
> + * @control_rate: index of rate assumed to be used to send control frames.
> + *     This can be used to set override the value on the rate duration
> + *     registers. This is only useful if we can override in the harware at
> + *     what rate we want to send control frames at. Note that IEEE-802.11
> + *     Ch. 9.6 (after IEEE 802.11g changes) defines the rate at which we
> + *     should send ACK/CTS, if we change this value we can be breaking
> + *     the spec.
> + *
> + * On RX after the &struct ath5k_desc is parsed by the appropriate
> + * ah_proc_rx_desc() the respective hardware rate value is set in
> + * &struct ath5k_rx_status.rs_rate. On TX the desired rate is set in
> + * &struct ath5k_tx_status.ts_rate which is later used to setup the
> + * &struct ath5k_desc correctly. This is the hardware rate map we are
> + * aware of:
> + *
> + * rate_code   1       2       3       4       5       6       7       8
> + * rate_kbps   3000    ?       ?       ?       ?       ?       ?       ?
> + *
> + * rate_code   9       10      11      12      13      14      15      16
> + * rate_kbps   ?       ?       ?       ?       ?       ?       ?       ?
> + *
> + * rate_code   17      18      19      20      21      22      23      24
> + * rate_kbps   ?       ?       ?       ?       ?       ?       ?       ?
> + *
> + * rate_code   25      26      27      28      29      30      31      32
> + * rate_kbps   ?       ?       ?       ?       ?       ?       ?       ?
> + *
> + */
>  struct ath5k_rate {
> -       u8      valid;          /* Valid for rate control */
> +       u8      valid;
>         u32     modulation;
> -       u16     rate_kbps;      /* Rate in kbps used in computetxtime */
> -       u8      rate_code;      /* Rate mapping for h/w descriptors */
> +       u16     rate_kbps;
> +       u8      rate_code;
>         u8      dot11_rate;
> -       u8      control_rate;   /* Rate for management frames -not used */
> -       u16     lp_ack_duration;/* long preamble ACK duration -not used */
> -       u16     sp_ack_duration;/* short preamble ACK duration -not used */
> +       u8      control_rate;
>  };
>
>  /* XXX: GRR all this stuff to get leds blinking ??? (check out setcurmode) */
> --
> 1.5.2.5
>
>

Acked-by: Nick Kossifidis <mickflemm@xxxxxxxxx>

Also checkout setcurmode function, i believe we can live without all
this led-timing stuff and clean up both rate tables and related
functions.

-- 
GPG ID: 0xD21DB2DB
As you read this post global entropy rises. Have Fun ;-)
Nick
-
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