Re: [PATCH] iw: display 5/10 MHz channel widths

[Bob Copeland <me@bobcopeland.com>]

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.

-- 
Bob Copeland %% http://bobcopeland.com/