On Wed, 2012-08-08 at 14:53 +0300, Victor Goldenshtein wrote:
> Add command to trigger radar detection in the driver/FW.
> Once radar detection is started it should continuously
> monitor for radars as long as the channel active.
> If radar is detected usermode notified with 'radar
> detected' event.
> + if (!local->ops->start_radar_detection)
> + return -EOPNOTSUPP;
> +
> + ret = drv_start_radar_detection(local, sdata, chan);
> + return ret;
Remove the ret variable please.
> +++ b/net/mac80211/iface.c
> @@ -731,6 +731,13 @@ static void ieee80211_do_stop(struct ieee80211_sub_if_data *sdata,
> /* free all potentially still buffered bcast frames */
> local->total_ps_buffered -= skb_queue_len(&sdata->u.ap.ps_bc_buf);
> skb_queue_purge(&sdata->u.ap.ps_bc_buf);
> +
> + /* reset DFS channel availability check */
> + if (local->oper_channel && sdata->wdev.preset_chan) {
> + local->oper_channel->cac_type = 0;
> + /* in case AP hasn't started yet */
> + sdata->wdev.preset_chan->cac_type = 0;
> + }
Why is mac80211 modifying cfg80211 fields? cfg80211 also uses this, no?
This seems strange to me.
> +DEFINE_EVENT(local_sdata_evt, drv_start_radar_detection,
> + TP_PROTO(struct ieee80211_local *local,
> + struct ieee80211_sub_if_data *sdata),
> + TP_ARGS(local, sdata)
> +);
Umm, parameters?
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
