On 11/08/2011 12:09 PM, Johannes Berg wrote:
On Tue, 2011-11-08 at 11:36 -0800, greearb@xxxxxxxxxxxxxxx wrote:
+ /*
+ * We always need to advert at least MCS0-7, to
+ * be a compliant HT station, for instance
+ */
+ if (((i * 8 + q)>= min_rates)&&
This is a little misleading -- why min_rates when the comment says
MCS0-7?
I let caller determine the min, but comment was to tell why
the min might be set. In APs, the min supported rates are 16, evidently...not
that this code supports APs at the moment...
When this is about local use instead of advertising, then any minimum
is OK.
Want me to just remove the comment entirely?
Thanks,
Ben
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
--
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