]> rtime.felk.cvut.cz Git - socketcan-devel.git/blobdiff - kernel/2.6/drivers/net/can/slcan.c
slcan: Fix whitespace issues for mainline.
[socketcan-devel.git] / kernel / 2.6 / drivers / net / can / slcan.c
index 75f3afc1d737097bbb8634bb743cd354af714bfb..cf7e59360b65ceaf5a472e35295c4d4bcef44a0c 100644 (file)
@@ -1,28 +1,26 @@
 /*
  * slcan.c - serial line CAN interface driver (using tty line discipline)
  *
- * Copyright (c) 2007 Volkswagen Group Electronic Research
- * All rights reserved.
+ * This file is derived from linux/drivers/net/slip.c
+ *
+ * slip.c Authors  : Laurence Culhane <loz@holmes.demon.co.uk>
+ *                   Fred N. van Kempen <waltje@uwalt.nl.mugnet.org>
+ * slcan.c Author  : Oliver Hartkopp <socketcan@hartkopp.net>
  *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- * 3. Neither the name of Volkswagen nor the names of its contributors
- *    may be used to endorse or promote products derived from this software
- *    without specific prior written permission.
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License as published by the
+ * Free Software Foundation; either version 2 of the License, or (at your
+ * option) any later version.
  *
- * Alternatively, provided that this notice is retained in full, this
- * software may be distributed under the terms of the GNU General
- * Public License ("GPL") version 2, in which case the provisions of the
- * GPL apply INSTEAD OF those given above.
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * General Public License for more details.
  *
- * The provided data structures and external interfaces from this code
- * are not restricted to be used by modules with a GPL compatible license.
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 59 Temple Place, Suite 330, Boston, MA 02111-1307. You can also get it
+ * at http://www.gnu.org/licenses/gpl.html
  *
  * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
  * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
  *
  */
 
-/*
- * This file is derived from linux/drivers/net/slip.c
- *
- * Therefore it has the same (strange?) behaviour not to unregister the
- * netdevice when detaching the tty. Is there any better solution?
- *
- * Do not try to attach, detach and re-attach a tty for this reason ...
- *
- * slip.c Authors: Laurence Culhane, <loz@holmes.demon.co.uk>
- *                 Fred N. van Kempen, <waltje@uwalt.nl.mugnet.org>
- */
-
 #include <linux/version.h>
 #include <linux/module.h>
 #include <linux/moduleparam.h>
 
 #include <asm/system.h>
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,17)
+#include <linux/uaccess.h>
+#else
 #include <asm/uaccess.h>
+#endif
 #include <linux/bitops.h>
 #include <linux/string.h>
-#include <linux/mm.h>
-#include <linux/interrupt.h>
-#include <linux/in.h>
 #include <linux/tty.h>
 #include <linux/errno.h>
 #include <linux/netdevice.h>
-#include <linux/etherdevice.h>
 #include <linux/skbuff.h>
 #include <linux/rtnetlink.h>
 #include <linux/if_arp.h>
 #include <linux/if_ether.h>
-#include <linux/if_slip.h>
 #include <linux/delay.h>
 #include <linux/init.h>
+#include <socketcan/can.h>
 
-#include <linux/can.h>
-
-#include <linux/can/version.h> /* for RCSID. Removed by mkpatch script */
+#include <socketcan/can/version.h> /* for RCSID. Removed by mkpatch script */
 RCSID("$Id$");
 
 static __initdata const char banner[] =
@@ -86,19 +70,18 @@ static __initdata const char banner[] =
 
 MODULE_ALIAS_LDISC(N_SLCAN);
 MODULE_DESCRIPTION("serial line CAN interface");
