On Sun, 2012-04-22 at 20:01 +0300, Emmanuel Grumbach wrote: > On Sun, Apr 22, 2012 at 19:00, Wey-Yi Guy <wey-yi.w.guy@xxxxxxxxx> wrote: > > Average beacon signal only keep tracked by managed interface, > > give warning and return 0 for the others. > > > > Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@xxxxxxxxx> > > --- > > net/mac80211/util.c | 5 +++++ > > 1 files changed, 5 insertions(+), 0 deletions(-) > > > > diff --git a/net/mac80211/util.c b/net/mac80211/util.c > > index 89c1e5b..2cbcb6a 100644 > > --- a/net/mac80211/util.c > > +++ b/net/mac80211/util.c > > @@ -1799,5 +1799,10 @@ int ieee80211_ave_rssi(struct ieee80211_vif *vif) > > struct ieee80211_sub_if_data *sdata = vif_to_sdata(vif); > > struct ieee80211_if_managed *ifmgd = &sdata->u.mgd; > > > > + if (sdata->vif.type != NL80211_IFTYPE_STATION) { > > + /* non-managed type inferfaces */ > > + WARN_ON(1); > > WARN_ON_ONCE ? > We don't need to flood the kernel log here if (WARN_ON_ONCE(...)) would also be nicer :-) 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