Revert to Fedora kernel-2.6.17-1.2187_FC5 patched with vs2.0.2.1; there are too many...
[linux-2.6.git] / drivers / usb / serial / visor.c
index 933d67c..f5c3841 100644 (file)
  *
  *     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.
+ *     the Free Software Foundation; either version 2 of the License.
  *
  * See Documentation/usb/usb-serial.txt for more information on using this driver
  *
- * (06/03/2003) Judd Montgomery <judd at jpilot.org>
- *     Added support for module parameter options for untested/unknown
- *     devices.
- *
- * (03/09/2003) gkh
- *     Added support for the Sony Clie NZ90V device.  Thanks to Martin Brachtl
- *     <brachtl@redgrep.cz> for the information.
- *
- * (03/05/2003) gkh
- *     Think Treo support is now working.
- *
- * (04/03/2002) gkh
- *     Added support for the Sony OS 4.1 devices.  Thanks to Hiroyuki ARAKI
- *     <hiro@zob.ne.jp> for the information.
- *
- * (03/27/2002) gkh
- *     Removed assumptions that port->tty was always valid (is not true
- *     for usb serial console devices.)
- *
- * (03/23/2002) gkh
- *     Added support for the Palm i705 device, thanks to Thomas Riemer
- *     <tom@netmech.com> for the information.
- *
- * (03/21/2002) gkh
- *     Added support for the Palm m130 device, thanks to Udo Eisenbarth
- *     <udo.eisenbarth@web.de> for the information.
- *
- * (02/27/2002) gkh
- *     Reworked the urb handling logic.  We have no more pool, but dynamically
- *     allocate the urb and the transfer buffer on the fly.  In testing this
- *     does not incure any measurable overhead.  This also relies on the fact
- *     that we have proper reference counting logic for urbs.
- *
- * (02/21/2002) SilaS
- *  Added initial support for the Palm m515 devices.
- *
- * (02/14/2002) gkh
- *     Added support for the Clie S-360 device.
- *
- * (12/18/2001) gkh
- *     Added better Clie support for 3.5 devices.  Thanks to Geoffrey Levand
- *     for the patch.
- *
- * (11/11/2001) gkh
- *     Added support for the m125 devices, and added check to prevent oopses
- *     for CliĆ© devices that lie about the number of ports they have.
- *
- * (08/30/2001) gkh
- *     Added support for the Clie devices, both the 3.5 and 4.0 os versions.
- *     Many thanks to Daniel Burke, and Bryan Payne for helping with this.
- *
- * (08/23/2001) gkh
- *     fixed a few potential bugs pointed out by Oliver Neukum.
- *
- * (05/30/2001) gkh
- *     switched from using spinlock to a semaphore, which fixes lots of problems.
- *
- * (05/28/2000) gkh
- *     Added initial support for the Palm m500 and Palm m505 devices.
- *
- * (04/08/2001) gb
- *     Identify version on module load.
- *
- * (01/21/2000) gkh
- *     Added write_room and chars_in_buffer, as they were previously using the
- *     generic driver versions which is all wrong now that we are using an urb
- *     pool.  Thanks to Wolfgang Grandegger for pointing this out to me.
- *     Removed count assignment in the write function, which was not needed anymore
- *     either.  Thanks to Al Borchers for pointing this out.
- *
- * (12/12/2000) gkh
- *     Moved MOD_DEC to end of visor_close to be nicer, as the final write 
- *     message can sleep.
- * 
- * (11/12/2000) gkh
- *     Fixed bug with data being dropped on the floor by forcing tty->low_latency
- *     to be on.  Hopefully this fixes the OHCI issue!
- *
- * (11/01/2000) Adam J. Richter
- *     usb_device_id table support
- * 
- * (10/05/2000) gkh
- *     Fixed bug with urb->dev not being set properly, now that the usb
- *     core needs it.
- * 
- * (09/11/2000) gkh
- *     Got rid of always calling kmalloc for every urb we wrote out to the
- *     device.
- *     Added visor_read_callback so we can keep track of bytes in and out for
- *     those people who like to know the speed of their device.
- *     Removed DEBUG #ifdefs with call to usb_serial_debug_data
- *
- * (09/06/2000) gkh
- *     Fixed oops in visor_exit.  Need to uncomment usb_unlink_urb call _after_
- *     the host controller drivers set urb->dev = NULL when the urb is finished.
- *
- * (08/28/2000) gkh
- *     Added locks for SMP safeness.
- *
- * (08/08/2000) gkh
- *     Fixed endian problem in visor_startup.
- *     Fixed MOD_INC and MOD_DEC logic and the ability to open a port more 
- *     than once.
- * 
- * (07/23/2000) gkh
- *     Added pool of write urbs to speed up transfers to the visor.
- * 
- * (07/19/2000) gkh
- *     Added module_init and module_exit functions to handle the fact that this
- *     driver is a loadable module now.
- *
- * (07/03/2000) gkh
- *     Added visor_set_ioctl and visor_set_termios functions (they don't do much
- *     of anything, but are good for debugging.)
- * 
- * (06/25/2000) gkh
- *     Fixed bug in visor_unthrottle that should help with the disconnect in PPP
- *     bug that people have been reporting.
- *
- * (06/23/2000) gkh
- *     Cleaned up debugging statements in a quest to find UHCI timeout bug.
- *
- * (04/27/2000) Ryan VanderBijl
- *     Fixed memory leak in visor_close
- *
- * (03/26/2000) gkh
- *     Split driver up into device specific pieces.
- * 
  */
 
 #include <linux/config.h>
 /*
  * Version Information
  */
