usb: gadget: use config_ep_by_speed() instead of ep_choose()

Remove obsolete functions:
1. ep_choose()
2. usb_find_endpoint()

Signed-off-by: Tatyana Brokhman <tlinder@codeaurora.org>
Signed-off-by: Felipe Balbi <balbi@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

1 files changed, 8 insertions, 24 deletions

diff --git a/drivers/usb/gadget/f_serial.c b/drivers/usb/gadget/f_serial.c
index a9ce6261d156..91fdf790ed20 100644
--- a/drivers/usb/gadget/f_serial.c
+++ b/drivers/usb/gadget/f_serial.c
@@ -27,18 +27,10 @@
  * if you can arrange appropriate host side drivers.
  */
 
-struct gser_descs {
-        struct usb_endpoint_descriptor  *in;
-        struct usb_endpoint_descriptor  *out;
-};
-
 struct f_gser {
         struct gserial                  port;
         u8                              data_id;
         u8                              port_num;
-
-        struct gser_descs               fs;
-        struct gser_descs               hs;
 };
 
 static inline struct f_gser *func_to_gser(struct usb_function *f)
@@ -136,12 +128,15 @@ static int gser_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
         if (gser->port.in->driver_data) {
                 DBG(cdev, "reset generic ttyGS%d\n", gser->port_num);
                 gserial_disconnect(&gser->port);
-        } else {
+        }
+        if (!gser->port.in->desc || !gser->port.out->desc) {
                 DBG(cdev, "activate generic ttyGS%d\n", gser->port_num);
-                gser->port.in->desc = ep_choose(cdev->gadget,
-                                gser->hs.in, gser->fs.in);
-                gser->port.out->desc = ep_choose(cdev->gadget,
-                                gser->hs.out, gser->fs.out);
+                if (!config_ep_by_speed(cdev->gadget, f, gser->port.in) ||
+                    !config_ep_by_speed(cdev->gadget, f, gser->port.out)) {
+                        gser->port.in->desc = NULL;
+                        gser->port.out->desc = NULL;
+                        return -EINVAL;
+                }
         }
         gserial_connect(&gser->port, gser->port_num);
         return 0;
@@ -193,12 +188,6 @@ gser_bind(struct usb_configuration *c, struct usb_function *f)
         /* copy descriptors, and track endpoint copies */
         f->descriptors = usb_copy_descriptors(gser_fs_function);
 
-        gser->fs.in = usb_find_endpoint(gser_fs_function,
-                        f->descriptors, &gser_fs_in_desc);
-        gser->fs.out = usb_find_endpoint(gser_fs_function,
-                        f->descriptors, &gser_fs_out_desc);
-
-
         /* support all relevant hardware speeds... we expect that when
          * hardware is dual speed, all bulk-capable endpoints work at
          * both speeds
@@ -211,11 +200,6 @@ gser_bind(struct usb_configuration *c, struct usb_function *f)
 
                 /* copy descriptors, and track endpoint copies */
                 f->hs_descriptors = usb_copy_descriptors(gser_hs_function);
-
-                gser->hs.in = usb_find_endpoint(gser_hs_function,
-                                f->hs_descriptors, &gser_hs_in_desc);
-                gser->hs.out = usb_find_endpoint(gser_hs_function,
-                                f->hs_descriptors, &gser_hs_out_desc);
         }
 
         DBG(cdev, "generic ttyGS%d: %s speed IN/%s OUT/%s\n",