Re: [PATCH 2/6] d80211: create "wifi.h" to define WIFI OUIs

[Jiri Benc <jbenc@suse.cz>]

On Thu, 14 Dec 2006 12:02:16 +0800, Zhu Yi wrote:
> --- /dev/null
> +++ b/net/d80211/wifi.h
> @@ -0,0 +1,28 @@
> +/*
> + * This file defines Wi-Fi(r) OUIs for 80211.o
> + * Copyright 2006, Zhu Yi <yi.zhu@intel.com>  Intel Corp.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#ifndef D802_11_WIFI_H
> +#define D802_11_WIFI_H
> +
> +/* WI-FI Alliance OUI Type and Subtype */
> +enum wifi_oui_type {
> +	WIFI_OUI_TYPE_WPA = 1,
> +	WIFI_OUI_TYPE_WMM = 2,
> +	WIFI_OUI_TYPE_WSC = 4,
> +	WIFI_OUI_TYPE_PSD = 6,
> +};
> +
> +enum wifi_oui_stype_wmm {
> +	WIFI_OUI_STYPE_WMM_INFO = 0,
> +	WIFI_OUI_STYPE_WMM_PARAM = 1,
> +	WIFI_OUI_STYPE_WMM_TSPEC = 2,
> +};
> +
> +
> +#endif /* D802_11_WIFI_H */

AFAIK "wifi" is a trademark and we want to avoid using it. "wlan" seems
to be a better alternative for the prefixes. Also, I don't see a reason
for a separate header file here.

Thanks,

 Jiri

-- 
Jiri Benc
SUSE Labs