On Mon, 2012-11-19 at 17:48 +0530, Mahesh Palivela wrote: > > + case NL80211_CHAN_WIDTH_160: > > + if (chandef->center_freq1 != control_freq + 70 && > > + chandef->center_freq1 != control_freq + 50 && > > + chandef->center_freq1 != control_freq + 30 && > > + chandef->center_freq1 != control_freq + 10 && > > + chandef->center_freq1 != control_freq - 10 && > > + chandef->center_freq1 != control_freq - 30 && > > + chandef->center_freq1 != control_freq - 50 && > > + chandef->center_freq1 != control_freq - 70) > > + return false; > > Should we create a macro instead of +70 to -70 to take care of all 3 > cases 80, 80p80 and 160? Not sure, I hope we won't duplicate this code anywhere else, so I don't really think we need to? 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