On Sat, 2008-11-22 at 22:00 +0200, Jouni Malinen wrote:
> In order to be consistent with NL80211_ATTR_POWER_RULE_MAX_EIRP,
> change NL80211_FREQUENCY_ATTR_MAX_TX_POWER to use mBm and U32 instead
> of dBm and U8. This is a userspace interface change, but the previous
> version had not yet been pushed upstream and there are no userspace
> programs using this yet, so there is justification to get this change in
> as long as it goes in before the previous version gets out.
Thanks.
> Signed-off-by: Jouni Malinen <j@xxxxx>
>
>
> Index: wireless-testing/include/linux/nl80211.h
> ===================================================================
> --- wireless-testing.orig/include/linux/nl80211.h 2008-11-22 21:22:06.000000000 +0200
> +++ wireless-testing/include/linux/nl80211.h 2008-11-22 21:22:54.000000000 +0200
> @@ -508,7 +508,8 @@
> * on this channel in current regulatory domain.
> * @NL80211_FREQUENCY_ATTR_RADAR: Radar detection is mandatory
> * on this channel in current regulatory domain.
> - * @NL80211_FREQUENCY_ATTR_MAX_TX_POWER: Maximum transmission power in dBm.
> + * @NL80211_FREQUENCY_ATTR_MAX_TX_POWER: Maximum transmission power in mBm
> + * (100 * dBm).
> */
> enum nl80211_frequency_attr {
> __NL80211_FREQUENCY_ATTR_INVALID,
> Index: wireless-testing/net/wireless/nl80211.c
> ===================================================================
> --- wireless-testing.orig/net/wireless/nl80211.c 2008-11-22 21:23:10.000000000 +0200
> +++ wireless-testing/net/wireless/nl80211.c 2008-11-22 21:23:51.000000000 +0200
> @@ -198,8 +198,8 @@
> if (chan->flags & IEEE80211_CHAN_RADAR)
> NLA_PUT_FLAG(msg, NL80211_FREQUENCY_ATTR_RADAR);
>
> - NLA_PUT_U8(msg, NL80211_FREQUENCY_ATTR_MAX_TX_POWER,
> - chan->max_power);
> + NLA_PUT_U32(msg, NL80211_FREQUENCY_ATTR_MAX_TX_POWER,
> + DBM_TO_MBM(chan->max_power));
>
> nla_nest_end(msg, nl_freq);
> }
>
Attachment:
signature.asc
Description: This is a digitally signed message part
