On Thu, 2012-09-27 at 11:59 -0700, Bing Zhao wrote:
> From: Sam Leffler <sleffler@xxxxxxxxxxxx>
>
> Add a flags word to direct and scheduled scan requests; it will
> be used for control of optional behaviours such as flushing the
> bss cache prior to doing a scan.
Why for scheduled scan as well?
> + * @NL80211_ATTR_SCAN_FLAGS: scan request control flags (u32)
One thing that might be useful is to advertise which flags are even
supported at all by a driver, if we add different ones? We might then
ignore the flags that we don't support anyway, but at least userspace
would know that it can't expect flushing (for example) on an older
kernel version and might have to use some workarounds or whatever.
> +/**
> + * enum nl80211_scan_flags - scan request control flags
> + *
> + * Scan request control flags are used to control the handling
> + * of NL80211_CMD_TRIGGER_SCAN and NL80211_CMD_START_SCHED_SCAN
> + * requests.
> + */
> +enum nl80211_scan_flags {
> +};
> +
> /**
> + * enum cfg80211_scan_flags - scan request control flags
> + */
> +enum cfg80211_scan_flags {
> +};
That doesn't make a lot of sense? A single enum seems sufficient?
> + nla_put_u32(msg, NL80211_ATTR_SCAN_FLAGS, req->flags);
Missing error check, also, is there nothing that re-publishes
information about scheduled scans?
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