-MODULE_LICENSE("Dual BSD/GPL");
-MODULE_AUTHOR("Oliver Hartkopp <oliver.hartkopp@volkswagen.de>");
-
-#ifdef CONFIG_CAN_DEBUG_DEVICES
-static int debug;
-module_param(debug, int, S_IRUGO);
-#define DBG(args...)       (debug & 1 ? \
-                              (printk(KERN_DEBUG "slcan %s: ", __func__), \
-                               printk(args)) : 0)
-#else
-#define DBG(args...)
+MODULE_LICENSE("GPL");
+MODULE_AUTHOR("Oliver Hartkopp <socketcan@hartkopp.net>");
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,14)
+static inline void *kzalloc(size_t size, unsigned int __nocast flags)
+{
+       void *ret = kmalloc(size, flags);
+       if (ret)
+               memset(ret, 0, size);
+       return ret;
+}
 #endif
-
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,25)
 #ifndef N_SLCAN
 #error Your kernel does not support tty line discipline N_SLCAN
 #endif
@@ -113,9 +96,20 @@ module_param(debug, int, S_IRUGO);
  *
  * NEW: Since Kernel 2.6.21 you only have to change include/linux/tty.h
  *
+ * HACK for precompiled Kernels:
+ *
+ * In order to use the slcan driver without rebuilding the kernel, the slcan
+ * driver must be compiled to use an existing line discipline.
+ * The N_MOUSE line discipline is documented to be free for custom use and
+ * using it *should* not cause any side effect.
+ *
+ * Then, before compiling the slcan driver, add a -DN_SLCAN=N_MOUSE  
+ * compilation option in its Makefile. The slcan_attach tool must(!!) also be
+ * rebuild to use the right value for N_SLCAN. This workaround will allow  
+ * to use the slcan driver with an existing kernel.
  */
+#endif
 
-#define SLC_CHECK_TRANSMIT
 #define SLCAN_MAGIC 0x53CA
 
 static int maxdev = 10;                /* MAX number of SLCAN channels;
@@ -158,8 +152,18 @@ struct slcan {
 
 static struct net_device **slcan_devs;
 
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
+/* Netdevice get statistics request */
+static struct net_device_stats *slc_get_stats(struct net_device *dev)
+{
+       struct slcan *sl = netdev_priv(dev);
+
+       return &sl->stats;
+}
+#endif
+
  /************************************************************************
-  *                    SLCAN ENCAPSULATION FORMAT                       *
+  *                    SLCAN ENCAPSULATION FORMAT                       *
   ************************************************************************/
 
 /*
@@ -194,7 +198,7 @@ static struct net_device **slcan_devs;
  */
 
  /************************************************************************
-  *                    STANDARD SLCAN DECAPSULATION                     *
+  *                    STANDARD SLCAN DECAPSULATION                     *
   ************************************************************************/
 
 static int asc2nibble(char c)
@@ -215,10 +219,13 @@ static int asc2nibble(char c)
 /* Send one completely decapsulated can_frame to the network layer */
 static void slc_bump(struct slcan *sl)
 {
-       struct net_device_stats *stats = sl->dev->get_stats(sl->dev);
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
+       struct net_device_stats *stats = slc_get_stats(sl->dev);
+#endif
        struct sk_buff *skb;
        struct can_frame cf;
        int i, dlc_pos, tmp;
+       unsigned long ultmp;
        char cmd = sl->rbuff[0];
 
        if ((cmd != 't') && (cmd != 'T') && (cmd != 'r') && (cmd != 'R'))
@@ -232,10 +239,18 @@ static void slc_bump(struct slcan *sl)
        if (!((sl->rbuff[dlc_pos] >= '0') && (sl->rbuff[dlc_pos] < '9')))
                return;
 
-       cf.can_dlc = sl->rbuff[dlc_pos] & 0x0F; /* get can_dlc */
+       cf.can_dlc = sl->rbuff[dlc_pos] - '0'; /* get can_dlc from ASCII val */
 
        sl->rbuff[dlc_pos] = 0; /* terminate can_id string */
-       cf.can_id = simple_strtoul(sl->rbuff+1, NULL, 16);
+
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,25)
+       ultmp = simple_strtoul(sl->rbuff+1, NULL, 16);
+#else
+       if (strict_strtoul(sl->rbuff+1, 16, &ultmp))
+               return;
+#endif
+
+       cf.can_id = ultmp;
 
        if (!(cmd & 0x20)) /* NO tiny chars => extended frame format */
                cf.can_id |= CAN_EFF_FLAG;
@@ -270,15 +285,24 @@ static void slc_bump(struct slcan *sl)
               &cf, sizeof(struct can_frame));
        netif_rx(skb);
 
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,32)
        sl->dev->last_rx = jiffies;
+#endif
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
        stats->rx_packets++;
        stats->rx_bytes += cf.can_dlc;
+#else
+       sl->dev->stats.rx_packets++;
+       sl->dev->stats.rx_bytes += cf.can_dlc;
+#endif
 }
 
 /* parse tty input stream */
 static void slcan_unesc(struct slcan *sl, unsigned char s)
 {
-       struct net_device_stats *stats = sl->dev->get_stats(sl->dev);
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
+       struct net_device_stats *stats = slc_get_stats(sl->dev);
+#endif
 
        if ((s == '\r') || (s == '\a')) { /* CR or BEL ends the pdu */
                if (!test_and_clear_bit(SLF_ERROR, &sl->flags) &&
@@ -292,7 +316,11 @@ static void slcan_unesc(struct slcan *sl, unsigned char s)
                                sl->rbuff[sl->rcount++] = s;
                                return;
                        } else {
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
                                stats->rx_over_errors++;
+#else
+                               sl->dev->stats.rx_over_errors++;
+#endif
                                set_bit(SLF_ERROR, &sl->flags);
                        }
                }
@@ -300,13 +328,15 @@ static void slcan_unesc(struct slcan *sl, unsigned char s)
 }
 
  /************************************************************************
-  *                    STANDARD SLCAN ENCAPSULATION                     *
+  *                    STANDARD SLCAN ENCAPSULATION                     *
   ************************************************************************/
 
 /* Encapsulate one can_frame and stuff into a TTY queue. */
 static void slc_encaps(struct slcan *sl, struct can_frame *cf)
 {
-       struct net_device_stats *stats = sl->dev->get_stats(sl->dev);
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
+       struct net_device_stats *stats = slc_get_stats(sl->dev);
+#endif
        int actual, idx, i;
        char cmd;
 
@@ -327,26 +357,29 @@ static void slc_encaps(struct slcan *sl, struct can_frame *cf)
        for (i = 0; i < cf->can_dlc; i++)
                sprintf(&sl->xbuff[idx + 2*i], "%02X", cf->data[i]);
 
-       DBG("ASCII frame = '%s'\n", sl->xbuff);
-
        strcat(sl->xbuff, "\r"); /* add terminating character */
 
        /* Order of next two lines is *very* important.
         * When we are sending a little amount of data,
-        * the transfer may be completed inside driver.write()
+        * the transfer may be completed inside the ops->write()
         * routine, because it's running with interrupts enabled.
         * In this case we *never* got WRITE_WAKEUP event,
         * if we did not request it before write operation.
         *       14 Oct 1994  Dmitry Gorodchanin.
         */
-       sl->tty->flags |= (1 << TTY_DO_WRITE_WAKEUP);
+       set_bit(TTY_DO_WRITE_WAKEUP, &sl->tty->flags);
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,26)
        actual = sl->tty->driver->write(sl->tty, sl->xbuff, strlen(sl->xbuff));
-#ifdef SLC_CHECK_TRANSMIT
-       sl->dev->trans_start = jiffies;
+#else
+       actual = sl->tty->ops->write(sl->tty, sl->xbuff, strlen(sl->xbuff));
 #endif
        sl->xleft = strlen(sl->xbuff) - actual;
        sl->xhead = sl->xbuff + actual;
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
        stats->tx_bytes += cf->can_dlc;
