On Mon, 2013-06-24 at 11:49 +0300, Vladimir Kondratiev wrote:
> +/**
> + * enum nl80211_rxmgmt_flags - flags for cfg80211_rx_mgmt
> + *
> + * Defined are bits, to be used in form like
> + * BIT(NL80211_RXMGMT_FLAG_xxx)
> + *
> + * @NL80211_RXMGMT_FLAG_REPLIED: frame was replied by device/driver.
> + */
> +enum nl80211_rxmgmt_flags {
> + NL80211_RXMGMT_FLAG_REPLIED,
> +};
I think you should define the bitmasks, i.e. use
NL80211_RXMGMT_FLAG_REPLIED = 1<<0,
I think that'd be clearer, and userspace doesn't even always have a
BIT() macro.
johannes
--
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
