Hi Bob, On Fri, Jul 22, 2016 at 8:38 PM, Bob Copeland <me@xxxxxxxxxxxxxxx> wrote: > On Fri, Jul 22, 2016 at 07:53:35PM +1000, Julian Calaby wrote: >> Hi Bob, > > Hi! > >> > --- a/interface.c >> > +++ b/interface.c >> > @@ -295,6 +295,10 @@ char *channel_width_name(enum nl80211_chan_width width) >> > return "80+80 MHz"; >> > case NL80211_CHAN_WIDTH_160: >> > return "160 MHz"; >> > + case NL80211_CHAN_WIDTH_5: >> > + return "5 MHz"; >> > + case NL80211_CHAN_WIDTH_10: >> > + return "10 MHz"; >> > default: >> > return "unknown"; >> > } >> >> Judging by the previous two entries, it looks like the case statements >> are sorted, so should these ones therefore be at the top of the list? > > These are sorted by NL80211_CHAN_WIDTH_* attribute value, which makes > a little more sense to me than sorting by the string or numerically by > width, but sure, I can do it either way. That's fine by me: I was asking because it looked like you might have just added them to the bottom. Thanks, -- Julian Calaby Email: julian.calaby@xxxxxxxxx Profile: http://www.google.com/profiles/julian.calaby/ -- 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
