On Mon, 2010-11-01 at 13:52 +0200, Luciano Coelho wrote: > + .periodic_start= ieee80211_periodic_start, > + .periodic_stop= ieee80211_periodic_stop, whitespace ... also, _scan really ... > + trace_drv_periodic_stop(local, sdata); > + ret = local->ops->periodic_stop(&local->hw, &sdata->vif); > + trace_drv_return_int(local, ret); > + return ret; why does this have a return value? > @@ -812,6 +812,7 @@ struct ieee80211_local { > struct cfg80211_ssid scan_ssid; > struct cfg80211_scan_request *int_scan_req; > struct cfg80211_scan_request *scan_req, *hw_scan_req; > + struct cfg80211_periodic_request *periodic_req; what do you need to store this for? 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