On Tue, 2008-11-25 at 21:05 +0200, Jouni Malinen wrote:
> + if (info->attrs[NL80211_ATTR_WIPHY_FREQ]) {
> + int sec_chan_offset = 0;
> +
> + if (!rdev->ops->set_channel) {
> + result = -EOPNOTSUPP;
> + goto bad_res;
> + }
> +
> + if (info->attrs[NL80211_ATTR_WIPHY_SEC_CHAN_OFFSET]) {
> + u8 val = nla_get_u8(
> + info->attrs[
> + NL80211_ATTR_WIPHY_SEC_CHAN_OFFSET]);
> + if (val == NL80211_SEC_CHAN_BELOW)
> + sec_chan_offset = -1;
> + else if (val == NL80211_SEC_CHAN_ABOVE)
> + sec_chan_offset = 1;
> + }
> + result = rdev->ops->set_channel(
> + &rdev->wiphy,
> + nla_get_u32(info->attrs[NL80211_ATTR_WIPHY_FREQ]),
> + sec_chan_offset);
Isn't that missing regulatory enforcement? I'd rather see that in
cfg80211 than re-implemented in all drivers.
johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
