On Thu, 2007-06-07 at 22:30 +0200, Jiri Benc wrote:
> > +#define SET_TSINFO_SCHEDULE(i, d) (i.byte3 |= (d << 0) & 0x01)
>
> Please add IEEE80211_ prefix here as well. For consistency and not
> polluting a name space.
Point accepted.
> > +enum wifi_oui_stype_wmm {
> > + WIFI_OUI_STYPE_WMM_INFO = 0,
> > + WIFI_OUI_STYPE_WMM_PARAM = 1,
> > + WIFI_OUI_STYPE_WMM_TSPEC = 2,
> > +};
>
> Please don't invent new useless prefixes. Also, a registered trademark
> doesn't seem to be a good prefix.
I think we have talked this early and now you know correctly WIFI is not
a trademark. You define IEEE80211_STYPE_ASSOC_REQ, etc, but defines
those for Wi-Fi with WLAN prefix? WLAN prefix should be used by symbols
defined by mac80211, for example: WLAN_CAPABILITY_ESS.
Thanks,
-yi
-
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
