drm/drm_crtc: return -EFAULT on copy_to_user errors
[safe/jmp/linux-2.6] / drivers / staging / wlan-ng / p80211netdev.c
index 8081058..763ab11 100644 (file)
 * --------------------------------------------------------------------
 */
 
-
-/*================================================================*/
-/* System Includes */
-
-
-#include <linux/version.h>
-
 #include <linux/module.h>
 #include <linux/kernel.h>
 #include <linux/sched.h>
@@ -71,9 +64,9 @@
 #include <linux/sockios.h>
 #include <linux/etherdevice.h>
 #include <linux/if_ether.h>
-
-#include <asm/bitops.h>
-#include <asm/uaccess.h>
+#include <linux/byteorder/generic.h>
+#include <linux/bitops.h>
+#include <linux/uaccess.h>
 #include <asm/byteorder.h>
 
 #ifdef SIOCETHTOOL
 #include <net/iw_handler.h>
 #include <net/net_namespace.h>
 
-/*================================================================*/
-/* Project Includes */
-
-#include "wlan_compat.h"
 #include "p80211types.h"
 #include "p80211hdr.h"
 #include "p80211conv.h"
 #include "p80211metastruct.h"
 #include "p80211metadef.h"
 
-/*================================================================*/
-/* Local Constants */
-
-/*================================================================*/
-/* Local Macros */
-
-
-/*================================================================*/
-/* Local Types */
-
-/*================================================================*/
-/* Local Function Declarations */
-
 /* Support functions */
 static void p80211netdev_rx_bh(unsigned long arg);
 
 /* netdevice method functions */
-static int p80211knetdev_init( netdevice_t *netdev);
-static struct net_device_stats* p80211knetdev_get_stats(netdevice_t *netdev);
-static int p80211knetdev_open( netdevice_t *netdev);
-static int p80211knetdev_stop( netdevice_t *netdev );
-static int p80211knetdev_hard_start_xmit( struct sk_buff *skb, netdevice_t *netdev);
+static int p80211knetdev_init(netdevice_t *netdev);
+static struct net_device_stats *p80211knetdev_get_stats(netdevice_t *netdev);
+static int p80211knetdev_open(netdevice_t *netdev);
+static int p80211knetdev_stop(netdevice_t *netdev);
+static int p80211knetdev_hard_start_xmit(struct sk_buff *skb,
+                                        netdevice_t *netdev);
 static void p80211knetdev_set_multicast_list(netdevice_t *dev);
-static int p80211knetdev_do_ioctl(netdevice_t *dev, struct ifreq *ifr, int cmd);
+static int p80211knetdev_do_ioctl(netdevice_t *dev, struct ifreq *ifr,
+                                 int cmd);
 static int p80211knetdev_set_mac_address(netdevice_t *dev, void *addr);
 static void p80211knetdev_tx_timeout(netdevice_t *netdev);
-static int p80211_rx_typedrop( wlandevice_t *wlandev, u16 fc);
+static int p80211_rx_typedrop(wlandevice_t *wlandev, u16 fc);
 
 int wlan_watchdog = 5000;
 module_param(wlan_watchdog, int, 0644);
@@ -134,15 +112,6 @@ int wlan_wext_write = 1;
 module_param(wlan_wext_write, int, 0644);
 MODULE_PARM_DESC(wlan_wext_write, "enable write wireless extensions");
 
-#ifdef WLAN_INCLUDE_DEBUG
-int wlan_debug=0;
-module_param(wlan_debug, int, 0644);
-MODULE_PARM_DESC(wlan_debug, "p80211 debug level");
-#endif
-
-/*================================================================*/
-/* Function Definitions */
-
 /*----------------------------------------------------------------
 * p80211knetdev_init
 *
@@ -155,18 +124,15 @@ MODULE_PARM_DESC(wlan_debug, "p80211 debug level");
 * Returns:
 *      nothing
 ----------------------------------------------------------------*/
-static int p80211knetdev_init( netdevice_t *netdev)
+static int p80211knetdev_init(netdevice_t *netdev)
 {
-       DBFENTER;
        /* Called in response to register_netdev */
        /* This is usually the probe function, but the probe has */
        /* already been done by the MSD and the create_kdev */
        /* function.  All we do here is return success */
-       DBFEXIT;
        return 0;
 }
 
-
 /*----------------------------------------------------------------
 * p80211knetdev_get_stats
 *
@@ -181,20 +147,16 @@ static int p80211knetdev_init( netdevice_t *netdev)
 * Returns:
 *      the address of the statistics structure
 ----------------------------------------------------------------*/
