[NET] sunhme: Kill useless loop over sdevs in quattro_sbus_find().
authorDavid S. Miller <davem@sunset.davemloft.net>
Sat, 24 Jun 2006 04:36:46 +0000 (21:36 -0700)
committerDavid S. Miller <davem@sunset.davemloft.net>
Sat, 24 Jun 2006 06:16:11 +0000 (23:16 -0700)
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/sunhme.c

index ec51f397e1ed526ab8c00c0c6db0fa6b1c7b8fec..c33ead3470db718886007204291a5fdc0861ad2b 100644 (file)
@@ -2568,14 +2568,10 @@ static void __init quattro_apply_ranges(struct quattro *qp, struct happy_meal *h
  */
 static struct quattro * __init quattro_sbus_find(struct sbus_dev *goal_sdev)
 {
-       struct sbus_bus *sbus;
        struct sbus_dev *sdev;
        struct quattro *qp;
        int i;
 
-       if (qfe_sbus_list == NULL)
-               goto found;
-
        for (qp = qfe_sbus_list; qp != NULL; qp = qp->next) {
                for (i = 0, sdev = qp->quattro_dev;
                     (sdev != NULL) && (i < 4);
@@ -2584,17 +2580,7 @@ static struct quattro * __init quattro_sbus_find(struct sbus_dev *goal_sdev)
                                return qp;
                }
        }
-       for_each_sbus(sbus) {
-               for_each_sbusdev(sdev, sbus) {
-                       if (sdev == goal_sdev)
-                               goto found;
-               }
-       }
-
-       /* Cannot find quattro parent, fail. */
-       return NULL;
 
-found:
        qp = kmalloc(sizeof(struct quattro), GFP_KERNEL);
        if (qp != NULL) {
                int i;