On Sun, 2006-10-29 at 14:35 +0200, [EMAIL PROTECTED] wrote:

> +#define MAC2STR(a) (a)[0], (a)[1], (a)[2], (a)[3], (a)[4], (a)[5]: 
> +#define MACSTR "%02x: %02x:%02x:%02x:%02x:%02x"

such defines should exist already in ieee80211 headers.

> -       /* Update our wireless stats, but do not schedule to often: 
> +       /* Update our wireless stats, but do not schedule to often : 

You could have fixed to->too while at it :)

> -       range->encoding_size[0] = 5;: 
> +       range->encoding_size[0] = KEY_SIZE_WEP40;: 
>         /* 128(104) bits WEP */
> -       range->encoding_size[1] = 13;: 
> +       range->encoding_size[1] = WEP_KEY_LEN;: 
>         /* 256 bits for WPA-PSK */
> -       range->encoding_size[2] = 32;: 
> +       range->encoding_size[2] = SCM_KEY_LEN;: 

Are there no constants for these in the ieee80211 headers? These look
home-grown.

> +               attach->type = DOT11_PKT_BEACON;: 

> +               attach->type = DOT11_PKT_PROBE_RESP;: 

These also look home-grown.


> -                        "%s %s %2.2X: %2.2X:%2.2X:%2.2X:%2.2X:%2.2X %s 
> (%2.2X)",
> +                        "%s %s %2.2X: %2.2X:%2.2X:%2.2X:%2.2X:%2.2X [%2.2x 
> %2.2X %4.4X]",
>                          str,
>                          ((priv->iw_mode == IW_MODE_MASTER) ? "from" : "to"),
>                          a[0], a[1], a[2], a[3], a[4], a[5],

Umm...

johannes
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to