[RAMEN9610-12100][COMMON] usb: gadget: remove unnecessary delay in functionfs bind
authorKisang Lee <kisang80.lee@samsung.com>
Wed, 13 Feb 2019 04:10:21 +0000 (13:10 +0900)
committerhskang <hs1218.kang@samsung.com>
Thu, 14 Feb 2019 12:36:02 +0000 (21:36 +0900)
Change-Id: I8bb6d6c26e505abbc8fe1fc493983d71410459cf
Signed-off-by: Kisang Lee <kisang80.lee@samsung.com>
drivers/usb/gadget/function/f_fs.c

index f4f5e5af360e9a56008c90108eadfb4ebd2075a9..2364b1cc6b60d52813296079a5cb1e8660e54274 100644 (file)
@@ -34,7 +34,6 @@
 #include <linux/mmu_context.h>
 #include <linux/poll.h>
 #include <linux/eventfd.h>
-#include <linux/delay.h>
 
 #include "u_fs.h"
 #include "u_f.h"
@@ -2920,7 +2919,6 @@ static inline struct f_fs_opts *ffs_do_functionfs_bind(struct usb_function *f,
        struct f_fs_opts *ffs_opts =
                container_of(f->fi, struct f_fs_opts, func_inst);
        int ret;
-       int retries = 500;
 
        ENTER();
 
@@ -2931,24 +2929,17 @@ static inline struct f_fs_opts *ffs_do_functionfs_bind(struct usb_function *f,
         *
         * Configfs-enabled gadgets however do need ffs_dev_lock.
         */
-       do {
-               if (!ffs_opts->no_configfs)
-                       ffs_dev_lock();
-               ret = ffs_opts->dev->desc_ready ? 0 : -ENODEV;
-               func->ffs = ffs_opts->dev->ffs_data;
-               if (!ffs_opts->no_configfs)
-                       ffs_dev_unlock();
-               if (ret)
-                       msleep(20);
-               else
-                       break;
-       } while (--retries);
-
-       pr_info("ffs_do_functionfs_bind %d %d\n", ret, retries);
-
+       if (!ffs_opts->no_configfs)
+               ffs_dev_lock();
+       ret = ffs_opts->dev->desc_ready ? 0 : -ENODEV;
+       func->ffs = ffs_opts->dev->ffs_data;
+       if (!ffs_opts->no_configfs)
+               ffs_dev_unlock();
        if (ret)
                return ERR_PTR(ret);
 
+       pr_info("KS : No waiting!!!\n");
+
        func->conf = c;
        func->gadget = c->cdev->gadget;