+#else
+       sl->dev->stats.tx_bytes += cf->can_dlc;
+#endif
 }
 
 /*
@@ -357,7 +390,9 @@ static void slcan_write_wakeup(struct tty_struct *tty)
 {
        int actual;
        struct slcan *sl = (struct slcan *) tty->disc_data;
-       struct net_device_stats *stats = sl->dev->get_stats(sl->dev);
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
+       struct net_device_stats *stats = slc_get_stats(sl->dev);
+#endif
 
        /* First make sure we're connected. */
        if (!sl || sl->magic != SLCAN_MAGIC || !netif_running(sl->dev))
@@ -366,54 +401,31 @@ static void slcan_write_wakeup(struct tty_struct *tty)
        if (sl->xleft <= 0)  {
                /* Now serial buffer is almost free & we can start
                 * transmission of another packet */
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
                stats->tx_packets++;
-               tty->flags &= ~(1 << TTY_DO_WRITE_WAKEUP);
+#else
+               sl->dev->stats.tx_packets++;
+#endif
+               clear_bit(TTY_DO_WRITE_WAKEUP, &tty->flags);
                netif_wake_queue(sl->dev);
                return;
        }
 
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,26)
        actual = tty->driver->write(tty, sl->xhead, sl->xleft);
+#else
+       actual = tty->ops->write(tty, sl->xhead, sl->xleft);
+#endif
        sl->xleft -= actual;
        sl->xhead += actual;
 }
 
-static void slc_tx_timeout(struct net_device *dev)
-{
-       struct slcan *sl = netdev_priv(dev);
-
-       spin_lock(&sl->lock);
-
-       if (netif_queue_stopped(dev)) {
-               if (!netif_running(dev))
-                       goto out;
-
-               /* May be we must check transmitter timeout here ?
-                *      14 Oct 1994 Dmitry Gorodchanin.
-                */
-#ifdef SLC_CHECK_TRANSMIT
-               if (time_before(jiffies, dev->trans_start + 20 * HZ))  {
-                       /* 20 sec timeout not reached */
-                       goto out;
-               }
-               printk(KERN_WARNING "%s: transmit timed out, %s?\n", dev->name,
-                      (sl->tty->driver->chars_in_buffer(sl->tty) || sl->xleft)
-                      ? "bad line quality" : "driver error");
-               sl->xleft = 0;
-               sl->tty->flags &= ~(1 << TTY_DO_WRITE_WAKEUP);
-               netif_wake_queue(sl->dev);
-#endif
-       }
-out:
-       spin_unlock(&sl->lock);
-}
-
-
-/******************************************
- *   Routines looking at netdevice side.
- ******************************************/
-
 /* Send a can_frame to a TTY queue. */
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,32)
 static int slc_xmit(struct sk_buff *skb, struct net_device *dev)
+#else
+static netdev_tx_t slc_xmit(struct sk_buff *skb, struct net_device *dev)
+#endif
 {
        struct slcan *sl = netdev_priv(dev);
 
@@ -426,7 +438,6 @@ static int slc_xmit(struct sk_buff *skb, struct net_device *dev)
                printk(KERN_WARNING "%s: xmit: iface is down\n", dev->name);
                goto out;
        }
-
        if (sl->tty == NULL) {
                spin_unlock(&sl->lock);
                goto out;
@@ -438,10 +449,18 @@ static int slc_xmit(struct sk_buff *skb, struct net_device *dev)
 
 out:
        kfree_skb(skb);
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,32)
        return 0;
+#else
+       return NETDEV_TX_OK;
+#endif
 }
 
 
