From: Márton Németh <nm...@freemail.hu>

Eliminate redundant code by reorganizing the loop.

Signed-off-by: Márton Németh <nm...@freemail.hu>
---
diff -r 064a82aa2daa linux/drivers/media/video/gspca/gspca.c
--- a/linux/drivers/media/video/gspca/gspca.c   Thu Nov 26 19:36:40 2009 +0100
+++ b/linux/drivers/media/video/gspca/gspca.c   Sun Nov 29 11:09:33 2009 +0100
@@ -623,12 +623,12 @@
                if (ret < 0)
                        goto out;
        }
-       ep = get_ep(gspca_dev);
-       if (ep == NULL) {
-               ret = -EIO;
-               goto out;
-       }
        for (;;) {
+               ep = get_ep(gspca_dev);
+               if (ep == NULL) {
+                       ret = -EIO;
+                       goto out;
+               }
                PDEBUG(D_STREAM, "init transfer alt %d", gspca_dev->alt);
                ret = create_urbs(gspca_dev, ep);
                if (ret < 0)
@@ -677,12 +677,6 @@
                        ret = gspca_dev->sd_desc->isoc_nego(gspca_dev);
                        if (ret < 0)
                                goto out;
-               } else {
-                       ep = get_ep(gspca_dev);
-                       if (ep == NULL) {
-                               ret = -EIO;
-                               goto out;
-                       }
                }
        }
 out:
--
To unsubscribe from this list: send the line "unsubscribe linux-media" 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