On Tue, Jun 11, 2013 at 5:26 AM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Mon, 2013-06-03 at 10:34 -0700, Ashok Nagarajan wrote: >> Allow user to configure frequency and channel type during mesh join command. >> >> Signed-off-by: Ashok Nagarajan <ashok@xxxxxxxxxxx> >> --- >> mesh.c | 49 ++++++++++++++++++++++++++++++++++++++++++++++--- >> 1 files changed, 46 insertions(+), 3 deletions(-) >> >> diff --git a/mesh.c b/mesh.c >> index 5a09b62..ec202c9 100644 >> --- a/mesh.c >> +++ b/mesh.c >> @@ -431,8 +431,18 @@ static int join_mesh(struct nl80211_state *state, struct nl_cb *cb, >> { >> struct nlattr *container; >> float rate; >> - int bintval, dtim_period; >> + int bintval, dtim_period, i; >> char *end; >> + static const struct { >> + const char *name; >> + unsigned int val; >> + } htmap[] = { >> + { .name = "HT20", .val = NL80211_CHAN_HT20, }, >> + { .name = "HT40+", .val = NL80211_CHAN_HT40PLUS, }, >> + { .name = "HT40-", .val = NL80211_CHAN_HT40MINUS, }, >> + { .name = "NOHT", .val = NL80211_CHAN_NO_HT, }, >> + }; > > I think you should use chandefs already, to allow for VHT. Also, it'd be > good to refactor the channel parsing for this, monitor and IBSS (though > I wonder why I haven't done that already?) > Sure. That makes sense. I will get that in the next version. Thanks, Ashok > johannes > -- Ashok Raj Nagarajan, cozybit Inc. http://www.cozybit.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