[SUNLANCE]: Fix probing problem.
authorKrzysztof Helt <krzysztof.h1@wp.pl>
Tue, 29 Aug 2006 06:28:16 +0000 (23:28 -0700)
committerDavid S. Miller <davem@sunset.davemloft.net>
Wed, 30 Aug 2006 04:23:32 +0000 (21:23 -0700)
The current probe table causes ledma and lebuffer
"le" devices to get probed twice which is not what
we want.

Match just "le" and look directly at the parent to get the correct
top-level node information.

Signed-off-by: Krzysztof Helt <krzysztof.h1@wp.pl>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/sunlance.c

index 0e3fdf7c6dd3a8cf321a063d61876a39d58b2c1d..ec0413609f36fd6b4bc8175f64e06bd330bb4157 100644 (file)
@@ -1566,20 +1566,21 @@ static int __exit sunlance_sun4_remove(void)
 static int __devinit sunlance_sbus_probe(struct of_device *dev, const struct of_device_id *match)
 {
        struct sbus_dev *sdev = to_sbus_device(&dev->dev);
-       struct device_node *dp = dev->node;
        int err;
 
-       if (!strcmp(dp->name, "le")) {
-               err = sparc_lance_probe_one(sdev, NULL, NULL);
-       } else if (!strcmp(dp->name, "ledma")) {
-               struct sbus_dma *ledma = find_ledma(sdev);
+       if (sdev->parent) {
+               struct of_device *parent = &sdev->parent->ofdev;
 
-               err = sparc_lance_probe_one(sdev->child, ledma, NULL);
-       } else {
-               BUG_ON(strcmp(dp->name, "lebuffer"));
+               if (!strcmp(parent->node->name, "ledma")) {
+                       struct sbus_dma *ledma = find_ledma(to_sbus_device(&parent->dev));
 
-               err = sparc_lance_probe_one(sdev->child, NULL, sdev);
-       }
+                       err = sparc_lance_probe_one(sdev, ledma, NULL);
+               } else if (!strcmp(parent->node->name, "lebuffer")) {
+                       err = sparc_lance_probe_one(sdev, NULL, to_sbus_device(&parent->dev));
+               } else
+                       err = sparc_lance_probe_one(sdev, NULL, NULL);
+       } else
+               err = sparc_lance_probe_one(sdev, NULL, NULL);
 
        return err;
 }
@@ -1604,12 +1605,6 @@ static struct of_device_id sunlance_sbus_match[] = {
        {
                .name = "le",
        },
-       {
-               .name = "ledma",
-       },
-       {
-               .name = "lebuffer",
-       },
        {},
 };