USB: symbolserial: use port interrupt-in urb
authorJohan Hovold <jhovold@gmail.com>
Tue, 16 Apr 2013 16:01:28 +0000 (18:01 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 17 Apr 2013 17:05:35 +0000 (10:05 -0700)
Use the port interrupt-in urb rather managing a private one.

Signed-off-by: Johan Hovold <jhovold@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/serial/symbolserial.c

index be05e6caf9a3c501f617c92cee83a6ba84469a28..32ebddf7f010ea6a6c0d18b6afb3f53b0a6ae16a 100644 (file)
@@ -29,13 +29,6 @@ MODULE_DEVICE_TABLE(usb, id_table);
 /* This structure holds all of the individual device information */
 struct symbol_private {
        struct usb_device *udev;
-       struct usb_serial *serial;
-       struct usb_serial_port *port;
-       unsigned char *int_buffer;
-       struct urb *int_urb;
-       int buffer_size;
-       u8 bInterval;
-       u8 int_address;
        spinlock_t lock;        /* protects the following flags */
        bool throttled;
        bool actually_throttled;
@@ -44,9 +37,9 @@ struct symbol_private {
 
 static void symbol_int_callback(struct urb *urb)
 {
-       struct symbol_private *priv = urb->context;
+       struct usb_serial_port *port = urb->context;
+       struct symbol_private *priv = usb_get_serial_data(port->serial);
        unsigned char *data = urb->transfer_buffer;
-       struct usb_serial_port *port = priv->port;
        int status = urb->status;
        int result;
        int data_length;
@@ -94,12 +87,7 @@ exit:
 
        /* Continue trying to always read if we should */
        if (!priv->throttled) {
-               usb_fill_int_urb(priv->int_urb, priv->udev,
-                                usb_rcvintpipe(priv->udev,
-                                               priv->int_address),
-                                priv->int_buffer, priv->buffer_size,
-                                symbol_int_callback, priv, priv->bInterval);
-               result = usb_submit_urb(priv->int_urb, GFP_ATOMIC);
+               result = usb_submit_urb(port->interrupt_in_urb, GFP_ATOMIC);
                if (result)
                        dev_err(&port->dev,
                            "%s - failed resubmitting read urb, error %d\n",
@@ -118,15 +106,10 @@ static int symbol_open(struct tty_struct *tty, struct usb_serial_port *port)
        spin_lock_irqsave(&priv->lock, flags);
        priv->throttled = false;
        priv->actually_throttled = false;
-       priv->port = port;
        spin_unlock_irqrestore(&priv->lock, flags);
 
        /* Start reading from the device */
-       usb_fill_int_urb(priv->int_urb, priv->udev,
-                        usb_rcvintpipe(priv->udev, priv->int_address),
-                        priv->int_buffer, priv->buffer_size,
-                        symbol_int_callback, priv, priv->bInterval);
-       result = usb_submit_urb(priv->int_urb, GFP_KERNEL);
+       result = usb_submit_urb(port->interrupt_in_urb, GFP_KERNEL);
        if (result)
                dev_err(&port->dev,
                        "%s - failed resubmitting read urb, error %d\n",
@@ -136,10 +119,7 @@ static int symbol_open(struct tty_struct *tty, struct usb_serial_port *port)
 
 static void symbol_close(struct usb_serial_port *port)
 {
-       struct symbol_private *priv = usb_get_serial_data(port->serial);
-
-       /* shutdown our urbs */
-       usb_kill_urb(priv->int_urb);
+       usb_kill_urb(port->interrupt_in_urb);
 }
 
 static void symbol_throttle(struct tty_struct *tty)
@@ -166,7 +146,7 @@ static void symbol_unthrottle(struct tty_struct *tty)
        spin_unlock_irq(&priv->lock);
 
        if (was_throttled) {
-               result = usb_submit_urb(priv->int_urb, GFP_KERNEL);
+               result = usb_submit_urb(port->interrupt_in_urb, GFP_KERNEL);
                if (result)
                        dev_err(&port->dev,
                                "%s - failed submitting read urb, error %d\n",
@@ -177,10 +157,11 @@ static void symbol_unthrottle(struct tty_struct *tty)
 static int symbol_startup(struct usb_serial *serial)
 {
        struct symbol_private *priv;
-       struct usb_host_interface *intf;
-       int i;
-       int retval = -ENOMEM;
-       bool int_in_found = false;
+
+       if (!serial->num_interrupt_in) {
+               dev_err(&serial->dev->dev, "no interrupt-in endpoint\n");
+               return -ENODEV;
+       }
 
        /* create our private serial structure */
        priv = kzalloc(sizeof(*priv), GFP_KERNEL);
@@ -189,75 +170,16 @@ static int symbol_startup(struct usb_serial *serial)
                return -ENOMEM;
        }
        spin_lock_init(&priv->lock);
-       priv->serial = serial;
-       priv->port = serial->port[0];
        priv->udev = serial->dev;
 
-       /* find our interrupt endpoint */
-       intf = serial->interface->altsetting;
-       for (i = 0; i < intf->desc.bNumEndpoints; ++i) {
-               struct usb_endpoint_descriptor *endpoint;
-
-               endpoint = &intf->endpoint[i].desc;
-               if (!usb_endpoint_is_int_in(endpoint))
-                       continue;
-
-               priv->int_urb = usb_alloc_urb(0, GFP_KERNEL);
-               if (!priv->int_urb) {
-                       dev_err(&priv->udev->dev, "out of memory\n");
-                       goto error;
-               }
-
-               priv->buffer_size = usb_endpoint_maxp(endpoint) * 2;
-               priv->int_buffer = kmalloc(priv->buffer_size, GFP_KERNEL);
-               if (!priv->int_buffer) {
-                       dev_err(&priv->udev->dev, "out of memory\n");
-                       goto error;
-               }
-
-               priv->int_address = endpoint->bEndpointAddress;
-               priv->bInterval = endpoint->bInterval;
-
-               /* set up our int urb */
-               usb_fill_int_urb(priv->int_urb, priv->udev,
-                                usb_rcvintpipe(priv->udev,
-                                               endpoint->bEndpointAddress),
-                                priv->int_buffer, priv->buffer_size,
-                                symbol_int_callback, priv, priv->bInterval);
-
-               int_in_found = true;
-               break;
-               }
-
-       if (!int_in_found) {
-               dev_err(&priv->udev->dev,
-                       "Error - the proper endpoints were not found!\n");
-               goto error;
-       }
-
        usb_set_serial_data(serial, priv);
        return 0;
-
-error:
-       usb_free_urb(priv->int_urb);
-       kfree(priv->int_buffer);
-       kfree(priv);
-       return retval;
-}
-
-static void symbol_disconnect(struct usb_serial *serial)
-{
-       struct symbol_private *priv = usb_get_serial_data(serial);
-
-       usb_kill_urb(priv->int_urb);
-       usb_free_urb(priv->int_urb);
 }
 
 static void symbol_release(struct usb_serial *serial)
 {
        struct symbol_private *priv = usb_get_serial_data(serial);
 
-       kfree(priv->int_buffer);
        kfree(priv);
 }
 
@@ -271,10 +193,10 @@ static struct usb_serial_driver symbol_device = {
        .attach =               symbol_startup,
        .open =                 symbol_open,
        .close =                symbol_close,
-       .disconnect =           symbol_disconnect,
        .release =              symbol_release,
        .throttle =             symbol_throttle,
        .unthrottle =           symbol_unthrottle,
+       .read_int_callback =    symbol_int_callback,
 };
 
 static struct usb_serial_driver * const serial_drivers[] = {