-#define DRIVER_VERSION "v2.1"
 #define DRIVER_AUTHOR "Greg Kroah-Hartman <greg@kroah.com>"
 #define DRIVER_DESC "USB HandSpring Visor / Palm OS driver"
 
 /* function prototypes for a handspring visor */
 static int  visor_open         (struct usb_serial_port *port, struct file *filp);
 static void visor_close                (struct usb_serial_port *port, struct file *filp);
-static int  visor_write                (struct usb_serial_port *port, int from_user, const unsigned char *buf, int count);
+static int  visor_write                (struct usb_serial_port *port, const unsigned char *buf, int count);
 static int  visor_write_room           (struct usb_serial_port *port);
 static int  visor_chars_in_buffer      (struct usb_serial_port *port);
 static void visor_throttle     (struct usb_serial_port *port);
@@ -199,6 +69,8 @@ static struct usb_device_id id_table [] = {
                .driver_info = (kernel_ulong_t)&palm_os_4_probe },
        { USB_DEVICE(HANDSPRING_VENDOR_ID, HANDSPRING_TREO600_ID),
                .driver_info = (kernel_ulong_t)&palm_os_4_probe },
+       { USB_DEVICE(GSPDA_VENDOR_ID, GSPDA_XPLORE_M68_ID),
+               .driver_info = (kernel_ulong_t)&palm_os_4_probe },
        { USB_DEVICE(PALM_VENDOR_ID, PALM_M500_ID),
                .driver_info = (kernel_ulong_t)&palm_os_4_probe },
        { USB_DEVICE(PALM_VENDOR_ID, PALM_M505_ID),
@@ -215,6 +87,8 @@ static struct usb_device_id id_table [] = {
                .driver_info = (kernel_ulong_t)&palm_os_4_probe },
        { USB_DEVICE(PALM_VENDOR_ID, PALM_TUNGSTEN_T_ID),
                .driver_info = (kernel_ulong_t)&palm_os_4_probe },
+       { USB_DEVICE(PALM_VENDOR_ID, PALM_TREO_650),
+               .driver_info = (kernel_ulong_t)&palm_os_4_probe },
        { USB_DEVICE(PALM_VENDOR_ID, PALM_TUNGSTEN_Z_ID),
                .driver_info = (kernel_ulong_t)&palm_os_4_probe },
        { USB_DEVICE(PALM_VENDOR_ID, PALM_ZIRE31_ID),
@@ -237,12 +111,16 @@ static struct usb_device_id id_table [] = {
                .driver_info = (kernel_ulong_t)&palm_os_4_probe },
        { USB_DEVICE(SAMSUNG_VENDOR_ID, SAMSUNG_SPH_I500_ID), 
                .driver_info = (kernel_ulong_t)&palm_os_4_probe },
+       { USB_DEVICE(TAPWAVE_VENDOR_ID, TAPWAVE_ZODIAC_ID),
+               .driver_info = (kernel_ulong_t)&palm_os_4_probe },
        { USB_DEVICE(GARMIN_VENDOR_ID, GARMIN_IQUE_3600_ID), 
                .driver_info = (kernel_ulong_t)&palm_os_4_probe },
        { USB_DEVICE(ACEECA_VENDOR_ID, ACEECA_MEZ1000_ID),
                .driver_info = (kernel_ulong_t)&palm_os_4_probe },
        { USB_DEVICE(KYOCERA_VENDOR_ID, KYOCERA_7135_ID),
                .driver_info = (kernel_ulong_t)&palm_os_4_probe },
+       { USB_DEVICE(FOSSIL_VENDOR_ID, FOSSIL_ABACUS_ID),
+               .driver_info = (kernel_ulong_t)&palm_os_4_probe },
        { },                                    /* optional parameter entry */
        { }                                     /* Terminating entry */
 };
@@ -263,6 +141,7 @@ static struct usb_device_id id_table_combined [] = {
        { USB_DEVICE(HANDSPRING_VENDOR_ID, HANDSPRING_VISOR_ID) },
        { USB_DEVICE(HANDSPRING_VENDOR_ID, HANDSPRING_TREO_ID) },
        { USB_DEVICE(HANDSPRING_VENDOR_ID, HANDSPRING_TREO600_ID) },
+       { USB_DEVICE(GSPDA_VENDOR_ID, GSPDA_XPLORE_M68_ID) },
        { USB_DEVICE(PALM_VENDOR_ID, PALM_M500_ID) },
        { USB_DEVICE(PALM_VENDOR_ID, PALM_M505_ID) },
        { USB_DEVICE(PALM_VENDOR_ID, PALM_M515_ID) },
@@ -271,6 +150,7 @@ static struct usb_device_id id_table_combined [] = {
        { USB_DEVICE(PALM_VENDOR_ID, PALM_M125_ID) },
        { USB_DEVICE(PALM_VENDOR_ID, PALM_M130_ID) },
        { USB_DEVICE(PALM_VENDOR_ID, PALM_TUNGSTEN_T_ID) },
+       { USB_DEVICE(PALM_VENDOR_ID, PALM_TREO_650) },
        { USB_DEVICE(PALM_VENDOR_ID, PALM_TUNGSTEN_Z_ID) },
        { USB_DEVICE(PALM_VENDOR_ID, PALM_ZIRE31_ID) },
        { USB_DEVICE(PALM_VENDOR_ID, PALM_ZIRE_ID) },
@@ -284,9 +164,11 @@ static struct usb_device_id id_table_combined [] = {
        { USB_DEVICE(SONY_VENDOR_ID, SONY_CLIE_TJ25_ID) },
        { USB_DEVICE(SAMSUNG_VENDOR_ID, SAMSUNG_SCH_I330_ID) },
        { USB_DEVICE(SAMSUNG_VENDOR_ID, SAMSUNG_SPH_I500_ID) },
+       { USB_DEVICE(TAPWAVE_VENDOR_ID, TAPWAVE_ZODIAC_ID) },
        { USB_DEVICE(GARMIN_VENDOR_ID, GARMIN_IQUE_3600_ID) },
        { USB_DEVICE(ACEECA_VENDOR_ID, ACEECA_MEZ1000_ID) },
        { USB_DEVICE(KYOCERA_VENDOR_ID, KYOCERA_7135_ID) },
+       { USB_DEVICE(FOSSIL_VENDOR_ID, FOSSIL_ABACUS_ID) },
        { },                                    /* optional parameter entry */
        { }                                     /* Terminating entry */
 };
@@ -294,18 +176,20 @@ static struct usb_device_id id_table_combined [] = {
 MODULE_DEVICE_TABLE (usb, id_table_combined);
 
 static struct usb_driver visor_driver = {
-       .owner =        THIS_MODULE,
        .name =         "visor",
        .probe =        usb_serial_probe,
        .disconnect =   usb_serial_disconnect,
        .id_table =     id_table_combined,
+       .no_dynamic_id =        1,
 };
 
 /* All of the device info needed for the Handspring Visor, and Palm 4.0 devices */
-static struct usb_serial_device_type handspring_device = {
-       .owner =                THIS_MODULE,
-       .name =                 "Handspring Visor / Palm OS",
-       .short_name =           "visor",
+static struct usb_serial_driver handspring_device = {
+       .driver = {
+               .owner =        THIS_MODULE,
+               .name =         "visor",
+       },
+       .description =          "Handspring Visor / Palm OS",
        .id_table =             id_table,
        .num_interrupt_in =     NUM_DONT_CARE,
        .num_bulk_in =          2,
@@ -330,10 +214,12 @@ static struct usb_serial_device_type handspring_device = {
 };
 
 /* All of the device info needed for the Clie UX50, TH55 Palm 5.0 devices */
-static struct usb_serial_device_type clie_5_device = {
-       .owner =                THIS_MODULE,
-       .name =                 "Sony Clie 5.0",
-       .short_name =           "clie_5",
+static struct usb_serial_driver clie_5_device = {
+       .driver = {
+               .owner =        THIS_MODULE,
+               .name =         "clie_5",
+       },
+       .description =          "Sony Clie 5.0",
        .id_table =             clie_id_5_table,
        .num_interrupt_in =     NUM_DONT_CARE,
        .num_bulk_in =          2,
@@ -358,10 +244,12 @@ static struct usb_serial_device_type clie_5_device = {
 };
 
 /* device info for the Sony Clie OS version 3.5 */
-static struct usb_serial_device_type clie_3_5_device = {
-       .owner =                THIS_MODULE,
-       .name =                 "Sony Clie 3.5",
-       .short_name =           "clie_3.5",
+static struct usb_serial_driver clie_3_5_device = {
+       .driver = {
+               .owner =        THIS_MODULE,
+               .name =         "clie_3.5",
+       },
+       .description =          "Sony Clie 3.5",
        .id_table =             clie_id_3_5_table,
        .num_interrupt_in =     0,
        .num_bulk_in =          1,
@@ -386,6 +274,7 @@ struct visor_private {
        int bytes_in;
        int bytes_out;
        int outstanding_urbs;
+       int throttled;
 };
 
 /* number of outstanding urbs to prevent userspace DoS from happening */
@@ -415,6 +304,7 @@ static int visor_open (struct usb_serial_port *port, struct file *filp)
        priv->bytes_in = 0;
        priv->bytes_out = 0;
        priv->outstanding_urbs = 0;
+       priv->throttled = 0;
        spin_unlock_irqrestore(&priv->lock, flags);
 
        /*
@@ -459,9 +349,9 @@ static void visor_close (struct usb_serial_port *port, struct file * filp)
        dbg("%s - port %d", __FUNCTION__, port->number);
                         
        /* shutdown our urbs */
-       usb_unlink_urb (port->read_urb);
+       usb_kill_urb(port->read_urb);
        if (port->interrupt_in_urb)
-               usb_unlink_urb (port->interrupt_in_urb);
+               usb_kill_urb(port->interrupt_in_urb);
 
        /* Try to send shutdown message, if the device is gone, this will just fail. */
        transfer_buffer =  kmalloc (0x12, GFP_KERNEL);
@@ -480,7 +370,7 @@ static void visor_close (struct usb_serial_port *port, struct file * filp)
 }
 
 
-static int visor_write (struct usb_serial_port *port, int from_user, const unsigned char *buf, int count)
+static int visor_write (struct usb_serial_port *port, const unsigned char *buf, int count)
 {
        struct visor_private *priv = usb_get_serial_port_data(port);
        struct usb_serial *serial = port->serial;
@@ -494,7 +384,7 @@ static int visor_write (struct usb_serial_port *port, int from_user, const unsig
        spin_lock_irqsave(&priv->lock, flags);
        if (priv->outstanding_urbs > URB_UPPER_LIMIT) {
                spin_unlock_irqrestore(&priv->lock, flags);
-               dev_dbg(&port->dev, "write limit hit\n");
+               dbg("%s - write limit hit\n", __FUNCTION__);
                return 0;
        }
        spin_unlock_irqrestore(&priv->lock, flags);
@@ -512,15 +402,7 @@ static int visor_write (struct usb_serial_port *port, int from_user, const unsig
                return -ENOMEM;
        }
 
-       if (from_user) {
-               if (copy_from_user (buffer, buf, count)) {
-                       kfree (buffer);
-                       usb_free_urb (urb);
-                       return -EFAULT;
-               }
-       } else {
-               memcpy (buffer, buf, count);
-       }
+       memcpy (buffer, buf, count);
 
        usb_serial_debug_data(debug, &port->dev, __FUNCTION__, count, buffer);
 
@@ -609,7 +491,7 @@ static void visor_read_bulk_callback (struct urb *urb, struct pt_regs *regs)
        unsigned char *data = urb->transfer_buffer;
        struct tty_struct *tty;
        unsigned long flags;
-       int i;
+       int throttled;
        int result;
 
        dbg("%s - port %d", __FUNCTION__, port->number);
@@ -623,30 +505,27 @@ static void visor_read_bulk_callback (struct urb *urb, struct pt_regs *regs)
 
        tty = port->tty;
        if (tty && urb->actual_length) {
-               for (i = 0; i < urb->actual_length ; ++i) {
-                       /* if we insert more than TTY_FLIPBUF_SIZE characters, we drop them. */
-                       if(tty->flip.count >= TTY_FLIPBUF_SIZE) {
-                               tty_flip_buffer_push(tty);
-                       }
-                       /* this doesn't actually push the data through unless tty->low_latency is set */
-                       tty_insert_flip_char(tty, data[i], 0);
-               }
+               tty_buffer_request_room(tty, urb->actual_length);
+               tty_insert_flip_string(tty, data, urb->actual_length);
                tty_flip_buffer_push(tty);
        }
        spin_lock_irqsave(&priv->lock, flags);
        priv->bytes_in += urb->actual_length;
+       throttled = priv->throttled;
        spin_unlock_irqrestore(&priv->lock, flags);
 
-       /* Continue trying to always read  */
-       usb_fill_bulk_urb (port->read_urb, port->serial->dev,
-                          usb_rcvbulkpipe(port->serial->dev,
-                                          port->bulk_in_endpointAddress),
-                          port->read_urb->transfer_buffer,
-                          port->read_urb->transfer_buffer_length,
-                          visor_read_bulk_callback, port);
-       result = usb_submit_urb(port->read_urb, GFP_ATOMIC);
-       if (result)
-               dev_err(&port->dev, "%s - failed resubmitting read urb, error %d\n", __FUNCTION__, result);
+       /* Continue trying to always read if we should */
+       if (!throttled) {
+               usb_fill_bulk_urb (port->read_urb, port->serial->dev,
+                                  usb_rcvbulkpipe(port->serial->dev,
+                                                  port->bulk_in_endpointAddress),
+                                  port->read_urb->transfer_buffer,
+                                  port->read_urb->transfer_buffer_length,
+                                  visor_read_bulk_callback, port);
+               result = usb_submit_urb(port->read_urb, GFP_ATOMIC);
+               if (result)
+                       dev_err(&port->dev, "%s - failed resubmitting read urb, error %d\n", __FUNCTION__, result);
+       }
        return;
 }
 
@@ -691,16 +570,26 @@ exit:
 
 static void visor_throttle (struct usb_serial_port *port)
 {
+       struct visor_private *priv = usb_get_serial_port_data(port);
+       unsigned long flags;
+
        dbg("%s - port %d", __FUNCTION__, port->number);
-       usb_unlink_urb (port->read_urb);
+       spin_lock_irqsave(&priv->lock, flags);
+       priv->throttled = 1;
+       spin_unlock_irqrestore(&priv->lock, flags);
 }
 
 
 static void visor_unthrottle (struct usb_serial_port *port)
 {
+       struct visor_private *priv = usb_get_serial_port_data(port);
+       unsigned long flags;
        int result;
 
        dbg("%s - port %d", __FUNCTION__, port->number);
+       spin_lock_irqsave(&priv->lock, flags);
+       priv->throttled = 0;
+       spin_unlock_irqrestore(&priv->lock, flags);
 
        port->read_urb->dev = port->serial->dev;
        result = usb_submit_urb(port->read_urb, GFP_ATOMIC);
@@ -742,9 +631,7 @@ static int palm_os_3_probe (struct usb_serial *serial, const struct usb_device_i
        if (retval == sizeof(*connection_info)) {
                connection_info = (struct visor_connection_info *)transfer_buffer;
 
-               le16_to_cpus(&connection_info->num_ports);
-               num_ports = connection_info->num_ports;
-
+               num_ports = le16_to_cpu(connection_info->num_ports);
                for (i = 0; i < num_ports; ++i) {
                        switch (connection_info->connections[i].port_function_id) {
                                case VISOR_FUNCTION_GENERIC:
@@ -767,7 +654,7 @@ static int palm_os_3_probe (struct usb_serial *serial, const struct usb_device_i
                                        break;
                        }
                        dev_info(dev, "%s: port %d, is for %s use\n",
-                               serial->type->name,
+                               serial->type->description,
                                connection_info->connections[i].port, string);
                }
        }
@@ -776,11 +663,11 @@ static int palm_os_3_probe (struct usb_serial *serial, const struct usb_device_i
        */
        if (num_ports == 0 || num_ports > 2) {
                dev_warn (dev, "%s: No valid connect info available\n",
-                       serial->type->name);
+                       serial->type->description);
                num_ports = 2;
        }
   
-       dev_info(dev, "%s: Number of ports: %d\n", serial->type->name,
+       dev_info(dev, "%s: Number of ports: %d\n", serial->type->description,
                num_ports);
 
        /*
@@ -876,10 +763,9 @@ static int generic_startup(struct usb_serial *serial)
        int i;
 
        for (i = 0; i < serial->num_ports; ++i) {
-               priv = kmalloc (sizeof(*priv), GFP_KERNEL);
+               priv = kzalloc (sizeof(*priv), GFP_KERNEL);
                if (!priv)
                        return -ENOMEM;
-               memset (priv, 0x00, sizeof(*priv));
                spin_lock_init(&priv->lock);
                usb_set_serial_port_data(serial->port[i], priv);
        }
@@ -901,7 +787,7 @@ static int clie_3_5_startup (struct usb_serial *serial)
        /* get the config number */
        result = usb_control_msg (serial->dev, usb_rcvctrlpipe(serial->dev, 0),
                                  USB_REQ_GET_CONFIGURATION, USB_DIR_IN,
-                                 0, 0, &data, 1, HZ * 3);
+                                 0, 0, &data, 1, 3000);
        if (result < 0) {
                dev_err(dev, "%s: get config number failed: %d\n", __FUNCTION__, result);
                return result;
@@ -915,7 +801,7 @@ static int clie_3_5_startup (struct usb_serial *serial)
        result = usb_control_msg (serial->dev, usb_rcvctrlpipe(serial->dev, 0),
                                  USB_REQ_GET_INTERFACE, 
                                  USB_DIR_IN | USB_RECIP_INTERFACE,
-                                 0, 0, &data, 1, HZ * 3);
+                                 0, 0, &data, 1, 3000);
        if (result < 0) {
                dev_err(dev, "%s: get interface number failed: %d\n", __FUNCTION__, result);
                return result;
@@ -934,8 +820,8 @@ static int treo_attach (struct usb_serial *serial)
 
        /* Only do this endpoint hack for the Handspring devices with
         * interrupt in endpoints, which for now are the Treo devices. */
-       if (!((serial->dev->descriptor.idVendor == HANDSPRING_VENDOR_ID) ||
-             (serial->dev->descriptor.idVendor == KYOCERA_VENDOR_ID)) ||
+       if (!((le16_to_cpu(serial->dev->descriptor.idVendor) == HANDSPRING_VENDOR_ID) ||
+             (le16_to_cpu(serial->dev->descriptor.idVendor) == KYOCERA_VENDOR_ID)) ||
            (serial->num_interrupt_in == 0))
                goto generic_startup;
 
@@ -1110,7 +996,7 @@ static int __init visor_init (void)
        retval = usb_register(&visor_driver);
        if (retval) 
                goto failed_usb_register;
-       info(DRIVER_DESC " " DRIVER_VERSION);
+       info(DRIVER_DESC);
 
        return 0;
 failed_usb_register: