usb: gadget: g_serial: split the three possible functions into three bind functions
authorSebastian Andrzej Siewior <bigeasy@linutronix.de>
Sun, 23 Dec 2012 20:10:03 +0000 (21:10 +0100)
committerFelipe Balbi <balbi@ti.com>
Mon, 21 Jan 2013 18:52:41 +0000 (20:52 +0200)
This patch factors out the three possible functions into three possible
bind functions which are passed as an argument to usb_add_config(). This
will ease the step by step converting of the individual functions to the
new function registration method.

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/gadget/serial.c

index 44752f531e85e7d75ccf1698a3dd0bb482dfcbfc..bc23905800d3a3e2b79b718a18c37678d2abd4d6 100644 (file)
@@ -129,19 +129,33 @@ MODULE_PARM_DESC(n_ports, "number of ports to create, default=1");
 
 /*-------------------------------------------------------------------------*/
 
-static int __init serial_bind_config(struct usb_configuration *c)
+static int __init serial_bind_acm_config(struct usb_configuration *c)
 {
        unsigned i;
        int status = 0;
 
-       for (i = 0; i < n_ports && status == 0; i++) {
-               if (use_acm)
-                       status = acm_bind_config(c, i);
-               else if (use_obex)
-                       status = obex_bind_config(c, i);
-               else
-                       status = gser_bind_config(c, i);
-       }
+       for (i = 0; i < n_ports && status == 0; i++)
+               status = acm_bind_config(c, i);
+       return status;
+}
+
+static int __init serial_bind_obex_config(struct usb_configuration *c)
+{
+       unsigned i;
+       int status = 0;
+
+       for (i = 0; i < n_ports && status == 0; i++)
+               status = obex_bind_config(c, i);
+       return status;
+}
+
+static int __init serial_bind_gser_config(struct usb_configuration *c)
+{
+       unsigned i;
+       int status = 0;
+
+       for (i = 0; i < n_ports && status == 0; i++)
+               status = gser_bind_config(c, i);
        return status;
 }
 
@@ -178,8 +192,15 @@ static int __init gs_bind(struct usb_composite_dev *cdev)
        }
 
        /* register our configuration */
-       status = usb_add_config(cdev, &serial_config_driver,
-                       serial_bind_config);
+       if (use_acm)
+               status = usb_add_config(cdev, &serial_config_driver,
+                               serial_bind_acm_config);
+       else if (use_obex)
+               status = usb_add_config(cdev, &serial_config_driver,
+                               serial_bind_obex_config);
+       else
+               status = usb_add_config(cdev, &serial_config_driver,
+                               serial_bind_gser_config);
        if (status < 0)
                goto fail;