Use usb_otg_descriptor_add to allocate and initialize usb_otg_descriptor,
free it while composite unbind.

Signed-off-by: Li Jun <jun...@freescale.com>
---
 drivers/usb/gadget/legacy/g_ffs.c | 34 +++++++++++++++++++---------------
 1 file changed, 19 insertions(+), 15 deletions(-)

diff --git a/drivers/usb/gadget/legacy/g_ffs.c 
b/drivers/usb/gadget/legacy/g_ffs.c
index 7b9ef7e..f607bbe 100644
--- a/drivers/usb/gadget/legacy/g_ffs.c
+++ b/drivers/usb/gadget/legacy/g_ffs.c
@@ -88,21 +88,7 @@ MODULE_PARM_DESC(bDeviceProtocol, "USB Device protocol");
 module_param_array_named(functions, func_names, charp, &func_num, 0);
 MODULE_PARM_DESC(functions, "USB Functions list");
 
-static const struct usb_descriptor_header *gfs_otg_desc[] = {
-       (const struct usb_descriptor_header *)
-       &(const struct usb_otg_descriptor) {
-               .bLength                = sizeof(struct usb_otg_descriptor),
-               .bDescriptorType        = USB_DT_OTG,
-
-               /*
-                * REVISIT SRP-only hardware is possible, although
-                * it would not be called "OTG" ...
-                */
-               .bmAttributes           = USB_OTG_SRP | USB_OTG_HNP,
-       },
-
-       NULL
-};
+static const struct usb_descriptor_header *gfs_otg_desc[2];
 
 /* String IDs are assigned dynamically */
 static struct usb_string gfs_strings[] = {
@@ -471,6 +457,9 @@ static int gfs_unbind(struct usb_composite_dev *cdev)
        for (i = 0; i < N_CONF * func_num; ++i)
                usb_put_function(*(f_ffs[0] + i));
 
+       kfree(gfs_otg_desc[0]);
+       gfs_otg_desc[0] = NULL;
+
        return 0;
 }
 
@@ -489,6 +478,21 @@ static int gfs_do_config(struct usb_configuration *c)
                return -ENODEV;
 
        if (gadget_is_otg(c->cdev->gadget)) {
+               if (!gfs_otg_desc[0]) {
+                       struct usb_descriptor_header *usb_desc;
+                       unsigned length;
+
+                       if (c->cdev->gadget->otg_rev >= 0x0200)
+                               length = sizeof(struct usb_otg20_descriptor);
+                       else
+                               length = sizeof(struct usb_otg_descriptor);
+                       usb_desc = kzalloc(length, GFP_KERNEL);
+                       if (!usb_desc)
+                               return -ENOMEM;
+                       usb_otg_descriptor_init(c->cdev->gadget, usb_desc);
+                       gfs_otg_desc[0] = usb_desc;
+                       gfs_otg_desc[1] = NULL;
+               }
                c->descriptors = gfs_otg_desc;
                c->bmAttributes |= USB_CONFIG_ATT_WAKEUP;
        }
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to