usb: gadget: use config_ep_by_speed() instead of ep_choose()
authorTatyana Brokhman <tlinder@codeaurora.org>
Tue, 28 Jun 2011 13:33:50 +0000 (16:33 +0300)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 28 Jun 2011 18:14:37 +0000 (11:14 -0700)
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>
16 files changed:
drivers/usb/gadget/config.c
drivers/usb/gadget/f_acm.c
drivers/usb/gadget/f_ecm.c
drivers/usb/gadget/f_eem.c
drivers/usb/gadget/f_hid.c
drivers/usb/gadget/f_loopback.c
drivers/usb/gadget/f_mass_storage.c
drivers/usb/gadget/f_ncm.c
drivers/usb/gadget/f_obex.c
drivers/usb/gadget/f_phonet.c
drivers/usb/gadget/f_rndis.c
drivers/usb/gadget/f_serial.c
drivers/usb/gadget/f_sourcesink.c
drivers/usb/gadget/f_subset.c
include/linux/usb/composite.h
include/linux/usb/gadget.h

index 09084fd646abccbc5c5402b99dde831faaf63e3c..b2c00133487626dd1bbe211071bf3e2fa04bbbe6 100644 (file)
@@ -165,28 +165,3 @@ usb_copy_descriptors(struct usb_descriptor_header **src)
        return ret;
 }
 
-/**
- * usb_find_endpoint - find a copy of an endpoint descriptor
- * @src: original vector of descriptors
- * @copy: copy of @src
- * @match: endpoint descriptor found in @src
- *
- * This returns the copy of the @match descriptor made for @copy.  Its
- * intended use is to help remembering the endpoint descriptor to use
- * when enabling a given endpoint.
- */
-struct usb_endpoint_descriptor *
-usb_find_endpoint(
-       struct usb_descriptor_header **src,
-       struct usb_descriptor_header **copy,
-       struct usb_endpoint_descriptor *match
-)
-{
-       while (*src) {
-               if (*src == (void *) match)
-                       return (void *)*copy;
-               src++;
-               copy++;
-       }
-       return NULL;
-}
index d04b4a68220da1d2fd1010773d30e6530ff3545f..3f8849339adefe65bc5c2bc068cea34bc809407a 100644 (file)
  * descriptors (roughly equivalent to CDC Unions) may sometimes help.
  */
 
-struct acm_ep_descs {
-       struct usb_endpoint_descriptor  *in;
-       struct usb_endpoint_descriptor  *out;
-       struct usb_endpoint_descriptor  *notify;
-};
-
 struct f_acm {
        struct gserial                  port;
        u8                              ctrl_id, data_id;
@@ -58,9 +52,6 @@ struct f_acm {
         */
        spinlock_t                      lock;
 
-       struct acm_ep_descs             fs;
-       struct acm_ep_descs             hs;
-
        struct usb_ep                   *notify;
        struct usb_request              *notify_req;
 
@@ -404,9 +395,8 @@ static int acm_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
                        usb_ep_disable(acm->notify);
                } else {
                        VDBG(cdev, "init acm ctrl interface %d\n", intf);
-                       acm->notify->desc = ep_choose(cdev->gadget,
-                                       acm->hs.notify,
-                                       acm->fs.notify);
+                       if (config_ep_by_speed(cdev->gadget, f, acm->notify))
+                               return -EINVAL;
                }
                usb_ep_enable(acm->notify);
                acm->notify->driver_data = acm;
@@ -415,12 +405,17 @@ static int acm_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
                if (acm->port.in->driver_data) {
                        DBG(cdev, "reset acm ttyGS%d\n", acm->port_num);
                        gserial_disconnect(&acm->port);
-               } else {
+               }
+               if (!acm->port.in->desc || !acm->port.out->desc) {
                        DBG(cdev, "activate acm ttyGS%d\n", acm->port_num);
-                       acm->port.in->desc = ep_choose(cdev->gadget,
-                                       acm->hs.in, acm->fs.in);
-                       acm->port.out->desc = ep_choose(cdev->gadget,
-                                       acm->hs.out, acm->fs.out);
+                       if (config_ep_by_speed(cdev->gadget, f,
+                                              acm->port.in) ||
+                           config_ep_by_speed(cdev->gadget, f,
+                                              acm->port.out)) {
+                               acm->port.in->desc = NULL;
+                               acm->port.out->desc = NULL;
+                               return -EINVAL;
+                       }
                }
                gserial_connect(&acm->port, acm->port_num);
 
@@ -628,18 +623,11 @@ acm_bind(struct usb_configuration *c, struct usb_function *f)
        acm->notify_req->complete = acm_cdc_notify_complete;
        acm->notify_req->context = acm;
 
-       /* copy descriptors, and track endpoint copies */
+       /* copy descriptors */
        f->descriptors = usb_copy_descriptors(acm_fs_function);
        if (!f->descriptors)
                goto fail;
 
-       acm->fs.in = usb_find_endpoint(acm_fs_function,
-                       f->descriptors, &acm_fs_in_desc);
-       acm->fs.out = usb_find_endpoint(acm_fs_function,
-                       f->descriptors, &acm_fs_out_desc);
-       acm->fs.notify = usb_find_endpoint(acm_fs_function,
-                       f->descriptors, &acm_fs_notify_desc);
-
        /* support all relevant hardware speeds... we expect that when
         * hardware is dual speed, all bulk-capable endpoints work at
         * both speeds
@@ -652,15 +640,8 @@ acm_bind(struct usb_configuration *c, struct usb_function *f)
                acm_hs_notify_desc.bEndpointAddress =
                                acm_fs_notify_desc.bEndpointAddress;
 
-               /* copy descriptors, and track endpoint copies */
+               /* copy descriptors */
                f->hs_descriptors = usb_copy_descriptors(acm_hs_function);
-
-               acm->hs.in = usb_find_endpoint(acm_hs_function,
-                               f->hs_descriptors, &acm_hs_in_desc);
-               acm->hs.out = usb_find_endpoint(acm_hs_function,
-                               f->hs_descriptors, &acm_hs_out_desc);
-               acm->hs.notify = usb_find_endpoint(acm_hs_function,
-                               f->hs_descriptors, &acm_hs_notify_desc);
        }
 
        DBG(cdev, "acm ttyGS%d: %s speed IN/%s OUT/%s NOTIFY/%s\n",
index 7c996f271ccb7a4348b7cad95f6ec171d4218088..ddedbc83bc375ac12710e4eecd27406602e8482d 100644 (file)
  * and also means that a get_alt() method is required.
  */
 
-struct ecm_ep_descs {
-       struct usb_endpoint_descriptor  *in;
-       struct usb_endpoint_descriptor  *out;
-       struct usb_endpoint_descriptor  *notify;
-};
 
 enum ecm_notify_state {
        ECM_NOTIFY_NONE,                /* don't notify */
@@ -64,9 +59,6 @@ struct f_ecm {
 
        char                            ethaddr[14];
 
-       struct ecm_ep_descs             fs;
-       struct ecm_ep_descs             hs;
-
        struct usb_ep                   *notify;
        struct usb_request              *notify_req;
        u8                              notify_state;
@@ -463,11 +455,11 @@ static int ecm_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
                if (ecm->notify->driver_data) {
                        VDBG(cdev, "reset ecm control %d\n", intf);
                        usb_ep_disable(ecm->notify);
-               } else {
+               }
+               if (!(ecm->notify->desc)) {
                        VDBG(cdev, "init ecm ctrl %d\n", intf);
-                       ecm->notify->desc = ep_choose(cdev->gadget,
-                                       ecm->hs.notify,
-                                       ecm->fs.notify);
+                       if (config_ep_by_speed(cdev->gadget, f, ecm->notify))
+                               goto fail;
                }
                usb_ep_enable(ecm->notify);
                ecm->notify->driver_data = ecm;
@@ -482,12 +474,17 @@ static int ecm_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
                        gether_disconnect(&ecm->port);
                }
 
-               if (!ecm->port.in_ep->desc) {
+               if (!ecm->port.in_ep->desc ||
+                   !ecm->port.out_ep->desc) {
                        DBG(cdev, "init ecm\n");
-                       ecm->port.in_ep->desc = ep_choose(cdev->gadget,
-                                       ecm->hs.in, ecm->fs.in);
-                       ecm->port.out_ep->desc = ep_choose(cdev->gadget,
-                                       ecm->hs.out, ecm->fs.out);
+                       if (config_ep_by_speed(cdev->gadget, f,
+                                              ecm->port.in_ep) ||
+                           config_ep_by_speed(cdev->gadget, f,
+                                              ecm->port.out_ep)) {
+                               ecm->port.in_ep->desc = NULL;
+                               ecm->port.out_ep->desc = NULL;
+                               goto fail;
+                       }
                }
 
                /* CDC Ethernet only sends data in non-default altsettings.
@@ -664,13 +661,6 @@ ecm_bind(struct usb_configuration *c, struct usb_function *f)
        if (!f->descriptors)
                goto fail;
 
-       ecm->fs.in = usb_find_endpoint(ecm_fs_function,
-                       f->descriptors, &fs_ecm_in_desc);
-       ecm->fs.out = usb_find_endpoint(ecm_fs_function,
-                       f->descriptors, &fs_ecm_out_desc);
-       ecm->fs.notify = usb_find_endpoint(ecm_fs_function,
-                       f->descriptors, &fs_ecm_notify_desc);
-
        /* support all relevant hardware speeds... we expect that when
         * hardware is dual speed, all bulk-capable endpoints work at
         * both speeds
@@ -687,13 +677,6 @@ ecm_bind(struct usb_configuration *c, struct usb_function *f)
                f->hs_descriptors = usb_copy_descriptors(ecm_hs_function);
                if (!f->hs_descriptors)
                        goto fail;
-
-               ecm->hs.in = usb_find_endpoint(ecm_hs_function,
-                               f->hs_descriptors, &hs_ecm_in_desc);
-               ecm->hs.out = usb_find_endpoint(ecm_hs_function,
-                               f->hs_descriptors, &hs_ecm_out_desc);
-               ecm->hs.notify = usb_find_endpoint(ecm_hs_function,
-                               f->hs_descriptors, &hs_ecm_notify_desc);
        }
 
        /* NOTE:  all that is done without knowing or caring about
index fea8e3b08b5f12408ca6a0fec9fbfa585ee676a5..3e412740e2eff6184471e07b49f48dcd77892998 100644 (file)
  * Ethernet link.
  */
 
-struct eem_ep_descs {
-       struct usb_endpoint_descriptor  *in;
-       struct usb_endpoint_descriptor  *out;
-};
-
 struct f_eem {
        struct gether                   port;
        u8                              ctrl_id;
-
-       struct eem_ep_descs             fs;
-       struct eem_ep_descs             hs;
 };
 
 static inline struct f_eem *func_to_eem(struct usb_function *f)
@@ -176,12 +168,16 @@ static int eem_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
                        gether_disconnect(&eem->port);
                }
 
-               if (!eem->port.in_ep->desc) {
+               if (!eem->port.in_ep->desc || !eem->port.out_ep->desc) {
                        DBG(cdev, "init eem\n");
-                       eem->port.in_ep->desc = ep_choose(cdev->gadget,
-                                       eem->hs.in, eem->fs.in);
-                       eem->port.out_ep->desc = ep_choose(cdev->gadget,
-                                       eem->hs.out, eem->fs.out);
+                       if (config_ep_by_speed(cdev->gadget, f,
+                                              eem->port.in_ep) ||
+                           config_ep_by_speed(cdev->gadget, f,
+                                              eem->port.out_ep)) {
+                               eem->port.in_ep->desc = NULL;
+                               eem->port.out_ep->desc = NULL;
+                               goto fail;
+                       }
                }
 
                /* zlps should not occur because zero-length EEM packets
@@ -253,11 +249,6 @@ eem_bind(struct usb_configuration *c, struct usb_function *f)
        if (!f->descriptors)
                goto fail;
 
-       eem->fs.in = usb_find_endpoint(eem_fs_function,
-                       f->descriptors, &eem_fs_in_desc);
-       eem->fs.out = usb_find_endpoint(eem_fs_function,
-                       f->descriptors, &eem_fs_out_desc);
-
        /* support all relevant hardware speeds... we expect that when
         * hardware is dual speed, all bulk-capable endpoints work at
         * both speeds
@@ -272,11 +263,6 @@ eem_bind(struct usb_configuration *c, struct usb_function *f)
                f->hs_descriptors = usb_copy_descriptors(eem_hs_function);
                if (!f->hs_descriptors)
                        goto fail;
-
-               eem->hs.in = usb_find_endpoint(eem_hs_function,
-                               f->hs_descriptors, &eem_hs_in_desc);
-               eem->hs.out = usb_find_endpoint(eem_hs_function,
-                               f->hs_descriptors, &eem_hs_out_desc);
        }
 
        DBG(cdev, "CDC Ethernet (EEM): %s speed IN/%s OUT/%s\n",
index 12879b6f787c353bebddce11be0c34aa27e0ec39..403a48bcf5609befe4db39dc9fe35a5c3685a8fc 100644 (file)
@@ -59,8 +59,6 @@ struct f_hidg {
        struct cdev                     cdev;
        struct usb_function             func;
        struct usb_ep                   *in_ep;
-       struct usb_endpoint_descriptor  *fs_in_ep_desc;
-       struct usb_endpoint_descriptor  *hs_in_ep_desc;
 };
 
 static inline struct f_hidg *func_to_hidg(struct usb_function *f)
@@ -425,8 +423,12 @@ static int hidg_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
                if (hidg->in_ep->driver_data != NULL)
                        usb_ep_disable(hidg->in_ep);
 
-               hidg->in_ep->desc = ep_choose(f->config->cdev->gadget,
-                               hidg->hs_in_ep_desc, hidg->fs_in_ep_desc);
+               status = config_ep_by_speed(f->config->cdev->gadget, f,
+                                           hidg->in_ep);
+               if (status) {
+                       ERROR(cdev, "config_ep_by_speed FAILED!\n");
+                       goto fail;
+               }
                status = usb_ep_enable(hidg->in_ep);
                if (status < 0) {
                        ERROR(cdev, "Enable endpoint FAILED!\n");
@@ -497,21 +499,12 @@ static int __init hidg_bind(struct usb_configuration *c, struct usb_function *f)
        if (!f->descriptors)
                goto fail;
 
-       hidg->fs_in_ep_desc = usb_find_endpoint(hidg_fs_descriptors,
-                                               f->descriptors,
-                                               &hidg_fs_in_ep_desc);
-
        if (gadget_is_dualspeed(c->cdev->gadget)) {
                hidg_hs_in_ep_desc.bEndpointAddress =
                        hidg_fs_in_ep_desc.bEndpointAddress;
                f->hs_descriptors = usb_copy_descriptors(hidg_hs_descriptors);
                if (!f->hs_descriptors)
                        goto fail;
-               hidg->hs_in_ep_desc = usb_find_endpoint(hidg_hs_descriptors,
-                                                       f->hs_descriptors,
-                                                       &hidg_hs_in_ep_desc);
-       } else {
-               hidg->hs_in_ep_desc = NULL;
        }
 
        mutex_init(&hidg->lock);
index 34e3ccaf0884c6657534d05838d418a7fe14d419..375632659a03c9b6aaa6904b9a39ef132885e250 100644 (file)
@@ -256,8 +256,9 @@ enable_loopback(struct usb_composite_dev *cdev, struct f_loopback *loop)
 
        /* one endpoint writes data back IN to the host */
        ep = loop->in_ep;
-       ep->desc = ep_choose(cdev->gadget,
-                       &hs_loop_source_desc, &fs_loop_source_desc);
+       result = config_ep_by_speed(cdev->gadget, &(loop->function), ep);
+       if (result)
+               return result;
        result = usb_ep_enable(ep);
        if (result < 0)
                return result;
@@ -265,8 +266,10 @@ enable_loopback(struct usb_composite_dev *cdev, struct f_loopback *loop)
 
        /* one endpoint just reads OUT packets */
        ep = loop->out_ep;
-       ep->desc = ep_choose(cdev->gadget,
-                       &hs_loop_sink_desc, &fs_loop_sink_desc);
+       result = config_ep_by_speed(cdev->gadget, &(loop->function), ep);
+       if (result)
+               goto fail0;
+
        result = usb_ep_enable(ep);
        if (result < 0) {
 fail0:
index 4eee43412121b06ab617124c84a9d7c86ee003eb..5b933958200772179fc8c6b28d4cf8cf056fcf55 100644 (file)
@@ -2324,19 +2324,6 @@ static int get_next_command(struct fsg_common *common)
 
 /*-------------------------------------------------------------------------*/
 
-static int enable_endpoint(struct fsg_common *common, struct usb_ep *ep,
-               const struct usb_endpoint_descriptor *d)
-{
-       int     rc;
-
-       ep->driver_data = common;
-       ep->desc = (struct usb_endpoint_descriptor *)d;
-       rc = usb_ep_enable(ep);
-       if (rc)
-               ERROR(common, "can't enable %s, result %d\n", ep->name, rc);
-       return rc;
-}
-
 static int alloc_request(struct fsg_common *common, struct usb_ep *ep,
                struct usb_request **preq)
 {
@@ -2350,7 +2337,6 @@ static int alloc_request(struct fsg_common *common, struct usb_ep *ep,
 /* Reset interface setting and re-init endpoint state (toggle etc). */
 static int do_set_interface(struct fsg_common *common, struct fsg_dev *new_fsg)
 {
-       const struct usb_endpoint_descriptor *d;
        struct fsg_dev *fsg;
        int i, rc = 0;
 
@@ -2397,20 +2383,26 @@ reset:
        fsg = common->fsg;
 
        /* Enable the endpoints */
-       d = fsg_ep_desc(common->gadget,
-                       &fsg_fs_bulk_in_desc, &fsg_hs_bulk_in_desc);
-       rc = enable_endpoint(common, fsg->bulk_in, d);
+       rc = config_ep_by_speed(common->gadget, &(fsg->function), fsg->bulk_in);
+       if (rc)
+               goto reset;
+       rc = usb_ep_enable(fsg->bulk_in);
        if (rc)
                goto reset;
+       fsg->bulk_in->driver_data = common;
        fsg->bulk_in_enabled = 1;
 
-       d = fsg_ep_desc(common->gadget,
-                       &fsg_fs_bulk_out_desc, &fsg_hs_bulk_out_desc);
-       rc = enable_endpoint(common, fsg->bulk_out, d);
+       rc = config_ep_by_speed(common->gadget, &(fsg->function),
+                               fsg->bulk_out);
+       if (rc)
+               goto reset;
+       rc = usb_ep_enable(fsg->bulk_out);
        if (rc)
                goto reset;
+       fsg->bulk_out->driver_data = common;
        fsg->bulk_out_enabled = 1;
-       common->bulk_out_maxpacket = le16_to_cpu(d->wMaxPacketSize);
+       common->bulk_out_maxpacket =
+               le16_to_cpu(fsg->bulk_out->desc->wMaxPacketSize);
        clear_bit(IGNORE_BULK_OUT, &fsg->atomic_bitflags);
 
        /* Allocate the requests */
index 06daa1bc9ffd9300599c0dbc73a19f5d89ba1422..ae69ed7e6b99a8f4c6d2bfd71f61511dad5671fd 100644 (file)
 #define NCM_NDP_HDR_CRC                0x01000000
 #define NCM_NDP_HDR_NOCRC      0x00000000
 
-struct ncm_ep_descs {
-       struct usb_endpoint_descriptor  *in;
-       struct usb_endpoint_descriptor  *out;
-       struct usb_endpoint_descriptor  *notify;
-};
-
 enum ncm_notify_state {
        NCM_NOTIFY_NONE,                /* don't notify */
        NCM_NOTIFY_CONNECT,             /* issue CONNECT next */
@@ -66,9 +60,6 @@ struct f_ncm {
 
        char                            ethaddr[14];
 
-       struct ncm_ep_descs             fs;
-       struct ncm_ep_descs             hs;
-
        struct usb_ep                   *notify;
        struct usb_request              *notify_req;
        u8                              notify_state;
@@ -801,11 +792,12 @@ static int ncm_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
                if (ncm->notify->driver_data) {
                        DBG(cdev, "reset ncm control %d\n", intf);
                        usb_ep_disable(ncm->notify);
-               } else {
+               }
+
+               if (!(ncm->notify->desc)) {
                        DBG(cdev, "init ncm ctrl %d\n", intf);
-                       ncm->notify->desc = ep_choose(cdev->gadget,
-                                       ncm->hs.notify,
-                                       ncm->fs.notify);
+                       if (config_ep_by_speed(cdev->gadget, f, ncm->notify))
+                               goto fail;
                }
                usb_ep_enable(ncm->notify);
                ncm->notify->driver_data = ncm;
@@ -828,14 +820,17 @@ static int ncm_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
                if (alt == 1) {
                        struct net_device       *net;
 
-                       if (!ncm->port.in_ep->desc) {
+                       if (!ncm->port.in_ep->desc ||
+                           !ncm->port.out_ep->desc) {
                                DBG(cdev, "init ncm\n");
-                               ncm->port.in_ep->desc = ep_choose(cdev->gadget,
-                                                        ncm->hs.in,
-                                                        ncm->fs.in);
-                               ncm->port.out_ep->desc = ep_choose(cdev->gadget,
-                                                         ncm->hs.out,
-                                                         ncm->fs.out);
+                               if (config_ep_by_speed(cdev->gadget, f,
+                                                      ncm->port.in_ep) ||
+                                   config_ep_by_speed(cdev->gadget, f,
+                                                      ncm->port.out_ep)) {
+                                       ncm->port.in_ep->desc = NULL;
+                                       ncm->port.out_ep->desc = NULL;
+                                       goto fail;
+                               }
                        }
 
                        /* TODO */
@@ -1227,13 +1222,6 @@ ncm_bind(struct usb_configuration *c, struct usb_function *f)
        if (!f->descriptors)
                goto fail;
 
-       ncm->fs.in = usb_find_endpoint(ncm_fs_function,
-                       f->descriptors, &fs_ncm_in_desc);
-       ncm->fs.out = usb_find_endpoint(ncm_fs_function,
-                       f->descriptors, &fs_ncm_out_desc);
-       ncm->fs.notify = usb_find_endpoint(ncm_fs_function,
-                       f->descriptors, &fs_ncm_notify_desc);
-
        /*
         * support all relevant hardware speeds... we expect that when
         * hardware is dual speed, all bulk-capable endpoints work at
@@ -1251,13 +1239,6 @@ ncm_bind(struct usb_configuration *c, struct usb_function *f)
                f->hs_descriptors = usb_copy_descriptors(ncm_hs_function);
                if (!f->hs_descriptors)
                        goto fail;
-
-               ncm->hs.in = usb_find_endpoint(ncm_hs_function,
-                               f->hs_descriptors, &hs_ncm_in_desc);
-               ncm->hs.out = usb_find_endpoint(ncm_hs_function,
-                               f->hs_descriptors, &hs_ncm_out_desc);
-               ncm->hs.notify = usb_find_endpoint(ncm_hs_function,
-                               f->hs_descriptors, &hs_ncm_notify_desc);
        }
 
        /*
index a6dbda090ca56c58fcd4f4896da9b3b6e493c825..394502abeb964a330130f981f0e9afd80d726930 100644 (file)
  * ready to handle the commands.
  */
 
-struct obex_ep_descs {
-       struct usb_endpoint_descriptor  *obex_in;
-       struct usb_endpoint_descriptor  *obex_out;
-};
-
 struct f_obex {
        struct gserial                  port;
        u8                              ctrl_id;
        u8                              data_id;
        u8                              port_num;
        u8                              can_activate;
-
-       struct obex_ep_descs            fs;
-       struct obex_ep_descs            hs;
 };
 
 static inline struct f_obex *func_to_obex(struct usb_function *f)
@@ -227,12 +219,16 @@ static int obex_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
                        gserial_disconnect(&obex->port);
                }
 
-               if (!obex->port.in->desc) {
+               if (!obex->port.in->desc || !obex->port.out->desc) {
                        DBG(cdev, "init obex ttyGS%d\n", obex->port_num);
-                       obex->port.in->desc = ep_choose(cdev->gadget,
-                                       obex->hs.obex_in, obex->fs.obex_in);
-                       obex->port.out->desc = ep_choose(cdev->gadget,
-                                       obex->hs.obex_out, obex->fs.obex_out);
+                       if (config_ep_by_speed(cdev->gadget, f,
+                                              obex->port.in) ||
+                           config_ep_by_speed(cdev->gadget, f,
+                                              obex->port.out)) {
+                               obex->port.out->desc = NULL;
+                               obex->port.in->desc = NULL;
+                               goto fail;
+                       }
                }
 
                if (alt == 1) {
@@ -346,11 +342,6 @@ obex_bind(struct usb_configuration *c, struct usb_function *f)
        /* copy descriptors, and track endpoint copies */
        f->descriptors = usb_copy_descriptors(fs_function);
 
-       obex->fs.obex_in = usb_find_endpoint(fs_function,
-                       f->descriptors, &obex_fs_ep_in_desc);
-       obex->fs.obex_out = usb_find_endpoint(fs_function,
-                       f->descriptors, &obex_fs_ep_out_desc);
-
        /* support all relevant hardware speeds... we expect that when
         * hardware is dual speed, all bulk-capable endpoints work at
         * both speeds
@@ -364,11 +355,6 @@ obex_bind(struct usb_configuration *c, struct usb_function *f)
 
                /* copy descriptors, and track endpoint copies */
                f->hs_descriptors = usb_copy_descriptors(hs_function);
-
-               obex->hs.obex_in = usb_find_endpoint(hs_function,
-                               f->hs_descriptors, &obex_hs_ep_in_desc);
-               obex->hs.obex_out = usb_find_endpoint(hs_function,
-                               f->hs_descriptors, &obex_hs_ep_out_desc);
        }
 
        /* Avoid letting this gadget enumerate until the userspace
index dc63f161a10a9e5973d1481fc932d6a14e83c01c..0d6d26090da90e617500bf913bb79488c58ed7a1 100644 (file)
@@ -429,12 +429,12 @@ static int pn_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
                if (alt == 1) {
                        int i;
 
-                       fp->out_ep->desc = ep_choose(gadget,
-                                       &pn_hs_sink_desc,
-                                       &pn_fs_sink_desc);
-                       fp->in_ep->desc = ep_choose(gadget,
-                                       &pn_hs_source_desc,
-                                       &pn_fs_source_desc);
+                       if (config_ep_by_speed(gadget, f, fp->in_ep) ||
+                           config_ep_by_speed(gadget, f, fp->out_ep)) {
+                               fp->in_ep->desc = NULL;
+                               fp->out_ep->desc = NULL;
+                               return -EINVAL;
+                       }
                        usb_ep_enable(fp->out_ep);
                        usb_ep_enable(fp->in_ep);
 
index 4646254a892423e7e4c020448cec90b88e9c3113..b324efa0773382c90e70bdc009cce3d8522b5ae7 100644 (file)
  *   - MS-Windows drivers sometimes emit undocumented requests.
  */
 
-struct rndis_ep_descs {
-       struct usb_endpoint_descriptor  *in;
-       struct usb_endpoint_descriptor  *out;
-       struct usb_endpoint_descriptor  *notify;
-};
-
 struct f_rndis {
        struct gether                   port;
        u8                              ctrl_id, data_id;
        u8                              ethaddr[ETH_ALEN];
        int                             config;
 
-       struct rndis_ep_descs           fs;
-       struct rndis_ep_descs           hs;
-
        struct usb_ep                   *notify;
        struct usb_request              *notify_req;
        atomic_t                        notify_count;
@@ -483,11 +474,11 @@ static int rndis_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
                if (rndis->notify->driver_data) {
                        VDBG(cdev, "reset rndis control %d\n", intf);
                        usb_ep_disable(rndis->notify);
-               } else {
+               }
+               if (!rndis->notify->desc) {
                        VDBG(cdev, "init rndis ctrl %d\n", intf);
-                       rndis->notify->desc = ep_choose(cdev->gadget,
-                                       rndis->hs.notify,
-                                       rndis->fs.notify);
+                       if (config_ep_by_speed(cdev->gadget, f, rndis->notify))
+                               goto fail;
                }
                usb_ep_enable(rndis->notify);
                rndis->notify->driver_data = rndis;
@@ -500,12 +491,16 @@ static int rndis_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
                        gether_disconnect(&rndis->port);
                }
 
-               if (!rndis->port.in_ep->desc) {
+               if (!rndis->port.in_ep->desc || !rndis->port.out_ep->desc) {
                        DBG(cdev, "init rndis\n");
-                       rndis->port.in_ep->desc = ep_choose(cdev->gadget,
-                                       rndis->hs.in, rndis->fs.in);
-                       rndis->port.out_ep->desc = ep_choose(cdev->gadget,
-                                       rndis->hs.out, rndis->fs.out);
+                       if (config_ep_by_speed(cdev->gadget, f,
+                                              rndis->port.in_ep) ||
+                           config_ep_by_speed(cdev->gadget, f,
+                                              rndis->port.out_ep)) {
+                               rndis->port.in_ep->desc = NULL;
+                               rndis->port.out_ep->desc = NULL;
+                               goto fail;
+                       }
                }
 
                /* Avoid ZLPs; they can be troublesome. */
@@ -661,13 +656,6 @@ rndis_bind(struct usb_configuration *c, struct usb_function *f)
        if (!f->descriptors)
                goto fail;
 
-       rndis->fs.in = usb_find_endpoint(eth_fs_function,
-                       f->descriptors, &fs_in_desc);
-       rndis->fs.out = usb_find_endpoint(eth_fs_function,
-                       f->descriptors, &fs_out_desc);
-       rndis->fs.notify = usb_find_endpoint(eth_fs_function,
-                       f->descriptors, &fs_notify_desc);
-
        /* support all relevant hardware speeds... we expect that when
         * hardware is dual speed, all bulk-capable endpoints work at
         * both speeds
@@ -685,13 +673,6 @@ rndis_bind(struct usb_configuration *c, struct usb_function *f)
 
                if (!f->hs_descriptors)
                        goto fail;
-
-               rndis->hs.in = usb_find_endpoint(eth_hs_function,
-                               f->hs_descriptors, &hs_in_desc);
-               rndis->hs.out = usb_find_endpoint(eth_hs_function,
-                               f->hs_descriptors, &hs_out_desc);
-               rndis->hs.notify = usb_find_endpoint(eth_hs_function,
-                               f->hs_descriptors, &hs_notify_desc);
        }
 
        rndis->port.open = rndis_open;
index a9ce6261d1567285307efd94f5f4459682b7357e..91fdf790ed20b122bf0a13df0d3c8aed5285ac3c 100644 (file)
  * 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",
index 0ffddd33a55c306f2f9d78c26ccd469637b26b3a..caf2f95e0346e6ea841fa3f5d59296bb0ac26147 100644 (file)
@@ -347,7 +347,9 @@ enable_source_sink(struct usb_composite_dev *cdev, struct f_sourcesink *ss)
 
        /* one endpoint writes (sources) zeroes IN (to the host) */
        ep = ss->in_ep;
-       ep->desc = ep_choose(cdev->gadget, &hs_source_desc, &fs_source_desc);
+       result = config_ep_by_speed(cdev->gadget, &(ss->function), ep);
+       if (result)
+               return result;
        result = usb_ep_enable(ep);
        if (result < 0)
                return result;
@@ -364,7 +366,9 @@ fail:
 
        /* one endpoint reads (sinks) anything OUT (from the host) */
        ep = ss->out_ep;
-       ep->desc = ep_choose(cdev->gadget, &hs_sink_desc, &fs_sink_desc);
+       result = config_ep_by_speed(cdev->gadget, &(ss->function), ep);
+       if (result)
+               goto fail;
        result = usb_ep_enable(ep);
        if (result < 0)
                goto fail;
index aecaed1724aa167ac1a45f6464f9fb6a1f033a22..93bf676ef50733328ef36203e7c3f2f3ce351c1a 100644 (file)
  * caring about specific product and vendor IDs.
  */
 
-struct geth_descs {
-       struct usb_endpoint_descriptor  *in;
-       struct usb_endpoint_descriptor  *out;
-};
-
 struct f_gether {
        struct gether                   port;
 
        char                            ethaddr[14];
-
-       struct geth_descs               fs;
-       struct geth_descs               hs;
 };
 
 static inline struct f_gether *func_to_geth(struct usb_function *f)
@@ -243,10 +235,12 @@ static int geth_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
        }
 
        DBG(cdev, "init + activate cdc subset\n");
-       geth->port.in_ep->desc = ep_choose(cdev->gadget,
-                       geth->hs.in, geth->fs.in);
-       geth->port.out_ep->desc = ep_choose(cdev->gadget,
-                       geth->hs.out, geth->fs.out);
+       if (config_ep_by_speed(cdev->gadget, f, geth->port.in_ep) ||
+           config_ep_by_speed(cdev->gadget, f, geth->port.out_ep)) {
+               geth->port.in_ep->desc = NULL;
+               geth->port.out_ep->desc = NULL;
+               return -EINVAL;
+       }
 
        net = gether_connect(&geth->port);
        return IS_ERR(net) ? PTR_ERR(net) : 0;
@@ -297,12 +291,6 @@ geth_bind(struct usb_configuration *c, struct usb_function *f)
        /* copy descriptors, and track endpoint copies */
        f->descriptors = usb_copy_descriptors(fs_eth_function);
 
-       geth->fs.in = usb_find_endpoint(fs_eth_function,
-                       f->descriptors, &fs_subset_in_desc);
-       geth->fs.out = usb_find_endpoint(fs_eth_function,
-                       f->descriptors, &fs_subset_out_desc);
-
-
        /* support all relevant hardware speeds... we expect that when
         * hardware is dual speed, all bulk-capable endpoints work at
         * both speeds
@@ -315,11 +303,6 @@ geth_bind(struct usb_configuration *c, struct usb_function *f)
 
                /* copy descriptors, and track endpoint copies */
                f->hs_descriptors = usb_copy_descriptors(hs_eth_function);
-
-               geth->hs.in = usb_find_endpoint(hs_eth_function,
-                               f->hs_descriptors, &hs_subset_in_desc);
-               geth->hs.out = usb_find_endpoint(hs_eth_function,
-                               f->hs_descriptors, &hs_subset_out_desc);
        }
 
        /* NOTE:  all that is done without knowing or caring about
index 2014d6b1babc0f7251cafcd1d43f09e87ffb5471..99830d63d8f1d2f06d82f2cbdbb8a6a2fa7af66f 100644 (file)
@@ -148,21 +148,6 @@ int usb_interface_id(struct usb_configuration *, struct usb_function *);
 int config_ep_by_speed(struct usb_gadget *g, struct usb_function *f,
                        struct usb_ep *_ep);
 
-/**
- * ep_choose - select descriptor endpoint at current device speed
- * @g: gadget, connected and running at some speed
- * @hs: descriptor to use for high speed operation
- * @fs: descriptor to use for full or low speed operation
- */
-static inline struct usb_endpoint_descriptor *
-ep_choose(struct usb_gadget *g, struct usb_endpoint_descriptor *hs,
-               struct usb_endpoint_descriptor *fs)
-{
-       if (gadget_is_dualspeed(g) && g->speed == USB_SPEED_HIGH)
-               return hs;
-       return fs;
-}
-
 #define        MAX_CONFIG_INTERFACES           16      /* arbitrary; max 255 */
 
 /**
index 0bcc2b76bcd8067de85f81f2aaf35102b0c86542..d22b4a4936cbd4898633a192dbc02a70b04a01d0 100644 (file)
@@ -879,12 +879,6 @@ int usb_gadget_config_buf(const struct usb_config_descriptor *config,
 struct usb_descriptor_header **usb_copy_descriptors(
                struct usb_descriptor_header **);
 
-/* return copy of endpoint descriptor given original descriptor set */
-struct usb_endpoint_descriptor *usb_find_endpoint(
-       struct usb_descriptor_header **src,
-       struct usb_descriptor_header **copy,
-       struct usb_endpoint_descriptor *match);
-
 /**
  * usb_free_descriptors - free descriptors returned by usb_copy_descriptors()
  * @v: vector of descriptors