On Mon, Sep 22, 2014 at 7:00 PM, Bob Copeland <me@xxxxxxxxxxxxxxx> wrote: > On Sat, Sep 20, 2014 at 02:40:43PM +0800, Chun-Yeow Yeoh via Devel wrote: >> + /* freq */ >> + if (argc > 1 && strcmp(argv[0], "freq") == 0) { >> + argv++; >> + argc--; > > [snip] > >> -COMMAND(mesh, join, "<mesh ID> [mcast-rate <rate in Mbps>]" >> +COMMAND(mesh, join, "<mesh ID> [freq in MHz] [HT20|HT40+|HT40-|NOHT]" > > Should this be something like: > >> +COMMAND(mesh, join, "<mesh ID> [freq <freq in MHz> <HT20|HT40+|HT40-|NOHT>]" > > ...so we know to specify the 'freq' keyword? Yes, it makes sense. I will resubmit v4 later on. ---- Chun-Yeow -- 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