+/******************************************
+ *   Routines looking at netdevice side.
+ ******************************************/
+
 /* Netdevice UP -> DOWN routine */
 static int slc_close(struct net_device *dev)
 {
@@ -450,7 +469,7 @@ static int slc_close(struct net_device *dev)
        spin_lock_bh(&sl->lock);
        if (sl->tty) {
                /* TTY discipline is running. */
-               sl->tty->flags &= ~(1 << TTY_DO_WRITE_WAKEUP);
+               clear_bit(TTY_DO_WRITE_WAKEUP, &sl->tty->flags);
        }
        netif_stop_queue(dev);
        sl->rcount   = 0;
@@ -473,26 +492,41 @@ static int slc_open(struct net_device *dev)
        return 0;
 }
 
-#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
-/* Netdevice get statistics request */
-static struct net_device_stats *slc_get_stats(struct net_device *dev)
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,31)
+/* Hook the destructor so we can free slcan devs at the right point in time */
+static void slc_free_netdev(struct net_device *dev)
 {
-       struct slcan *sl = netdev_priv(dev);
-
-       return (&sl->stats);
+       int i = dev->base_addr;
+       free_netdev(dev);
+       slcan_devs[i] = NULL;
 }
 #endif
 
-/* Netdevice register callback */
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,28)
+static const struct net_device_ops slc_netdev_ops = {
+       .ndo_open               = slc_open,
+       .ndo_stop               = slc_close,
+       .ndo_start_xmit         = slc_xmit,
+};
+#endif
+
 static void slc_setup(struct net_device *dev)
 {
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,28)
+       dev->netdev_ops         = &slc_netdev_ops;
+#else
        dev->open               = slc_open;
-       dev->destructor         = free_netdev;
        dev->stop               = slc_close;
+       dev->hard_start_xmit    = slc_xmit;
+#endif
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,31)
+       dev->destructor         = slc_free_netdev;
+#else
+       dev->destructor         = free_netdev;
+#endif
 #if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
-       dev->get_stats          = slc_get_stats;
+       dev->get_stats          = slc_get_stats;
 #endif
-       dev->hard_start_xmit    = slc_xmit;
 
        dev->hard_header_len    = 0;
        dev->addr_len           = 0;
@@ -500,10 +534,6 @@ static void slc_setup(struct net_device *dev)
 
        dev->mtu                = sizeof(struct can_frame);
        dev->type               = ARPHRD_CAN;
-#ifdef SLC_CHECK_TRANSMIT
-       dev->tx_timeout         = slc_tx_timeout;
-       dev->watchdog_timeo     = 20*HZ;
-#endif
 
        /* New-style flags. */
        dev->flags              = IFF_NOARP;
@@ -511,7 +541,7 @@ static void slc_setup(struct net_device *dev)
 }
 
 /******************************************
* Routines looking at TTY side.
+  Routines looking at TTY side.
  ******************************************/
 
 #if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,16)
@@ -534,17 +564,22 @@ static void slcan_receive_buf(struct tty_struct *tty,
                              const unsigned char *cp, char *fp, int count)
 {
        struct slcan *sl = (struct slcan *) tty->disc_data;
-       struct net_device_stats *stats = sl->dev->get_stats(sl->dev);
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
+       struct net_device_stats *stats = slc_get_stats(sl->dev);
+#endif
 
-       if (!sl || sl->magic != SLCAN_MAGIC ||
-           !netif_running(sl->dev))
+       if (!sl || sl->magic != SLCAN_MAGIC || !netif_running(sl->dev))
                return;
 
        /* Read the characters out of the buffer */
        while (count--) {
                if (fp && *fp++) {
                        if (!test_and_set_bit(SLF_ERROR, &sl->flags))
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
                                stats->rx_errors++;
+#else
+                               sl->dev->stats.rx_errors++;
+#endif
                        cp++;
                        continue;
                }
@@ -557,7 +592,6 @@ static void slcan_receive_buf(struct tty_struct *tty,
  ************************************/
 
 /* Collect hanged up channels */
-
 static void slc_sync(void)
 {
        int i;
@@ -572,18 +606,19 @@ static void slc_sync(void)
                sl = netdev_priv(dev);
                if (sl->tty || sl->leased)
                        continue;
-               if (dev->flags&IFF_UP)
+               if (dev->flags & IFF_UP)
                        dev_close(dev);
        }
 }
 
-
 /* Find a free SLCAN channel, and link in this `tty' line. */
 static struct slcan *slc_alloc(dev_t line)
 {
        int i;
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,32)
        int sel = -1;
        int score = -1;
+#endif
        struct net_device *dev = NULL;
        struct slcan       *sl;
 
@@ -595,6 +630,7 @@ static struct slcan *slc_alloc(dev_t line)
                if (dev == NULL)
                        break;
 
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,32)
                sl = netdev_priv(dev);
                if (sl->leased) {
                        if (sl->line != line)
@@ -641,6 +677,7 @@ static struct slcan *slc_alloc(dev_t line)
                        sl->flags &= (1 << SLF_INUSE);
                        return sl;
                }
+#endif
        }
 
        /* Sorry, too many, all slots in use */