-static struct net_device_stats*
-p80211knetdev_get_stats(netdevice_t *netdev)
+static struct net_device_stats *p80211knetdev_get_stats(netdevice_t * netdev)
 {
-       wlandevice_t    *wlandev = netdev->ml_priv;
-       DBFENTER;
+       wlandevice_t *wlandev = netdev->ml_priv;
 
        /* TODO: review the MIB stats for items that correspond to
-               linux stats */
+          linux stats */
 
-       DBFEXIT;
        return &(wlandev->linux_stats);
 }
 
-
 /*----------------------------------------------------------------
 * p80211knetdev_open
 *
@@ -209,22 +171,19 @@ p80211knetdev_get_stats(netdevice_t *netdev)
 * Returns:
 *      zero on success, non-zero otherwise
 ----------------------------------------------------------------*/
-static int p80211knetdev_open( netdevice_t *netdev )
+static int p80211knetdev_open(netdevice_t *netdev)
 {
-       int             result = 0; /* success */
-       wlandevice_t    *wlandev = netdev->ml_priv;
-
-       DBFENTER;
+       int result = 0;         /* success */
+       wlandevice_t *wlandev = netdev->ml_priv;
 
        /* Check to make sure the MSD is running */
-       if ( wlandev->msdstate != WLAN_MSD_RUNNING ) {
+       if (wlandev->msdstate != WLAN_MSD_RUNNING)
                return -ENODEV;
-       }
 
        /* Tell the MSD to open */
-       if ( wlandev->open != NULL) {
+       if (wlandev->open != NULL) {
                result = wlandev->open(wlandev);
-               if ( result == 0 ) {
+               if (result == 0) {
                        netif_start_queue(wlandev->netdev);
                        wlandev->state = WLAN_DEVICE_OPEN;
                }
@@ -232,11 +191,9 @@ static int p80211knetdev_open( netdevice_t *netdev )
                result = -EAGAIN;
        }
 
-       DBFEXIT;
        return result;
 }
 
-
 /*----------------------------------------------------------------
 * p80211knetdev_stop
 *
@@ -249,21 +206,17 @@ static int p80211knetdev_open( netdevice_t *netdev )
 * Returns:
 *      zero on success, non-zero otherwise
 ----------------------------------------------------------------*/
-static int p80211knetdev_stop( netdevice_t *netdev )
+static int p80211knetdev_stop(netdevice_t *netdev)
 {
-       int             result = 0;
-       wlandevice_t    *wlandev = netdev->ml_priv;
-
-       DBFENTER;
+       int result = 0;
+       wlandevice_t *wlandev = netdev->ml_priv;
 
-       if ( wlandev->close != NULL ) {
+       if (wlandev->close != NULL)
                result = wlandev->close(wlandev);
-       }
 
        netif_stop_queue(wlandev->netdev);
        wlandev->state = WLAN_DEVICE_CLOSED;
 
-       DBFEXIT;
        return result;
 }
 
@@ -280,17 +233,13 @@ static int p80211knetdev_stop( netdevice_t *netdev )
 * Side effects:
 *
 ----------------------------------------------------------------*/
-void
-p80211netdev_rx(wlandevice_t *wlandev, struct sk_buff *skb )
+void p80211netdev_rx(wlandevice_t *wlandev, struct sk_buff *skb)
 {
-       DBFENTER;
-
        /* Enqueue for post-irq processing */
        skb_queue_tail(&wlandev->nsd_rxq, skb);
 
        tasklet_schedule(&wlandev->rx_bh);
 
-        DBFEXIT;
        return;
 }
 
@@ -311,19 +260,17 @@ static void p80211netdev_rx_bh(unsigned long arg)
 {
        wlandevice_t *wlandev = (wlandevice_t *) arg;
        struct sk_buff *skb = NULL;
-       netdevice_t     *dev = wlandev->netdev;
+       netdevice_t *dev = wlandev->netdev;
        p80211_hdr_a3_t *hdr;
        u16 fc;
 
-        DBFENTER;
-
        /* Let's empty our our queue */
-       while ( (skb = skb_dequeue(&wlandev->nsd_rxq)) ) {
+       while ((skb = skb_dequeue(&wlandev->nsd_rxq))) {
                if (wlandev->state == WLAN_DEVICE_OPEN) {
 
                        if (dev->type != ARPHRD_ETHER) {
                                /* RAW frame; we shouldn't convert it */
-                               // XXX Append the Prism Header here instead.
+                               /* XXX Append the Prism Header here instead. */
 
                                /* set up various data fields */
                                skb->dev = dev;
@@ -338,8 +285,8 @@ static void p80211netdev_rx_bh(unsigned long arg)
                                netif_rx_ni(skb);
                                continue;
                        } else {
-                               hdr = (p80211_hdr_a3_t *)skb->data;
-                               fc = ieee2host16(hdr->fc);
+                               hdr = (p80211_hdr_a3_t *) skb->data;
+                               fc = le16_to_cpu(hdr->fc);
                                if (p80211_rx_typedrop(wlandev, fc)) {
                                        dev_kfree_skb(skb);
                                        continue;
@@ -348,7 +295,9 @@ static void p80211netdev_rx_bh(unsigned long arg)
                                /* perform mcast filtering */
                                if (wlandev->netdev->flags & IFF_ALLMULTI) {
                                        /* allow my local address through */
-                                       if (memcmp(hdr->a1, wlandev->netdev->dev_addr, ETH_ALEN) != 0) {
+                                       if (memcmp
+                                           (hdr->a1, wlandev->netdev->dev_addr,
+                                            ETH_ALEN) != 0) {
                                                /* but reject anything else that isn't multicast */
                                                if (!(hdr->a1[0] & 0x01)) {
                                                        dev_kfree_skb(skb);
@@ -357,23 +306,22 @@ static void p80211netdev_rx_bh(unsigned long arg)
                                        }
                                }
 
-                               if ( skb_p80211_to_ether(wlandev, wlandev->ethconv, skb) == 0 ) {
+                               if (skb_p80211_to_ether
+                                   (wlandev, wlandev->ethconv, skb) == 0) {
                                        skb->dev->last_rx = jiffies;
                                        wlandev->linux_stats.rx_packets++;
-                                       wlandev->linux_stats.rx_bytes += skb->len;
+                                       wlandev->linux_stats.rx_bytes +=
+                                           skb->len;
                                        netif_rx_ni(skb);
                                        continue;
                                }
-                               WLAN_LOG_DEBUG(1, "p80211_to_ether failed.\n");
+                               pr_debug("p80211_to_ether failed.\n");
                        }
                }
                dev_kfree_skb(skb);
        }
-
-        DBFEXIT;
 }
 
-
 /*----------------------------------------------------------------
 * p80211knetdev_hard_start_xmit
 *
@@ -393,21 +341,19 @@ static void p80211netdev_rx_bh(unsigned long arg)
 * Returns:
 *      zero on success, non-zero on failure.
 ----------------------------------------------------------------*/
-static int p80211knetdev_hard_start_xmit( struct sk_buff *skb, netdevice_t *netdev)
+static int p80211knetdev_hard_start_xmit(struct sk_buff *skb,
+                                        netdevice_t *netdev)
 {
-       int             result = 0;
-       int             txresult = -1;
-       wlandevice_t    *wlandev = netdev->ml_priv;
-       p80211_hdr_t    p80211_hdr;
+       int result = 0;
+       int txresult = -1;
+       wlandevice_t *wlandev = netdev->ml_priv;
+       p80211_hdr_t p80211_hdr;
        p80211_metawep_t p80211_wep;
 
-       DBFENTER;
-
-       if (skb == NULL) {
-               return 0;
-       }
+       if (skb == NULL)
+               return NETDEV_TX_OK;
 
-        if (wlandev->state != WLAN_DEVICE_OPEN) {
+       if (wlandev->state != WLAN_DEVICE_OPEN) {
                result = 1;
                goto failed;
        }
@@ -415,8 +361,8 @@ static int p80211knetdev_hard_start_xmit( struct sk_buff *skb, netdevice_t *netd
        memset(&p80211_hdr, 0, sizeof(p80211_hdr_t));
        memset(&p80211_wep, 0, sizeof(p80211_metawep_t));
 
-       if ( netif_queue_stopped(netdev) ) {
-               WLAN_LOG_DEBUG(1, "called when queue stopped.\n");
+       if (netif_queue_stopped(netdev)) {
+               pr_debug("called when queue stopped.\n");
                result = 1;
                goto failed;
        }
@@ -424,7 +370,7 @@ static int p80211knetdev_hard_start_xmit( struct sk_buff *skb, netdevice_t *netd
        netif_stop_queue(netdev);
 
        /* Check to see that a valid mode is set */
-       switch( wlandev->macmode ) {
+       switch (wlandev->macmode) {
        case WLAN_MACMODE_IBSS_STA:
        case WLAN_MACMODE_ESS_STA:
        case WLAN_MACMODE_ESS_AP:
@@ -434,10 +380,10 @@ static int p80211knetdev_hard_start_xmit( struct sk_buff *skb, netdevice_t *netd
                 * and return success .
                 * TODO: we need a saner way to handle this
                 */
-               if(skb->protocol != ETH_P_80211_RAW) {
+               if (skb->protocol != ETH_P_80211_RAW) {
                        netif_start_queue(wlandev->netdev);
-                       WLAN_LOG_NOTICE(
-                               "Tx attempt prior to association, frame dropped.\n");
+                       printk(KERN_NOTICE
+                              "Tx attempt prior to association, frame dropped.\n");
                        wlandev->linux_stats.tx_dropped++;
                        result = 0;
                        goto failed;
@@ -446,7 +392,7 @@ static int p80211knetdev_hard_start_xmit( struct sk_buff *skb, netdevice_t *netd
        }
 
        /* Check for raw transmits */
-       if(skb->protocol == ETH_P_80211_RAW) {
+       if (skb->protocol == ETH_P_80211_RAW) {
                if (!capable(CAP_NET_ADMIN)) {
                        result = 1;
                        goto failed;
@@ -455,15 +401,17 @@ static int p80211knetdev_hard_start_xmit( struct sk_buff *skb, netdevice_t *netd
                memcpy(&p80211_hdr, skb->data, sizeof(p80211_hdr_t));
                skb_pull(skb, sizeof(p80211_hdr_t));
        } else {
-               if ( skb_ether_to_p80211(wlandev, wlandev->ethconv, skb, &p80211_hdr, &p80211_wep) != 0 ) {
+               if (skb_ether_to_p80211
+                   (wlandev, wlandev->ethconv, skb, &p80211_hdr,
+                    &p80211_wep) != 0) {
                        /* convert failed */
-                       WLAN_LOG_DEBUG(1, "ether_to_80211(%d) failed.\n",
-                                       wlandev->ethconv);
+                       pr_debug("ether_to_80211(%d) failed.\n",
+                                wlandev->ethconv);
                        result = 1;
                        goto failed;
                }
        }
-       if ( wlandev->txframe == NULL ) {
+       if (wlandev->txframe == NULL) {
                result = 1;
                goto failed;
        }
@@ -476,41 +424,39 @@ static int p80211knetdev_hard_start_xmit( struct sk_buff *skb, netdevice_t *netd
 
        txresult = wlandev->txframe(wlandev, skb, &p80211_hdr, &p80211_wep);
 
-       if ( txresult == 0) {
+       if (txresult == 0) {
                /* success and more buf */
                /* avail, re: hw_txdata */
                netif_wake_queue(wlandev->netdev);
-               result = 0;
-       } else if ( txresult == 1 ) {
+               result = NETDEV_TX_OK;
+       } else if (txresult == 1) {
                /* success, no more avail */
-               WLAN_LOG_DEBUG(3, "txframe success, no more bufs\n");
+               pr_debug("txframe success, no more bufs\n");
                /* netdev->tbusy = 1;  don't set here, irqhdlr */
                /*   may have already cleared it */
-               result = 0;
-       } else if ( txresult == 2 ) {
+               result = NETDEV_TX_OK;
+       } else if (txresult == 2) {
                /* alloc failure, drop frame */
-               WLAN_LOG_DEBUG(3, "txframe returned alloc_fail\n");
-               result = 1;
+               pr_debug("txframe returned alloc_fail\n");
+               result = NETDEV_TX_BUSY;
        } else {
                /* buffer full or queue busy, drop frame. */
-               WLAN_LOG_DEBUG(3, "txframe returned full or busy\n");
-               result = 1;
+               pr_debug("txframe returned full or busy\n");
+               result = NETDEV_TX_BUSY;
        }
 
- failed:
+failed:
        /* Free up the WEP buffer if it's not the same as the skb */
        if ((p80211_wep.data) && (p80211_wep.data != skb->data))
-               kfree(p80211_wep.data);
+               kzfree(p80211_wep.data);
 
        /* we always free the skb here, never in a lower level. */
        if (!result)
                dev_kfree_skb(skb);
 
-       DBFEXIT;
        return result;
 }
 
-
 /*----------------------------------------------------------------
 * p80211knetdev_set_multicast_list
 *
@@ -525,16 +471,13 @@ static int p80211knetdev_hard_start_xmit( struct sk_buff *skb, netdevice_t *netd
 ----------------------------------------------------------------*/
 static void p80211knetdev_set_multicast_list(netdevice_t *dev)
 {
-       wlandevice_t    *wlandev = dev->ml_priv;
-
-       DBFENTER;
+       wlandevice_t *wlandev = dev->ml_priv;
 
        /* TODO:  real multicast support as well */
 
        if (wlandev->set_multicast_list)
                wlandev->set_multicast_list(wlandev, dev);
 
-       DBFEXIT;
 }
 
 #ifdef SIOCETHTOOL
@@ -559,9 +502,6 @@ static int p80211netdev_ethtool(wlandevice_t *wlandev, void __user *useraddr)
                snprintf(info.version, sizeof(info.version), "%s",
                         WLAN_RELEASE);
 
-               // info.fw_version
-               // info.bus_info
-
                if (copy_to_user(useraddr, &info, sizeof(info)))
                        return -EFAULT;
                return 0;
@@ -577,7 +517,7 @@ static int p80211netdev_ethtool(wlandevice_t *wlandev, void __user *useraddr)
                }
 
                if (copy_to_user(useraddr, &edata, sizeof(edata)))
-                        return -EFAULT;
+                       return -EFAULT;
                return 0;
        }
 #endif
@@ -616,45 +556,46 @@ static int p80211netdev_ethtool(wlandevice_t *wlandev, void __user *useraddr)
 ----------------------------------------------------------------*/
 static int p80211knetdev_do_ioctl(netdevice_t *dev, struct ifreq *ifr, int cmd)
 {
-       int                     result = 0;
-       p80211ioctl_req_t       *req = (p80211ioctl_req_t*)ifr;
-       wlandevice_t            *wlandev = dev->ml_priv;
-       u8                      *msgbuf;
-       DBFENTER;
+       int result = 0;
+       p80211ioctl_req_t *req = (p80211ioctl_req_t *) ifr;
+       wlandevice_t *wlandev = dev->ml_priv;
+       u8 *msgbuf;
 
-       WLAN_LOG_DEBUG(2, "rx'd ioctl, cmd=%d, len=%d\n", cmd, req->len);
+       pr_debug("rx'd ioctl, cmd=%d, len=%d\n", cmd, req->len);
 
 #ifdef SIOCETHTOOL
        if (cmd == SIOCETHTOOL) {
-               result = p80211netdev_ethtool(wlandev, (void __user *) ifr->ifr_data);
+               result =
+                   p80211netdev_ethtool(wlandev, (void __user *)ifr->ifr_data);
                goto bail;
        }
 #endif
 
        /* Test the magic, assume ifr is good if it's there */
-       if ( req->magic != P80211_IOCTL_MAGIC ) {
+       if (req->magic != P80211_IOCTL_MAGIC) {
                result = -ENOSYS;
                goto bail;
        }
 
-       if ( cmd == P80211_IFTEST ) {
+       if (cmd == P80211_IFTEST) {
                result = 0;
                goto bail;
-       } else if ( cmd != P80211_IFREQ ) {
+       } else if (cmd != P80211_IFREQ) {
                result = -ENOSYS;
                goto bail;
        }
 
        /* Allocate a buf of size req->len */
-       if ((msgbuf = kmalloc( req->len, GFP_KERNEL))) {
-               if ( copy_from_user( msgbuf, (void __user *) req->data, req->len) ) {
+       msgbuf = kmalloc(req->len, GFP_KERNEL);
+       if (msgbuf) {
+               if (copy_from_user(msgbuf, (void __user *)req->data, req->len))
                        result = -EFAULT;
-               } else {
-                       result = p80211req_dorequest( wlandev, msgbuf);
-               }
+               else
+                       result = p80211req_dorequest(wlandev, msgbuf);
 
-               if ( result == 0 ) {
-                       if ( copy_to_user( (void __user *) req->data, msgbuf, req->len)) {
+               if (result == 0) {
+                       if (copy_to_user
+                           ((void __user *)req->data, msgbuf, req->len)) {
                                result = -EFAULT;
                        }
                }
@@ -663,9 +604,7 @@ static int p80211knetdev_do_ioctl(netdevice_t *dev, struct ifreq *ifr, int cmd)
                result = -ENOMEM;
        }
 bail:
-       DBFEXIT;
-
-       return result; /* If allocate,copyfrom or copyto fails, return errno */
+       return result;          /* If allocate,copyfrom or copyto fails, return errno */
 }
 
 /*----------------------------------------------------------------
@@ -695,22 +634,20 @@ bail:
 ----------------------------------------------------------------*/
 static int p80211knetdev_set_mac_address(netdevice_t *dev, void *addr)
 {
-       struct sockaddr                 *new_addr = addr;
-       p80211msg_dot11req_mibset_t     dot11req;
-       p80211item_unk392_t             *mibattr;
-       p80211item_pstr6_t              *macaddr;
-       p80211item_uint32_t             *resultcode;
+       struct sockaddr *new_addr = addr;
+       p80211msg_dot11req_mibset_t dot11req;
+       p80211item_unk392_t *mibattr;
+       p80211item_pstr6_t *macaddr;
+       p80211item_uint32_t *resultcode;
        int result = 0;
 
-       DBFENTER;
        /* If we're running, we don't allow MAC address changes */
-       if (netif_running(dev)) {
+       if (netif_running(dev))
                return -EBUSY;
-       }
 
        /* Set up some convenience pointers. */
        mibattr = &dot11req.mibattribute;
-       macaddr = (p80211item_pstr6_t*)&mibattr->data;
+       macaddr = (p80211item_pstr6_t *) &mibattr->data;
        resultcode = &dot11req.resultcode;
 
        /* Set up a dot11req_mibset */
@@ -718,8 +655,7 @@ static int p80211knetdev_set_mac_address(netdevice_t *dev, void *addr)
        dot11req.msgcode = DIDmsg_dot11req_mibset;
        dot11req.msglen = sizeof(p80211msg_dot11req_mibset_t);
        memcpy(dot11req.devname,
-               ((wlandevice_t *)dev->ml_priv)->name,
-               WLAN_DEVNAMELEN_MAX - 1);
+              ((wlandevice_t *) dev->ml_priv)->name, WLAN_DEVNAMELEN_MAX - 1);
 
        /* Set up the mibattribute argument */
        mibattr->did = DIDmsg_dot11req_mibset_mibattribute;
@@ -739,40 +675,48 @@ static int p80211knetdev_set_mac_address(netdevice_t *dev, void *addr)
        resultcode->data = 0;
 
        /* now fire the request */
-       result = p80211req_dorequest(dev->ml_priv, (u8 *)&dot11req);
+       result = p80211req_dorequest(dev->ml_priv, (u8 *) &dot11req);
 
        /* If the request wasn't successful, report an error and don't
         * change the netdev address
         */
-       if ( result != 0 || resultcode->data != P80211ENUM_resultcode_success) {
-               WLAN_LOG_ERROR(
-               "Low-level driver failed dot11req_mibset(dot11MACAddress).\n");
+       if (result != 0 || resultcode->data != P80211ENUM_resultcode_success) {
+               printk(KERN_ERR
+                      "Low-level driver failed dot11req_mibset(dot11MACAddress).\n");
                result = -EADDRNOTAVAIL;
        } else {
                /* everything's ok, change the addr in netdev */
                memcpy(dev->dev_addr, new_addr->sa_data, dev->addr_len);
        }
 
-       DBFEXIT;
        return result;
 }
 
 static int wlan_change_mtu(netdevice_t *dev, int new_mtu)
 {
-       DBFENTER;
-       // 2312 is max 802.11 payload, 20 is overhead, (ether + llc +snap)
-       // and another 8 for wep.
-        if ( (new_mtu < 68) || (new_mtu > (2312 - 20 - 8)))
-                return -EINVAL;
+       /* 2312 is max 802.11 payload, 20 is overhead, (ether + llc +snap)
+          and another 8 for wep. */
+       if ((new_mtu < 68) || (new_mtu > (2312 - 20 - 8)))
+               return -EINVAL;
 
-        dev->mtu = new_mtu;
+       dev->mtu = new_mtu;
 
-       DBFEXIT;
-
-        return 0;
+       return 0;
 }
 
-
+static const struct net_device_ops p80211_netdev_ops = {
+       .ndo_init = p80211knetdev_init,
+       .ndo_open = p80211knetdev_open,
+       .ndo_stop = p80211knetdev_stop,
+       .ndo_get_stats = p80211knetdev_get_stats,
+       .ndo_start_xmit = p80211knetdev_hard_start_xmit,
+       .ndo_set_multicast_list = p80211knetdev_set_multicast_list,
+       .ndo_do_ioctl = p80211knetdev_do_ioctl,
+       .ndo_set_mac_address = p80211knetdev_set_mac_address,
+       .ndo_tx_timeout = p80211knetdev_tx_timeout,
+       .ndo_change_mtu = wlan_change_mtu,
+       .ndo_validate_addr = eth_validate_addr,
+};
 
 /*----------------------------------------------------------------
 * wlan_setup
@@ -798,10 +742,8 @@ static int wlan_change_mtu(netdevice_t *dev, int new_mtu)
 ----------------------------------------------------------------*/
 int wlan_setup(wlandevice_t *wlandev)
 {
-       int             result = 0;
-       netdevice_t     *dev;
-
-       DBFENTER;
+       int result = 0;
+       netdevice_t *dev;
 
        /* Set up the wlandev */
        wlandev->state = WLAN_DEVICE_CLOSED;
@@ -811,49 +753,24 @@ int wlan_setup(wlandevice_t *wlandev)
        /* Set up the rx queue */
        skb_queue_head_init(&wlandev->nsd_rxq);
        tasklet_init(&wlandev->rx_bh,
-                    p80211netdev_rx_bh,
-                    (unsigned long)wlandev);
+                    p80211netdev_rx_bh, (unsigned long)wlandev);
 
        /* Allocate and initialize the struct device */
-       dev = alloc_netdev(0,"wlan%d",ether_setup);
-       if ( dev == NULL ) {
-               WLAN_LOG_ERROR("Failed to alloc netdev.\n");
+       dev = alloc_netdev(0, "wlan%d", ether_setup);
+       if (dev == NULL) {
+               printk(KERN_ERR "Failed to alloc netdev.\n");
                result = 1;
        } else {
                wlandev->netdev = dev;
                dev->ml_priv = wlandev;
-               dev->hard_start_xmit =  p80211knetdev_hard_start_xmit;
-               dev->get_stats =        p80211knetdev_get_stats;
-#ifdef HAVE_PRIVATE_IOCTL
-               dev->do_ioctl =         p80211knetdev_do_ioctl;
-#endif
-#ifdef HAVE_MULTICAST
-               dev->set_multicast_list = p80211knetdev_set_multicast_list;
-#endif
-               dev->init =             p80211knetdev_init;
-               dev->open =             p80211knetdev_open;
-               dev->stop =             p80211knetdev_stop;
+               dev->netdev_ops = &p80211_netdev_ops;
 
-#if (WIRELESS_EXT < 21)
-               dev->get_wireless_stats = p80211wext_get_wireless_stats;
-#endif
                dev->wireless_handlers = &p80211wext_handler_def;
 
                netif_stop_queue(dev);
-#ifdef HAVE_CHANGE_MTU
-               dev->change_mtu = wlan_change_mtu;
-#endif
-#ifdef HAVE_SET_MAC_ADDR
-               dev->set_mac_address =  p80211knetdev_set_mac_address;
-#endif
-#ifdef HAVE_TX_TIMEOUT
-               dev->tx_timeout      =  &p80211knetdev_tx_timeout;
-               dev->watchdog_timeo  =  (wlan_watchdog * HZ) / 1000;
-#endif
                netif_carrier_off(dev);
        }
 
-       DBFEXIT;
        return result;
 }
 
@@ -880,26 +797,21 @@ int wlan_setup(wlandevice_t *wlandev)
 ----------------------------------------------------------------*/
 int wlan_unsetup(wlandevice_t *wlandev)
 {
-       int             result = 0;
-
-       DBFENTER;
+       int result = 0;
 
        tasklet_kill(&wlandev->rx_bh);
 
-       if (wlandev->netdev == NULL ) {
-               WLAN_LOG_ERROR("called without wlandev->netdev set.\n");
+       if (wlandev->netdev == NULL) {
+               printk(KERN_ERR "called without wlandev->netdev set.\n");
                result = 1;
        } else {
                free_netdev(wlandev->netdev);
                wlandev->netdev = NULL;
        }
 
-       DBFEXIT;
        return 0;
 }
 
-
-
 /*----------------------------------------------------------------
 * register_wlandev
 *
@@ -921,19 +833,15 @@ int wlan_unsetup(wlandevice_t *wlandev)
 ----------------------------------------------------------------*/
 int register_wlandev(wlandevice_t *wlandev)
 {
-       int             i = 0;
-
-       DBFENTER;
+       int i = 0;
 
        i = register_netdev(wlandev->netdev);
        if (i)
                return i;
 
-       DBFEXIT;
        return 0;
 }
 
-
 /*----------------------------------------------------------------
 * unregister_wlandev
 *
@@ -955,20 +863,15 @@ int unregister_wlandev(wlandevice_t *wlandev)
 {
        struct sk_buff *skb;
 
-       DBFENTER;
-
        unregister_netdev(wlandev->netdev);
 
        /* Now to clean out the rx queue */
-       while ( (skb = skb_dequeue(&wlandev->nsd_rxq)) ) {
+       while ((skb = skb_dequeue(&wlandev->nsd_rxq)))
                dev_kfree_skb(skb);
-       }
 
-       DBFEXIT;
        return 0;
 }
 
-
 /*----------------------------------------------------------------
 * p80211netdev_hwremoved
 *
@@ -1001,18 +904,13 @@ int unregister_wlandev(wlandevice_t *wlandev)
 ----------------------------------------------------------------*/
 void p80211netdev_hwremoved(wlandevice_t *wlandev)
 {
-       DBFENTER;
        wlandev->hwremoved = 1;
-       if ( wlandev->state == WLAN_DEVICE_OPEN) {
+       if (wlandev->state == WLAN_DEVICE_OPEN)
                netif_stop_queue(wlandev->netdev);
-       }
 
        netif_device_detach(wlandev->netdev);
-
-       DBFEXIT;
 }
 
-
 /*----------------------------------------------------------------
 * p80211_rx_typedrop
 *
@@ -1034,28 +932,27 @@ void p80211netdev_hwremoved(wlandevice_t *wlandev)
 * Call context:
 *      interrupt
 ----------------------------------------------------------------*/
-static int p80211_rx_typedrop( wlandevice_t *wlandev, u16 fc)
+static int p80211_rx_typedrop(wlandevice_t *wlandev, u16 fc)
 {
-       u16     ftype;
-       u16     fstype;
-       int     drop = 0;
+       u16 ftype;
+       u16 fstype;
+       int drop = 0;
        /* Classify frame, increment counter */
        ftype = WLAN_GET_FC_FTYPE(fc);
        fstype = WLAN_GET_FC_FSTYPE(fc);
 #if 0
-       WLAN_LOG_DEBUG(4,
-               "rx_typedrop : ftype=%d fstype=%d.\n", ftype, fstype);
+       pr_debug("rx_typedrop : ftype=%d fstype=%d.\n", ftype, fstype);
 #endif
-       switch ( ftype ) {
+       switch (ftype) {
        case WLAN_FTYPE_MGMT:
                if ((wlandev->netdev->flags & IFF_PROMISC) ||
-                       (wlandev->netdev->flags & IFF_ALLMULTI)) {
+                   (wlandev->netdev->flags & IFF_ALLMULTI)) {
                        drop = 1;
                        break;
                }
-               WLAN_LOG_DEBUG(3, "rx'd mgmt:\n");
+               pr_debug("rx'd mgmt:\n");
                wlandev->rx.mgmt++;
-               switch( fstype ) {
+               switch (fstype) {
                case WLAN_FSTYPE_ASSOCREQ:
                        /* printk("assocreq"); */
                        wlandev->rx.assocreq++;
@@ -1111,13 +1008,13 @@ static int p80211_rx_typedrop( wlandevice_t *wlandev, u16 fc)
 
        case WLAN_FTYPE_CTL:
                if ((wlandev->netdev->flags & IFF_PROMISC) ||
-                       (wlandev->netdev->flags & IFF_ALLMULTI)) {
+                   (wlandev->netdev->flags & IFF_ALLMULTI)) {
                        drop = 1;
                        break;
                }
-               WLAN_LOG_DEBUG(3, "rx'd ctl:\n");
+               pr_debug("rx'd ctl:\n");
                wlandev->rx.ctl++;
-               switch( fstype ) {
+               switch (fstype) {
                case WLAN_FSTYPE_PSPOLL:
                        /* printk("pspoll"); */
                        wlandev->rx.pspoll++;
@@ -1153,7 +1050,7 @@ static int p80211_rx_typedrop( wlandevice_t *wlandev, u16 fc)
 
        case WLAN_FTYPE_DATA:
                wlandev->rx.data++;
-               switch( fstype ) {
+               switch (fstype) {
                case WLAN_FSTYPE_DATAONLY:
                        wlandev->rx.dataonly++;
                        break;
@@ -1167,19 +1064,19 @@ static int p80211_rx_typedrop( wlandevice_t *wlandev, u16 fc)
                        wlandev->rx.data__cfack_cfpoll++;
                        break;
                case WLAN_FSTYPE_NULL:
-                       WLAN_LOG_DEBUG(3, "rx'd data:null\n");
+                       pr_debug("rx'd data:null\n");
                        wlandev->rx.null++;
                        break;
                case WLAN_FSTYPE_CFACK:
-                       WLAN_LOG_DEBUG(3, "rx'd data:cfack\n");
+                       pr_debug("rx'd data:cfack\n");
                        wlandev->rx.cfack++;
                        break;
                case WLAN_FSTYPE_CFPOLL:
-                       WLAN_LOG_DEBUG(3, "rx'd data:cfpoll\n");
+                       pr_debug("rx'd data:cfpoll\n");
                        wlandev->rx.cfpoll++;
                        break;
                case WLAN_FSTYPE_CFACK_CFPOLL:
-                       WLAN_LOG_DEBUG(3, "rx'd data:cfack_cfpoll\n");
+                       pr_debug("rx'd data:cfack_cfpoll\n");
                        wlandev->rx.cfack_cfpoll++;
                        break;
                default:
@@ -1193,18 +1090,15 @@ static int p80211_rx_typedrop( wlandevice_t *wlandev, u16 fc)
        return drop;
 }
 
-static void p80211knetdev_tx_timeout( netdevice_t *netdev)
+static void p80211knetdev_tx_timeout(netdevice_t *netdev)
 {
-       wlandevice_t    *wlandev = netdev->ml_priv;
-       DBFENTER;
+       wlandevice_t *wlandev = netdev->ml_priv;
 
        if (wlandev->tx_timeout) {
                wlandev->tx_timeout(wlandev);
        } else {
-               WLAN_LOG_WARNING("Implement tx_timeout for %s\n",
-                                wlandev->nsdname);
+               printk(KERN_WARNING "Implement tx_timeout for %s\n",
+                      wlandev->nsdname);
                netif_wake_queue(wlandev->netdev);
        }
-
-       DBFEXIT;
 }