On Thu, 15 Feb 2007 15:42:47 +0100, Johannes Berg wrote:
>
> --- wireless-dev.orig/include/net/d80211.h 2007-02-15 13:28:04.547940064 +0100
> +++ wireless-dev/include/net/d80211.h 2007-02-15 13:28:52.417940064 +0100
> @@ -16,6 +16,8 @@
> #include <linux/wireless.h>
> #include <linux/device.h>
> #include <linux/ieee80211.h>
> +#include <net/wireless.h>
> +#include <net/cfg80211.h>
>
> /* Note! Only ieee80211_tx_status_irqsafe() and ieee80211_rx_irqsafe() can be
> * called in hardware interrupt context. The low-level driver must not call any
> @@ -457,8 +459,12 @@ typedef enum {
>
> /* This is driver-visible part of the per-hw state the stack keeps. */
> struct ieee80211_hw {
> - /* these are assigned by d80211, don't write */
> - int index;
> + /* points to the cfg80211 wiphy for this piece. Note
> + * that you must fill in the perm_addr and dev fields
> + * of this structure, use the macros provided below. */
> + struct wiphy *wiphy;
> +
> + /* assigned by d80211, don't write */
> struct ieee80211_conf conf;
>
> /* Pointer to the private area that was
Couldn't we just allow embedding of wiphy into a custom structure?
Something like:
struct ieee80211_hw {
struct wiphy wiphy;
...
}
This way the memory could become less fragmented and less pointer
dereferencing would be needed.
Jiri
--
Jiri Benc
SUSE Labs
-
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
