Re: [PATCH v3 2/7] mac80211: add radar detection command/event

[Johannes Berg <johannes@sipsolutions.net>]

On Mon, 2012-08-20 at 10:21 +0300, Goldenshtein, Victor wrote:
> On Mon, Aug 13, 2012 at 10:36 AM, Stanislaw Gruszka <sgruszka@redhat.com> wrote:
> > On Sun, Aug 12, 2012 at 10:31:40AM +0300, Goldenshtein, Victor wrote:
> >> On Fri, Aug 10, 2012 at 7:28 PM, Stanislaw Gruszka <sgruszka@redhat.com> wrote:
> >> > On Wed, Aug 08, 2012 at 02:53:38PM +0300, Victor Goldenshtein wrote:
> >> >> --- a/include/net/mac80211.h
> >> >> +++ b/include/net/mac80211.h
> >> >> @@ -2262,6 +2262,10 @@ enum ieee80211_rate_control_changed {
> >> >>   *   The callback will be called before each transmission and upon return
> >> >>   *   mac80211 will transmit the frame right away.
> >> >>   *   The callback is optional and can (should!) sleep.
> >> >> + *
> >> >> +* @start_radar_detection: Start radar detection on current operational
> >> >> + *   channel, once started it will continuously monitor for radars as long
> >> >> + *   as the channel active.
> >> >>   */
> >> >>  struct ieee80211_ops {
> >> >>       void (*tx)(struct ieee80211_hw *hw, struct sk_buff *skb);
> >> >> @@ -2404,6 +2408,10 @@ struct ieee80211_ops {
> >> >>
> >> >>       void    (*mgd_prepare_tx)(struct ieee80211_hw *hw,
> >> >>                                 struct ieee80211_vif *vif);
> >> >> +
> >> >> +     int (*start_radar_detection)(struct ieee80211_hw *hw,
> >> >> +                                  struct ieee80211_vif *vif,
> >> >> +                                  struct ieee80211_channel *chan);
> >> >
> >> > Why we need new callback? I.e. why this can be done by ->config ?
> >> >
> >>
> >> See proposed RFC and previous threads.
> >
> > I can not find mail where this is explained, could you please provide
> > link.
> >
> 
> http://article.gmane.org/gmane.linux.kernel.wireless.general/83760/match=

Well, that doesn't really explain it, but I do think that it's easier
for drivers to implement as a separate callback and we can then also
tell whether it's supported or not.

johannes