Hi Gábor, On Fri, 2009-09-11 at 10:46 -0700, Gábor Stefanik wrote: > On Fri, Sep 11, 2009 at 7:38 PM, Reinette Chatre > <reinette.chatre@xxxxxxxxx> wrote: > > From: Johannes Berg <johannes@xxxxxxxxxxxxxxxx> > > > > is_ht can be bool instead of u8, and there's > > no need to use IWL_CHANNEL_WIDTH_* constants > > in supported_chan_width when that could just > > be named is_40mhz instead. > > What about is_ht40? It seems that this structure distinguishes explicitly between being ht (with "is_ht") and the supported channel width (now with "is_40mhz"). To me it seems a bit awkward to have "is_ht" and "is_ht40". As you can tell Johannes has been cleaning this up, so maybe he noticed a way in which this can be clarified. Reinette -- 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