@@ -651,7 +688,6 @@ static struct slcan *slc_alloc(dev_t line)
                sl = netdev_priv(dev);
                if (test_bit(SLF_INUSE, &sl->flags)) {
                        unregister_netdevice(dev);
-                       free_netdev(dev); /* new in slcan.c */
                        dev = NULL;
                        slcan_devs[i] = NULL;
                }
@@ -659,7 +695,7 @@ static struct slcan *slc_alloc(dev_t line)
 
        if (!dev) {
                char name[IFNAMSIZ];
-               sprintf(name, "slc%d", i);
+               sprintf(name, "slcan%d", i);
 
                dev = alloc_netdev(sizeof(*sl), name, slc_setup);
                if (!dev)
@@ -670,8 +706,8 @@ static struct slcan *slc_alloc(dev_t line)
        sl = netdev_priv(dev);
 
        /* Initialize channel control data */
-       sl->magic       = SLCAN_MAGIC;
-       sl->dev         = dev;
+       sl->magic = SLCAN_MAGIC;
+       sl->dev = dev;
        spin_lock_init(&sl->lock);
        slcan_devs[i] = dev;
 
@@ -696,6 +732,11 @@ static int slcan_open(struct tty_struct *tty)
        if (!capable(CAP_NET_ADMIN))
                return -EPERM;
 
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,25)
+       if (tty->ops->write == NULL)
+               return -EOPNOTSUPP;
+#endif
+
        /* RTnetlink lock is misused here to serialize concurrent
           opens of slcan channels. There are better ways, but it is
           the simplest one.
@@ -705,7 +746,7 @@ static int slcan_open(struct tty_struct *tty)
        /* Collect hanged up channels. */
        slc_sync();
 
-       sl = (struct slcan *) tty->disc_data;
+       sl = tty->disc_data;
 
        err = -EEXIST;
        /* First make sure we're not already connected. */
@@ -743,11 +784,9 @@ static int slcan_open(struct tty_struct *tty)
 
        /* Done.  We have linked the TTY line to a channel. */
        rtnl_unlock();
-
 #if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,16)
        tty->receive_room = 65536;      /* We don't flow control */
 #endif
-
        return sl->dev->base_addr;
 
 err_free_chan:
@@ -762,31 +801,14 @@ err_exit:
        return err;
 }
 
-/*
-
-  FIXME: 1,2 are fixed 3 was never true anyway.
-
-   Let me to blame a bit.
-   1. TTY module calls this funstion on soft interrupt.
-   2. TTY module calls this function WITH MASKED INTERRUPTS!
-   3. TTY module does not notify us about line discipline
-      shutdown,
-
-   Seems, now it is clean. The solution is to consider netdevice and
-   line discipline sides as two independent threads.
-
-   By-product (not desired): slc? does not feel hangups and remains open.
-   It is supposed, that user level program (dip, diald, slattach...)
-   will catch SIGHUP and make the rest of work.
-
-   I see no way to make more with current tty code. --ANK
- */
-
 /*
  * Close down a SLCAN channel.
  * This means flushing out any pending queues, and then returning. This
  * call is serialized against other ldisc functions.
+ *
+ * We also use this method for a hangup event.
  */
+
 static void slcan_close(struct tty_struct *tty)
 {
        struct slcan *sl = (struct slcan *) tty->disc_data;
@@ -800,9 +822,23 @@ static void slcan_close(struct tty_struct *tty)
        if (!sl->leased)
                sl->line = 0;
 
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,31)
+       /* Flush network side */
+       unregister_netdev(sl->dev);
+       /* This will complete via sl_free_netdev */
+#else
        /* Count references from TTY module */
+#endif
 }
 
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,31)
+static int slcan_hangup(struct tty_struct *tty)
+{
+       slcan_close(tty);
+       return 0;
+}
+#endif
+
 /* Perform I/O control on an active SLCAN channel. */
 static int slcan_ioctl(struct tty_struct *tty, struct file *file,
                       unsigned int cmd, unsigned long arg)
@@ -824,7 +860,7 @@ static int slcan_ioctl(struct tty_struct *tty, struct file *file,
        case SIOCSIFHWADDR:
                return -EINVAL;
 
-
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,27)
        /* Allow stty to read, but not set, the serial port */
        case TCGETS:
        case TCGETA:
@@ -832,27 +868,34 @@ static int slcan_ioctl(struct tty_struct *tty, struct file *file,
 
        default:
                return -ENOIOCTLCMD;
+#else
+       default:
+               return tty_mode_ioctl(tty, file, cmd, arg);
+#endif
        }
 }
 
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,27)
 static struct tty_ldisc        slc_ldisc = {
-       .owner          = THIS_MODULE,
-       .magic          = TTY_LDISC_MAGIC,
-       .name           = "slcan",
-       .open           = slcan_open,
-       .close          = slcan_close,
+#else
+static struct tty_ldisc_ops slc_ldisc = {
+#endif
+       .owner          = THIS_MODULE,
+       .magic          = TTY_LDISC_MAGIC,
+       .name           = "slcan",
+       .open           = slcan_open,
+       .close          = slcan_close,
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,31)
+       .hangup         = slcan_hangup,
+#endif
        .ioctl          = slcan_ioctl,
        .receive_buf    = slcan_receive_buf,
 #if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,16)
-       .receive_room   = slcan_receive_room,
+       .receive_room   = slcan_receive_room,
 #endif
        .write_wakeup   = slcan_write_wakeup,
 };
 
-/************************************
- * general slcan module init/exit
- ************************************/
-
 static int __init slcan_init(void)
 {
        int status;
@@ -863,18 +906,15 @@ static int __init slcan_init(void)
        printk(banner);
        printk(KERN_INFO "slcan: %d dynamic interface channels.\n", maxdev);
 
-       slcan_devs = kmalloc(sizeof(struct net_device *)*maxdev, GFP_KERNEL);
+       slcan_devs = kzalloc(sizeof(struct net_device *)*maxdev, GFP_KERNEL);
        if (!slcan_devs) {
                printk(KERN_ERR "slcan: can't allocate slcan device array!\n");
                return -ENOMEM;
        }
 
-       /* Clear the pointer array, we allocate devices when we need them */
-       memset(slcan_devs, 0, sizeof(struct net_device *)*maxdev);
-
        /* Fill in our line protocol discipline, and register it */
        status = tty_register_ldisc(N_SLCAN, &slc_ldisc);
-       if (status != 0)  {
+       if (status)  {
                printk(KERN_ERR "slcan: can't register line discipline\n");
                kfree(slcan_devs);
        }
@@ -913,6 +953,8 @@ static void __exit slcan_exit(void)
                }
        } while (busy && time_before(jiffies, timeout));
 
+       /* FIXME: hangup is async so we should wait when doing this second
+          phase */
 
        for (i = 0; i < maxdev; i++) {
                dev = slcan_devs[i];