USB: serial: make bulk_out_size a lower limit
authorJohan Hovold <jhovold@gmail.com>
Wed, 12 Mar 2014 18:09:38 +0000 (19:09 +0100)
committerGreg Kroah-Hartman <greg@kroah.com>
Wed, 12 Mar 2014 19:44:49 +0000 (12:44 -0700)
Drivers are allowed to override the default bulk-out buffer size
(endpoint maximum packet size) in order to increase throughput, but it
does not make much sense to allow buffers smaller than the default.

Note that this is already how bulk_in_size is defined.

Signed-off-by: Johan Hovold <jhovold@gmail.com>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>
drivers/usb/serial/usb-serial.c
include/linux/usb/serial.h

index 7c9dc28640bb9654c9d3b6a6a31e70112ba69692..c68fc9fb7598797d93ed020361fda620309bb503 100644 (file)
@@ -923,9 +923,8 @@ static int usb_serial_probe(struct usb_interface *interface,
                port = serial->port[i];
                if (kfifo_alloc(&port->write_fifo, PAGE_SIZE, GFP_KERNEL))
                        goto probe_error;
-               buffer_size = serial->type->bulk_out_size;
-               if (!buffer_size)
-                       buffer_size = usb_endpoint_maxp(endpoint);
+               buffer_size = max_t(int, serial->type->bulk_out_size,
+                                               usb_endpoint_maxp(endpoint));
                port->bulk_out_size = buffer_size;
                port->bulk_out_endpointAddress = endpoint->bEndpointAddress;
 
index 704a1ab8240ca124f29c5ce361c871090d28ea5b..9bb547c7bce7c7ce0942fdb4e313188d4878b371 100644 (file)
@@ -190,7 +190,8 @@ static inline void usb_set_serial_data(struct usb_serial *serial, void *data)
  * @num_ports: the number of different ports this device will have.
  * @bulk_in_size: minimum number of bytes to allocate for bulk-in buffer
  *     (0 = end-point size)
- * @bulk_out_size: bytes to allocate for bulk-out buffer (0 = end-point size)
+ * @bulk_out_size: minimum number of bytes to allocate for bulk-out buffer
+ *     (0 = end-point size)
  * @calc_num_ports: pointer to a function to determine how many ports this
  *     device has dynamically.  It will be called after the probe()
  *     callback is called, but before attach()