usb: gadget: FunctionFS: fix typos and coding style
[platform/adaptation/renesas_rcar/renesas_kernel.git] / drivers / usb / gadget / g_ffs.c
index a9474f8..3dc1989 100644 (file)
@@ -1,7 +1,27 @@
+/*
+ * g_ffs.c -- user mode file system API for USB composite function controllers
+ *
+ * Copyright (C) 2010 Samsung Electronics
+ * Author: Michal Nazarewicz <m.nazarewicz@samsung.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+ */
+
 #include <linux/module.h>
 #include <linux/utsname.h>
 
-
 /*
  * kbuild is not very cooperative with respect to linking separately
  * compiled library objects into one module.  So for now we won't use
@@ -43,7 +63,6 @@ static int eth_bind_config(struct usb_configuration *c, u8 ethaddr[ETH_ALEN]);
 
 #include "f_fs.c"
 
-
 #define DRIVER_NAME    "g_ffs"
 #define DRIVER_DESC    "USB Function Filesystem"
 #define DRIVER_VERSION "24 Aug 2004"
@@ -52,9 +71,8 @@ MODULE_DESCRIPTION(DRIVER_DESC);
 MODULE_AUTHOR("Michal Nazarewicz");
 MODULE_LICENSE("GPL");
 
-
-static unsigned short gfs_vendor_id    = 0x0525;       /* XXX NetChip */
-static unsigned short gfs_product_id   = 0xa4ac;       /* XXX */
+#define GFS_VENDOR_ID  0x1d6b  /* Linux Foundation */
+#define GFS_PRODUCT_ID 0x0105  /* FunctionFS Gadget */
 
 static struct usb_device_descriptor gfs_dev_desc = {
        .bLength                = sizeof gfs_dev_desc,
@@ -63,31 +81,16 @@ static struct usb_device_descriptor gfs_dev_desc = {
        .bcdUSB                 = cpu_to_le16(0x0200),
        .bDeviceClass           = USB_CLASS_PER_INTERFACE,
 
-       /* Vendor and product id can be overridden by module parameters.  */
-       /* .idVendor            = cpu_to_le16(gfs_vendor_id), */
-       /* .idProduct           = cpu_to_le16(gfs_product_id), */
-       /* .bcdDevice           = f(hardware) */
-       /* .iManufacturer       = DYNAMIC */
-       /* .iProduct            = DYNAMIC */
-       /* NO SERIAL NUMBER */
-       .bNumConfigurations     = 1,
+       .idVendor               = cpu_to_le16(GFS_VENDOR_ID),
+       .idProduct              = cpu_to_le16(GFS_PRODUCT_ID),
 };
 
-#define GFS_MODULE_PARAM_DESC(name, field) \
-       MODULE_PARM_DESC(name, "Value of the " #field " field of the device descriptor sent to the host.  Takes effect only prior to the user-space driver registering to the FunctionFS.")
-
-module_param_named(usb_class,    gfs_dev_desc.bDeviceClass,    byte,   0644);
-GFS_MODULE_PARAM_DESC(usb_class, bDeviceClass);
-module_param_named(usb_subclass, gfs_dev_desc.bDeviceSubClass, byte,   0644);
-GFS_MODULE_PARAM_DESC(usb_subclass, bDeviceSubClass);
-module_param_named(usb_protocol, gfs_dev_desc.bDeviceProtocol, byte,   0644);
-GFS_MODULE_PARAM_DESC(usb_protocol, bDeviceProtocol);
-module_param_named(usb_vendor,   gfs_vendor_id,                ushort, 0644);
-GFS_MODULE_PARAM_DESC(usb_vendor, idVendor);
-module_param_named(usb_product,  gfs_product_id,               ushort, 0644);
-GFS_MODULE_PARAM_DESC(usb_product, idProduct);
-
-
+module_param_named(bDeviceClass,    gfs_dev_desc.bDeviceClass,    byte,   0644);
+MODULE_PARM_DESC(bDeviceClass, "USB Device class");
+module_param_named(bDeviceSubClass, gfs_dev_desc.bDeviceSubClass, byte,   0644);
+MODULE_PARM_DESC(bDeviceSubClass, "USB Device subclass");
+module_param_named(bDeviceProtocol, gfs_dev_desc.bDeviceProtocol, byte,   0644);
+MODULE_PARM_DESC(bDeviceProtocol, "USB Device protocol");
 
 static const struct usb_descriptor_header *gfs_otg_desc[] = {
        (const struct usb_descriptor_header *)
@@ -95,29 +98,18 @@ static const struct usb_descriptor_header *gfs_otg_desc[] = {
                .bLength                = sizeof(struct usb_otg_descriptor),
                .bDescriptorType        = USB_DT_OTG,
 
-               /* REVISIT SRP-only hardware is possible, although
-                * it would not be called "OTG" ... */
+               /*
+                * REVISIT SRP-only hardware is possible, although
+                * it would not be called "OTG" ...
+                */
                .bmAttributes           = USB_OTG_SRP | USB_OTG_HNP,
        },
 
        NULL
 };
 
-/* string IDs are assigned dynamically */
-
-enum {
-       GFS_STRING_MANUFACTURER_IDX,
-       GFS_STRING_PRODUCT_IDX,
-       GFS_STRING_FIRST_CONFIG_IDX,
-};
-
-static       char gfs_manufacturer[50];
-static const char gfs_driver_desc[] = DRIVER_DESC;
-static const char gfs_short_name[]  = DRIVER_NAME;
-
+/* String IDs are assigned dynamically */
 static struct usb_string gfs_strings[] = {
-       [GFS_STRING_MANUFACTURER_IDX].s = gfs_manufacturer,
-       [GFS_STRING_PRODUCT_IDX].s = gfs_driver_desc,
 #ifdef CONFIG_USB_FUNCTIONFS_RNDIS
        { .s = "FunctionFS + RNDIS" },
 #endif
@@ -138,8 +130,6 @@ static struct usb_gadget_strings *gfs_dev_strings[] = {
        NULL,
 };
 
-
-
 struct gfs_configuration {
        struct usb_configuration c;
        int (*eth)(struct usb_configuration *c, u8 *ethaddr);
@@ -162,24 +152,21 @@ struct gfs_configuration {
 #endif
 };
 
-
 static int gfs_bind(struct usb_composite_dev *cdev);
 static int gfs_unbind(struct usb_composite_dev *cdev);
 static int gfs_do_config(struct usb_configuration *c);
 
 static struct usb_composite_driver gfs_driver = {
-       .name           = gfs_short_name,
+       .name           = DRIVER_NAME,
        .dev            = &gfs_dev_desc,
        .strings        = gfs_dev_strings,
-       .bind           = gfs_bind,
        .unbind         = gfs_unbind,
+       .iProduct       = DRIVER_DESC,
 };
 
-
 static struct ffs_data *gfs_ffs_data;
 static unsigned long gfs_registered;
 
-
 static int  gfs_init(void)
 {
        ENTER();
@@ -199,7 +186,6 @@ static void  gfs_exit(void)
 }
 module_exit(gfs_exit);
 
-
 static int functionfs_ready_callback(struct ffs_data *ffs)
 {
        int ret;
@@ -210,7 +196,7 @@ static int functionfs_ready_callback(struct ffs_data *ffs)
                return -EBUSY;
 
        gfs_ffs_data = ffs;
-       ret = usb_composite_register(&gfs_driver);
+       ret = usb_composite_probe(&gfs_driver, gfs_bind);
        if (unlikely(ret < 0))
                clear_bit(0, &gfs_registered);
        return ret;
@@ -224,14 +210,11 @@ static void functionfs_closed_callback(struct ffs_data *ffs)
                usb_composite_unregister(&gfs_driver);
 }
 
-
 static int functionfs_check_dev_callback(const char *dev_name)
 {
        return 0;
 }
 
-
-
 static int gfs_bind(struct usb_composite_dev *cdev)
 {
        int ret, i;
@@ -245,20 +228,10 @@ static int gfs_bind(struct usb_composite_dev *cdev)
        if (unlikely(ret < 0))
                goto error_quick;
 
-       gfs_dev_desc.idVendor  = cpu_to_le16(gfs_vendor_id);
-       gfs_dev_desc.idProduct = cpu_to_le16(gfs_product_id);
-
-       snprintf(gfs_manufacturer, sizeof gfs_manufacturer, "%s %s with %s",
-                init_utsname()->sysname, init_utsname()->release,
-                cdev->gadget->name);
-
        ret = usb_string_ids_tab(cdev, gfs_strings);
        if (unlikely(ret < 0))
                goto error;
 
-       gfs_dev_desc.iManufacturer = gfs_strings[GFS_STRING_MANUFACTURER_IDX].id;
-       gfs_dev_desc.iProduct      = gfs_strings[GFS_STRING_PRODUCT_IDX].id;
-
        ret = functionfs_bind(gfs_ffs_data, cdev);
        if (unlikely(ret < 0))
                goto error;
@@ -266,14 +239,12 @@ static int gfs_bind(struct usb_composite_dev *cdev)
        for (i = 0; i < ARRAY_SIZE(gfs_configurations); ++i) {
                struct gfs_configuration *c = gfs_configurations + i;
 
-               ret = GFS_STRING_FIRST_CONFIG_IDX + i;
-               c->c.label                      = gfs_strings[ret].s;
-               c->c.iConfiguration             = gfs_strings[ret].id;
-               c->c.bind                       = gfs_do_config;
+               c->c.label                      = gfs_strings[i].s;
+               c->c.iConfiguration             = gfs_strings[i].id;
                c->c.bConfigurationValue        = 1 + i;
                c->c.bmAttributes               = USB_CONFIG_ATT_SELFPOWER;
 
-               ret = usb_add_config(cdev, &c->c);
+               ret = usb_add_config(cdev, &c->c, gfs_do_config);
                if (unlikely(ret < 0))
                        goto error_unbind;
        }
@@ -293,13 +264,14 @@ static int gfs_unbind(struct usb_composite_dev *cdev)
 {
        ENTER();
 
-       /* We may have been called in an error recovery frem
+       /*
+        * We may have been called in an error recovery from
         * composite_bind() after gfs_unbind() failure so we need to
         * check if gfs_ffs_data is not NULL since gfs_bind() handles
         * all error recovery itself.  I'd rather we werent called
         * from composite on orror recovery, but what you're gonna
-        * do...? */
-
+        * do...?
+        */
        if (gfs_ffs_data) {
                gether_cleanup();
                functionfs_unbind(gfs_ffs_data);
@@ -309,7 +281,6 @@ static int gfs_unbind(struct usb_composite_dev *cdev)
        return 0;
 }
 
-
 static int gfs_do_config(struct usb_configuration *c)
 {
        struct gfs_configuration *gc =
@@ -334,26 +305,29 @@ static int gfs_do_config(struct usb_configuration *c)
        if (unlikely(ret < 0))
                return ret;
 
-       /* After previous do_configs there may be some invalid
+       /*
+        * After previous do_configs there may be some invalid
         * pointers in c->interface array.  This happens every time
         * a user space function with fewer interfaces than a user
         * space function that was run before the new one is run.  The
         * compasit's set_config() assumes that if there is no more
         * then MAX_CONFIG_INTERFACES interfaces in a configuration
         * then there is a NULL pointer after the last interface in
-        * c->interface array.  We need to make sure this is true. */
+        * c->interface array.  We need to make sure this is true.
+        */
        if (c->next_interface_id < ARRAY_SIZE(c->interface))
                c->interface[c->next_interface_id] = NULL;
 
        return 0;
 }
 
-
 #ifdef CONFIG_USB_FUNCTIONFS_ETH
+
 static int eth_bind_config(struct usb_configuration *c, u8 ethaddr[ETH_ALEN])
 {
        return can_support_ecm(c->cdev->gadget)
                ? ecm_bind_config(c, ethaddr)
                : geth_bind_config(c, ethaddr);
 }
+
 #endif